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

frontend: fix component theming issues #2894

Merged
merged 7 commits into from
Jan 11, 2024
Merged

frontend: fix component theming issues #2894

merged 7 commits into from
Jan 11, 2024

Conversation

jecr
Copy link
Contributor

@jecr jecr commented Jan 10, 2024

Covers the following issues found in components in an ongoing theming update:

App Layout - Header: Search autocomplete highlight color is too dark
before:
image

after:
image

Feedback: Darken text in feedback components for legibility
before:
image

after:
image

Input - Switch: gray colors make it look disabled
before:
image
after:
image

Card: Landing card, icon character should be dark
before:
image

after:
image

Accordion: Missing label (white text)
before:
image

after:
image

Dialog: Pressed state should be a darker shade
normal state:
image

before:
image

after:
image

Testing Performed
manual, unit tests

@jecr jecr marked this pull request as ready for review January 10, 2024 21:33
@jecr jecr requested a review from a team as a code owner January 10, 2024 21:33
Copy link
Contributor

@jdslaugh jdslaugh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@jecr jecr merged commit dbc9d95 into use-mui-theming Jan 11, 2024
1 check passed
@jecr jecr deleted the bug-bash-items branch January 11, 2024 19:33
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.

2 participants