From 88e313021808b50f378ea6ddf2a9909d82ed5f57 Mon Sep 17 00:00:00 2001 From: Nicola Corti Date: Thu, 4 May 2023 09:33:23 -0700 Subject: [PATCH] Fix unreadable dev menu header on dark theme apps (#37253) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/37253 Currently the Dev Menu has unreadable header/subtitle on apps with Dark theme. Here I'm fixing it: * First I'm adding support for Dark Theme to RN Tester (we were forcign ligth theme). * Then I'm removing any color reference and defaulting to the theme provided colors. Changelog: [Android] [Fixed] - Fix unreadable dev menu header on dark theme apps Reviewed By: mdvacca Differential Revision: D45566321 fbshipit-source-id: 947693aefdea846fec927a2cfa4d8236e02e0bfd --- .../facebook/react/devsupport/DevSupportManagerBase.java | 9 +++------ .../rn-tester/android/app/src/main/res/values/styles.xml | 3 +-- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.java index d133412e38a71a..cd6471491d089a 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.java @@ -17,7 +17,6 @@ import android.content.IntentFilter; import android.content.pm.PackageInfo; import android.content.pm.PackageManager; -import android.graphics.Color; import android.graphics.Typeface; import android.hardware.SensorManager; import android.util.Pair; @@ -498,23 +497,21 @@ public void onOptionSelected() { return; } - final LinearLayout header = new LinearLayout(getApplicationContext()); + final LinearLayout header = new LinearLayout(context); header.setOrientation(LinearLayout.VERTICAL); - final TextView title = new TextView(getApplicationContext()); + final TextView title = new TextView(context); title.setText(context.getString(R.string.catalyst_dev_menu_header, getUniqueTag())); title.setPadding(0, 50, 0, 0); title.setGravity(Gravity.CENTER); - title.setTextColor(Color.DKGRAY); title.setTextSize(16); title.setTypeface(title.getTypeface(), Typeface.BOLD); - final TextView jsExecutorLabel = new TextView(getApplicationContext()); + final TextView jsExecutorLabel = new TextView(context); jsExecutorLabel.setText( context.getString(R.string.catalyst_dev_menu_sub_header, getJSExecutorDescription())); jsExecutorLabel.setPadding(0, 20, 0, 0); jsExecutorLabel.setGravity(Gravity.CENTER); - jsExecutorLabel.setTextColor(Color.GRAY); jsExecutorLabel.setTextSize(14); header.addView(title); diff --git a/packages/rn-tester/android/app/src/main/res/values/styles.xml b/packages/rn-tester/android/app/src/main/res/values/styles.xml index 42c90f3e99a984..7ba83a2ad5a2c9 100644 --- a/packages/rn-tester/android/app/src/main/res/values/styles.xml +++ b/packages/rn-tester/android/app/src/main/res/values/styles.xml @@ -1,9 +1,8 @@ -