Skip to content

Commit 0cbd2a3

Browse files
cortinicofacebook-github-bot
authored andcommitted
Update Element Inspector string to "Toggle" (facebook#41595)
Summary: Pull Request resolved: facebook#41595 Instead of having to select between Enable/Hide the Element Inspector String, let's just use "Toggle". Changelog: [Internal] [Changed] - Update Element Inspector string to "Toggle" Reviewed By: cipolleschi Differential Revision: D51503403 fbshipit-source-id: 2ad24df8324eb789b0016fe7ac5d439cba6f5952
1 parent 2a4338d commit 0cbd2a3

File tree

3 files changed

+3
-7
lines changed

3 files changed

+3
-7
lines changed

packages/react-native/React/CoreModules/RCTDevMenu.mm

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -283,8 +283,7 @@ - (void)setDefaultJSBundle
283283

284284
[items addObject:[RCTDevMenuItem
285285
buttonItemWithTitleBlock:^NSString * {
286-
return devSettings.isElementInspectorShown ? @"Hide Element Inspector"
287-
: @"Show Element Inspector";
286+
return @"Toggle Element Inspector";
288287
}
289288
handler:^{
290289
[devSettings toggleElementInspector];

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -414,9 +414,7 @@ public void onClick(DialogInterface dialog, int which) {
414414
});
415415

416416
options.put(
417-
mDevSettings.isElementInspectorEnabled()
418-
? mApplicationContext.getString(R.string.catalyst_inspector_stop)
419-
: mApplicationContext.getString(R.string.catalyst_inspector),
417+
mApplicationContext.getString(R.string.catalyst_inspector_toggle),
420418
new DevOptionHandler() {
421419
@Override
422420
public void onOptionSelected() {

packages/react-native/ReactAndroid/src/main/res/devsupport/values/strings.xml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@
1111
<string name="catalyst_hot_reloading_stop" project="catalyst" translatable="false">Disable Fast Refresh</string>
1212
<string name="catalyst_hot_reloading_auto_disable" project="catalyst" translatable="false">Disabling Fast Refresh because it requires a development bundle.</string>
1313
<string name="catalyst_hot_reloading_auto_enable" project="catalyst" translatable="false">Switching to development bundle in order to enable Fast Refresh.</string>
14-
<string name="catalyst_inspector" project="catalyst" translatable="false">Show Element Inspector</string>
15-
<string name="catalyst_inspector_stop" project="catalyst" translatable="false">Hide Element Inspector</string>
14+
<string name="catalyst_inspector_toggle" project="catalyst" translatable="false">Toggle Element Inspector</string>
1615
<string name="catalyst_perf_monitor" project="catalyst" translatable="false">Show Perf Monitor</string>
1716
<string name="catalyst_perf_monitor_stop" project="catalyst" translatable="false">Hide Perf Monitor</string>
1817
<string name="catalyst_settings" project="catalyst" translatable="false">Settings</string>

0 commit comments

Comments
 (0)