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

Unify modal button styles with ThemedButton #779

Open
chriswilty opened this issue Jan 16, 2024 · 0 comments
Open

Unify modal button styles with ThemedButton #779

chriswilty opened this issue Jan 16, 2024 · 0 comments
Labels
frontend Requires work on the frontend
Milestone

Comments

@chriswilty
Copy link
Member

While we intend to rework the styling for modals (see #578 and related issues) it would be nice in the meantime to unify the styling of their buttons to match the action buttons used elsewhere in the application (e.g. header bar actions, defence reset buttons).

Same colours as action buttons:

Default
Image
Focused
Image
Hover
Image
Focused + Hover
Image

Same colours as level buttons (according to new styles in #777):

Selected, with or without hover
Image
Selected + Focused
Image

@chriswilty chriswilty added the frontend Requires work on the frontend label Jan 16, 2024
@chriswilty chriswilty added this to the UI Update milestone Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Requires work on the frontend
Projects
None yet
Development

No branches or pull requests

1 participant