mne-qt-browser
mne-qt-browser copied to clipboard
Enhance Help Menu for Better Functionality Discoverability
Reference issue
Example: Fixes #84
What does this fix?
This PR improves the Help menu by replacing the old HelpDialog with a simpler and more accessible menu. Now, users can find keyboard and mouse shortcuts directly in a dropdown menu instead of a separate dialog.
Key changes
- Removed the HelpDialog class.
- Introduced a Help button (QToolButton) for quick access.
Additional information
Please review the changes and let me know if there are any improvements or issues that need to be addressed.
Here are the screenshots of the UI
@mohitkumhar I think what @hoechenberger had in mind was a top-level menu like that in https://github.com/mne-tools/mne-qt-browser/pull/126. It looks like you added a menu that is accessed via a toolbar button instead
Hey @larsoner , Kindly check this UI , is it the expected output?
@mohitkumhar Thanks for your work on this!
What I actually had in mind was adding a full menu bar, though. Something like what you'd see in other applications like e.g. MS Word:
Since menu items can be linked to keyboard shortcuts (which will be displayed in the menu itself), these would be kind of self-documenting.
Sure @hoechenberger I understand it, let me create menu and share sample
You can take #126 as a starting point
Hey @hoechenberger , Kindly check it out!
https://github.com/user-attachments/assets/e8451c65-37ec-4e9b-b21b-f8c6f8149135
Hey , kindly check the commit I made