commit 24e1eb8b6a278cc229b4cbb655c10672cae53072
author Pranav Vashi <neobuddy89@gmail.com> Sun Jul 31 13:08:48 2022 +0530
committer Jis G Jacob <studiokeys@blissroms.org> Mon May 06 01:01:58 2024 -0400
tree 5ecab0adf9eb3e3a37fbf08459f3a27745e9b030
parent 64d3e4ca49c7142f8eb079133d8ef55668cf6d9c

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: https://github.com/crdroidandroid/android_packages_apps_Dialer/issues/9

Signed-off-by: Pranav Vashi <neobuddy89@gmail.com>
Signed-off-by: Pranav Temkar <pranavtemkar@gmail.com>
Signed-off-by: Jis G Jacob <studiokeys@blissroms.org>

java/com/android/dialer/lettertile/LetterTileDrawable.java

diff --git a/java/com/android/dialer/lettertile/LetterTileDrawable.java b/java/com/android/dialer/lettertile/LetterTileDrawable.java
index 583731e..282a1c8 100644
--- a/java/com/android/dialer/lettertile/LetterTileDrawable.java
+++ b/java/com/android/dialer/lettertile/LetterTileDrawable.java
@@ -176,10 +176,6 @@
     }
   }
 
-  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();
@@ -336,7 +332,8 @@
 
   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;