mikepenz/Android-Iconics

Some icons are incorrectly rendered with `IconicsTextView` + XML

Open

#592 opened on Jul 17, 2021

View on GitHub
 (13 comments) (0 reactions) (1 assignee)Kotlin (613 forks)batch import
help wanted

Repository metrics

Stars
 (5,277 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

About this issue

Some icons are incorrectly rendered with IconicsTextView + XML.

  • Pixel C emulator: image

  • Samsung Galaxy S20 Ultra: image

Sample code:

<com.mikepenz.iconics.view.IconicsTextView
  android:layout_width="match_parent"
  android:layout_height="wrap_content"
  android:text="@string/title_statistics_chart_success"
  android:textAppearance="?attr/textAppearanceBody2" />

<string name="title_statistics_chart_success">{gmd-trending-up} Success</string>

Details

  • Used support library version: androidx.appcompat:appcompat:1.4.0-alpha03
  • Used gradle build tools version 4.2.2 / 7.1.1
  • Used tooling / Android Studio version: 4.2.2
  • Used library version: 5.2.8, 5.3.0-b01

Fonts:

  • com.mikepenz:google-material-typeface:4.0.0.1-kotlin@aar
  • com.mikepenz:community-material-typeface:5.8.55.0-kotlin@aar

Contributor guide

Research direction
Investigate the rendering code of IconicsTextView, particularly how it handles XML defined icons. Compare the implementation with the expected behavior and trace the font loading and rendering path.
Tech stack
kotlinandroid
Domain
mobile
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Android developmentGit
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60