-
Notifications
You must be signed in to change notification settings - Fork 24.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Loading from Metro" banner rendering twice and remaining visible for ~15 seconds on each reload #43943
Labels
DX
Issues concerning how the developer experience can be improved.
p: Expo
Partner: Expo
Partner
Type: New Architecture
Issues and PRs related to new architecture (Fabric/Turbo Modules)
Comments
brentvatne
added
Needs: Triage 🔍
Type: New Architecture
Issues and PRs related to new architecture (Fabric/Turbo Modules)
labels
Apr 5, 2024
cortinico
added
DX
Issues concerning how the developer experience can be improved.
and removed
Needs: Triage 🔍
labels
Apr 8, 2024
cipolleschi
added a commit
to cipolleschi/react-native
that referenced
this issue
Apr 8, 2024
Summary: Following up facebook#43943, the metro loading banner is presented twice in Bridgeless mode. This happens because both the RCTInstance and the RCTHost are listening to the Reload Command and issuing the instructions to refetch the JSBundle and to present the banner. The RCTInstance should not concern itself with lifecycle events, owned by the RCTHost. ## Changelog: [iOS][Fixed] - Avoid to show Metro Loading banner twice. Reviewed By: cortinico Differential Revision: D55870640
cipolleschi
added a commit
to cipolleschi/react-native
that referenced
this issue
Apr 8, 2024
Summary: Following up facebook#43943, the metro loading banner is presented twice in Bridgeless mode. This happens because both the RCTInstance and the RCTHost are listening to the Reload Command and issuing the instructions to refetch the JSBundle and to present the banner. The RCTInstance should not concern itself with lifecycle events, owned by the RCTHost. ## Changelog: [iOS][Fixed] - Avoid to show Metro Loading banner twice. Reviewed By: cortinico Differential Revision: D55870640
cipolleschi
added a commit
to cipolleschi/react-native
that referenced
this issue
Apr 8, 2024
Summary: Following up facebook#43943, the metro loading banner is presented twice in Bridgeless mode. This happens because both the RCTInstance and the RCTHost are listening to the Reload Command and issuing the instructions to refetch the JSBundle and to present the banner. The RCTInstance should not concern itself with lifecycle events, owned by the RCTHost. ## Changelog: [iOS][Fixed] - Avoid to show Metro Loading banner twice. Reviewed By: cortinico Differential Revision: D55870640
facebook-github-bot
pushed a commit
that referenced
this issue
Apr 8, 2024
Summary: Pull Request resolved: #43967 Following up #43943, the metro loading banner is presented twice in Bridgeless mode. This happens because both the RCTInstance and the RCTHost are listening to the Reload Command and issuing the instructions to refetch the JSBundle and to present the banner. The RCTInstance should not concern itself with lifecycle events, owned by the RCTHost. ## Changelog: [iOS][Fixed] - Avoid to show Metro Loading banner twice. Reviewed By: cortinico Differential Revision: D55870640 fbshipit-source-id: addb67d3226f7d7db20736309172a42fc15f3aa3
cortinico
pushed a commit
that referenced
this issue
Apr 9, 2024
Summary: Pull Request resolved: #43967 Following up #43943, the metro loading banner is presented twice in Bridgeless mode. This happens because both the RCTInstance and the RCTHost are listening to the Reload Command and issuing the instructions to refetch the JSBundle and to present the banner. The RCTInstance should not concern itself with lifecycle events, owned by the RCTHost. ## Changelog: [iOS][Fixed] - Avoid to show Metro Loading banner twice. Reviewed By: cortinico Differential Revision: D55870640 fbshipit-source-id: addb67d3226f7d7db20736309172a42fc15f3aa3
✅ Picked, this will land in 0.74 RC8 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
DX
Issues concerning how the developer experience can be improved.
p: Expo
Partner: Expo
Partner
Type: New Architecture
Issues and PRs related to new architecture (Fabric/Turbo Modules)
Description
It looks like the loading "Loading from Metro" banner is rendered twice on each app reload (when you press
r
), and it takes about 15 seconds for the second banner to disappear. Reproduced on a new project with rc.6 created using https://gist.github.com/cipolleschi/82b7a9561b8861330efabbd3eb08c6f5. More details below.Steps to reproduce
r
to reload it, and observe as the banner rudely overstays its welcomeReact Native Version
0.74.0-rc.6
Affected Platforms
Runtime - iOS
Areas
Fabric - The New Renderer, TurboModule - The New Native Module System, Bridgeless - The New Initialization Flow
Output of
npx react-native info
Stacktrace or Logs
n/a
Reproducer
https://github.com/brentvatne/repro-metro-loading-status
Screenshots and Videos
New arch enabled
Screen_Recording_2024-04-04_at_5.55.23_PM.mov
New arch disabled
Screen_Recording_2024-04-04_at_5.55.23_PM.mov
The text was updated successfully, but these errors were encountered: