Reset zoom level on component unmount #69087
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Mamaduka
added
[Type] Bug
|
Do you know what part of the code disables zoomout mode for smaller screens? Controlling a global state-based component's mounted state is tricky and usually causes more bugs. This bug probably has a different root cause; I suggest finding it and fixing the issue there. |
|
Sure @Mamaduka, I’ll check what disables zoom-out on smaller screens and find the root cause. |
What?
Closes #69086
Reset zoom level when editor component unmounts to prevent stale zoom states.
Why?
The zoom state and button remained active after responsive changes, causing inconsistency between the visual zoom effect and the button state. This led to a confusing user experience in which the zoom button appeared pressed, but no zoom effect was visible.
How?
Added a cleanup effect in the ZoomOutToggle component that calls resetZoomLevel when the component unmounts. This ensures the zoom state is properly reset between editor sessions and responsive changes.
Testing Instructions
Screencast
Screen.Recording.2025-02-07.at.12.27.40.PM.mov