-
Notifications
You must be signed in to change notification settings - Fork 428
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
Improvement disable control button when change state we do not redraw panel #1094
Improvement disable control button when change state we do not redraw panel #1094
Conversation
Release Branch
… the control panel
Please review my PR @Falke-Design |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you think it is important to merge this? Clearly the difference is that the Toolbar is not rendered new, but every change on the Toolbar cause a re-render so why should we handle this different?
Anyway thank you for your contribution.
For example, I can give an example of code from the leaflet library, they have buttons for zooming the map and don't render after disable. The redraw render is complex for the browser. |
… panel (geoman-io#1094) (patch) * Improvement disable control button when change state we do not redraw the control panel * Fixed test * Fixed test * Fix removing event listener Co-authored-by: Sumit Kumar <sk@outlook.com> Co-authored-by: Florian Bischof <design.falke@gmail.com>
When studying the code, it was found that the optimal movement, which causes a complete redrawing of the control panel, when changes disable state.
This PR makes the following changes: