Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

🚸 Update Marlin Mode defaults #2924

Merged

Conversation

thisiskeithb
Copy link
Contributor

@thisiskeithb thisiskeithb commented Apr 7, 2024

Description

  • White text is a better default than orange. This also matches the current defaults in Configuration.h:
    * Options: [ WHITE: 0, BLACK: 1, RED: 2, GREEN: 3, BLUE: 4, CYAN: 5, MAGENTA: 6, YELLOW: 7,
    * ORANGE: 8, PURPLE: 9, LIME: 10, BROWN: 11, DARKBLUE: 12, DARKGREEN: 13, GRAY: 14, DARKGRAY: 15]
    */
    #define MARLIN_BACKGROUND_COLOR 1 // Marlin Mode background color (Default: 1)
    #define MARLIN_FONT_COLOR 0 // Marlin Mode font color (Default: 0)
  • Disable title text by default. You should already know that you're in "Marlin Mode" based on the lack of Touch Mode UI.

Benefits

Better Marlin Mode defaults

@thisiskeithb thisiskeithb force-pushed the pr/update_marlin_mode_defaults branch from ec9b6f4 to 1ae80f4 Compare April 25, 2024 07:08
@thisiskeithb thisiskeithb force-pushed the pr/update_marlin_mode_defaults branch from 1ae80f4 to 29c73c2 Compare May 18, 2024 04:00
@bigtreetech bigtreetech merged commit 59d8457 into bigtreetech:master May 23, 2024
1 check passed
@thisiskeithb thisiskeithb deleted the pr/update_marlin_mode_defaults branch May 23, 2024 06:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants