Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Jun 17, 2025. It is now read-only.

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
@kakysha
Copy link

kakysha commented Jun 10, 2024

@vincelwt can you merge this please? Kinda annoying that CMD+Shift+G is exclusively held by your app, when it is used by many other apps and people need to break their habits now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.