Skip to content
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

Pull Request: Enhanced Modularity and Customizable Shortcuts #132

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

katzkuma
Copy link

@katzkuma katzkuma commented Aug 14, 2023

This pull request enhances code modularity and introduces customizable shortcuts for improved user experience. Here's a summary of the changes:

Code Modularity:

  • Moved menubar initialization to a dedicated module ('menubar.js').
  • Updated import and usage of Menubar functions in relevant files.

Customizable Shortcuts and Enhanced Settings View:

  • Introduced a custom shortcut feature for user-defined actions.
  • Developed a settings view ('index.html' and 'index.css') to configure shortcuts.
  • Integrated the 'electron-settings' package for efficient settings management.
  • Created a 'settings.js' module to handle settings view interaction.

These changes improve code organization and user interaction, setting the stage for future enhancements. Your review and feedback are welcome.

These are related issues:
#90
#47
#77
#112

This commit refactors the code to extract the initialization of the menubar instance (mb) into a separate module called 'menubar.js'. By moving the menubar creation logic to its own module, I'm gonna to create customized shortcut function later.

Changes Made:
- Moved menubar instance creation logic from main.js to mbManager.js.
- Updated index.js to import and use the createMenubar function from menubar.js.
- Adjusted otherScript.js to import and use the getMenubarInstance function from mbManager.js.
…enhanced settings view

To enhance user experience and provide greater customization options, this commit introduces a new feature allowing users to define custom shortcuts. Additionally, a new settings view has been developed to streamline the process of configuring shortcut. Here's a breakdown of the changes made:

Changes Made:
- New settings view with index.html and index.css
- Integration of 'electron-settings' Package
- Creation of `settings.js` Module for handling settings view
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant