Partial-Disable toolbar functionalities during 'Start Registration' - 771 #1 #826
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…on of issue invesalius#771 - Updated the ObjectToolBar class to disable toolbar buttons (zoom, rotate, move, etc.) when StartRegistration is active. - Passed an argument from StartRegistration to _EnableState to manage toolbar state based on registration activity. - Attempted to implement a warning message popup using the wx library to notify users that toolbar buttons are disabled when 'StartRegistration' is active, but was unable to get the popup to work even after many tries. This commit addresses part of issue invesalius#771 by disabling toolbar functionality during 'StartRegistration'. Further changes are needed to implement the warning message.
bhavana-github-acc
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Partial-Disable toolbar functionalities during registration; partial solution.
This commit addresses part of issue #771 by disabling toolbar functionality during 'StartRegistration'. Further changes are needed to implement the warning message.
Please Note! The PR has made some formatting changes such as extra spaces and quote replacements. These were automatically applied and are not related to the functional changes. Please let me know if this harms the code in any way.
I have made changes only to the frame.py and task_navigator.py files. I was using PyCharm for the fork; if I am to use a different IDE which not cause any formatting changes, please let me know; I will edit it again.