meerk40t / meerk40t

Hackable Laser software for K40 / GRBL / Fibre Lasers
MIT License
233 stars 62 forks source link

Some minor Linux tweaks #2662

Closed jpirnay closed 2 weeks ago

jpirnay commented 2 weeks ago

Summary by Sourcery

Enhance the GUI by replacing the '*' symbol with an icon for the property button in the laser panel, allowing brighter color defaults for dark themes, and adding 'exit' as a program termination command.

Enhancements:

sourcery-ai[bot] commented 2 weeks ago

Reviewer's Guide by Sourcery

This pull request introduces minor Linux tweaks and UI improvements, including changes to icon display, color settings for dark themes, and program termination commands. The changes primarily affect the GUI components and preferences handling in the MeerK40t application.

Updated class diagram for GUI preferences

classDiagram
    class Preferences {
        +bool color_reset_brighter
        +void color_reset_brighter(value)
    }
    note for Preferences "Added color_reset_brighter property and setter for brighter color defaults in dark themes."

Updated class diagram for LaserPanel

classDiagram
    class LaserPanel {
        +wxButton btn_config_laser
    }
    note for LaserPanel "Updated btn_config_laser to use an icon instead of '*'."

Updated class diagram for Plugin

classDiagram
    class Plugin {
        +void shutdown()
    }
    note for Plugin "Added 'exit' as a program termination command."

File-Level Changes

Change Details Files
Replace asterisk button with an icon for device configuration
  • Replace '*' text with an icon for the device configuration button
  • Adjust button size and layout for better consistency across platforms
  • Update event binding for the configuration button
meerk40t/gui/laserpanel.py
Add support for brighter color defaults in dark themes
  • Introduce a new option to reset colors to brighter defaults for dark themes
  • Modify the color reset functionality to accommodate the new brighter option
  • Update the preferences panel to include the new brighter color reset option
meerk40t/gui/preferences.py
meerk40t/gui/guicolors.py
meerk40t/gui/wxmscene.py
Add 'exit' as an alternative program termination command
  • Include 'exit' alongside 'quit' and 'shutdown' as valid termination commands
meerk40t/gui/plugin.py
Optimize startup process for Linux systems
  • Skip displaying the busy info dialog during startup on Linux systems
meerk40t/gui/plugin.py

Tips and commands #### Interacting with Sourcery - **Trigger a new review:** Comment `@sourcery-ai review` on the pull request. - **Continue discussions:** Reply directly to Sourcery's review comments. - **Generate a GitHub issue from a review comment:** Ask Sourcery to create an issue from a review comment by replying to it. - **Generate a pull request title:** Write `@sourcery-ai` anywhere in the pull request title to generate a title at any time. - **Generate a pull request summary:** Write `@sourcery-ai summary` anywhere in the pull request body to generate a PR summary at any time. You can also use this command to specify where the summary should be inserted. #### Customizing Your Experience Access your [dashboard](https://app.sourcery.ai) to: - Enable or disable review features such as the Sourcery-generated pull request summary, the reviewer's guide, and others. - Change the review language. - Add, remove or edit custom review instructions. - Adjust other review settings. #### Getting Help - [Contact our support team](mailto:support@sourcery.ai) for questions or feedback. - Visit our [documentation](https://docs.sourcery.ai) for detailed guides and information. - Keep in touch with the Sourcery team by following us on [X/Twitter](https://x.com/SourceryAI), [LinkedIn](https://www.linkedin.com/company/sourcery-ai/) or [GitHub](https://github.com/sourcery-ai).