react-native icon indicating copy to clipboard operation
react-native copied to clipboard

disable "Open Debugger" menu item if the bundler is disconnected

Open EdmondChuiHW opened this issue 1 year ago • 9 comments

Summary: Changelog: [Android][Fixed] Disable the "Open Debugger" item from dev menu if the bundler is disconnected

Existing

In our dev menu, the "Open Debugger" menu item is shown even if the packager isn't connected.

{F1451833462}

In this PR

The "Open Debugger" menu item is disabled when the packager is disconnected with the message "Connect to the bundler to debug JavaScript"

{F1451834019}

Reference

  • There are existing checks, but they're async. We don't need the check to be accurate; and we don't want to delay the dev menu from opening
  • Also on iOS: D53354110

Differential Revision: D53428914

EdmondChuiHW avatar Feb 05 '24 17:02 EdmondChuiHW

This pull request was exported from Phabricator. Differential Revision: D53428914

facebook-github-bot avatar Feb 05 '24 17:02 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D53428914

facebook-github-bot avatar Feb 05 '24 17:02 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D53428914

facebook-github-bot avatar Feb 05 '24 17:02 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D53428914

facebook-github-bot avatar Feb 05 '24 17:02 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D53428914

facebook-github-bot avatar Feb 05 '24 17:02 facebook-github-bot

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 17,233,732 +448
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 20,600,535 +424
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: db066acfe3994787d706ad082ce718a91b8249f5 Branch: main

analysis-bot avatar Feb 05 '24 17:02 analysis-bot

This pull request was exported from Phabricator. Differential Revision: D53428914

facebook-github-bot avatar Feb 05 '24 22:02 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D53428914

facebook-github-bot avatar Feb 05 '24 22:02 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D53428914

facebook-github-bot avatar Feb 06 '24 11:02 facebook-github-bot

This pull request was successfully merged by @EdmondChuiHW in 7afc8b8623cc499c3054b9fb5b08cc2b65edb19c.

When will my fix make it into a release? | Upcoming Releases

github-actions[bot] avatar Feb 09 '24 11:02 github-actions[bot]