Dialer: Fix LetterTileDrawable for all languages

* Currently alphabet tile is rendered only for english language. However, Google Dialer / Googler Contacts do support rendering for all languages. Let's do same here! * Also, following Google, we do not want to handle numbers. * Fixes: crdroidandroid/android_packages_apps_Dialer#9 Signed-off-by: Pranav Vashi <[email protected]> Signed-off-by: strwbrry143 <[email protected]> Change-Id: I6aaf7a0c1436634a080b338e60ab295cd9b73a59 Signed-off-by: Mohammad Hasan Keramat J <[email protected]>
Fusion-OS-Twelve · Apr 2, 2023 · 7009125 · 7009125
1 parent e7431f4
commit 7009125
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/java/com/android/dialer/lettertile/LetterTileDrawable.java b/java/com/android/dialer/lettertile/LetterTileDrawable.java
@@ -173,10 +173,6 @@ private Drawable getDrawableForContactType(int contactType) {
     }
   }
 
-  private static boolean isEnglishLetter(final char c) {
-    return ('A' <= c && c <= 'Z') || ('a' <= c && c <= 'z');
-  }
-
   @Override
   public void draw(@NonNull final Canvas canvas) {
     final Rect bounds = getBounds();
@@ -335,7 +331,8 @@ public Character getLetter() {
 
   private LetterTileDrawable setLetterAndColorFromContactDetails(
       final String displayName, final String identifier) {
-    if (!TextUtils.isEmpty(displayName) && isEnglishLetter(displayName.charAt(0))) {
+    if (displayName != null && !TextUtils.isEmpty(displayName)
+            && Character.isLetter(displayName.charAt(0))) {
       letter = Character.toUpperCase(displayName.charAt(0));
     } else {
       letter = null;