Skip to content

Commit

Permalink
DevMenu: Change "Toggle Inspector" to Show/Hide Element Inspector
Browse files Browse the repository at this point in the history
Summary:
Rationale:
- This makes the element inspector button consistent with the Fast Refresh, Perf Monitor and other buttons in the DevMenu
- This makes the button more informative

Changelog: [Android][Changed] Rename the "Toggle Inspector" DevMenu item to "Hide/Show Element Inspector"

Reviewed By: JoshuaGross

Differential Revision: D29146871

fbshipit-source-id: 8e8c19217ea2ff2f1d176521aa22200058e7e643
  • Loading branch information
RSNara authored and facebook-github-bot committed Jun 16, 2021
1 parent 1816536 commit e91fb05
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -474,8 +474,9 @@ public void onClick(DialogInterface dialog, int which) {
});

options.put(
// NOTE: `isElementInspectorEnabled` is not guaranteed to be accurate.
mApplicationContext.getString(R.string.catalyst_inspector),
mDevSettings.isElementInspectorEnabled()
? mApplicationContext.getString(R.string.catalyst_inspector_stop)
: mApplicationContext.getString(R.string.catalyst_inspector),
new DevOptionHandler() {
@Override
public void onOptionSelected() {
Expand Down
3 changes: 2 additions & 1 deletion ReactAndroid/src/main/res/devsupport/values/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@
<string name="catalyst_hot_reloading_stop" project="catalyst" translatable="false">Disable Fast Refresh</string>
<string name="catalyst_hot_reloading_auto_disable" project="catalyst" translatable="false">Disabling Fast Refresh because it requires a development bundle.</string>
<string name="catalyst_hot_reloading_auto_enable" project="catalyst" translatable="false">Switching to development bundle in order to enable Fast Refresh.</string>
<string name="catalyst_inspector" project="catalyst" translatable="false">Toggle Inspector</string>
<string name="catalyst_inspector" project="catalyst" translatable="false">Show Element Inspector</string>
<string name="catalyst_inspector_stop" project="catalyst" translatable="false">Hide Element Inspector</string>
<string name="catalyst_perf_monitor" project="catalyst" translatable="false">Show Perf Monitor</string>
<string name="catalyst_perf_monitor_stop" project="catalyst" translatable="false">Hide Perf Monitor</string>
<string name="catalyst_settings" project="catalyst" translatable="false">Settings</string>
Expand Down

0 comments on commit e91fb05

Please sign in to comment.