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

fix a weird line if we have set menu color #2158

Closed
wants to merge 5 commits into from

Conversation

zxkmm
Copy link
Contributor

@zxkmm zxkmm commented May 21, 2024

this PR aim to fix that the "Menu Color" feature have a not quite elegant line of button grid top
and when i was writing this, i think we can have a better global theme set, should be beautiful.

after discussion, we can decide if we agree to cost some SPI flash (we need to include pmem to Style file) to have a Style class impl to always match the theme color, which can set title bar color and status bar color and button color to match the theme
i gave up the second plan cuz idk how to do the style things.

@zxkmm zxkmm marked this pull request as ready for review May 21, 2024 16:03
@zxkmm zxkmm marked this pull request as draft May 22, 2024 01:42
@zxkmm zxkmm changed the title global theme global theme color May 22, 2024
@zxkmm zxkmm changed the title global theme color fix a weird line if we have set menu color May 22, 2024
@zxkmm zxkmm marked this pull request as ready for review May 22, 2024 10:49
@zxkmm
Copy link
Contributor Author

zxkmm commented May 22, 2024

this seems impact the outline of other buttons, that maybe not quite elegant.
maybe we should do the global theme thing

@zxkmm zxkmm closed this May 22, 2024
zxkmm added a commit that referenced this pull request Sep 11, 2024
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.

1 participant