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(dashboard): use fixed text color for stat panels #21

Merged
merged 1 commit into from
Feb 9, 2023

Conversation

tthvo
Copy link
Member

@tthvo tthvo commented Feb 9, 2023

Related to cryostatio/cryostat-web#856

This includes fix to panel text to specify single text color with text variant.

Dark Theme

Screenshot from 2023-02-08 20-19-39

Light Theme

Screenshot from 2023-02-08 20-19-51

@andrewazores
Copy link
Member

Looks good. I wonder if there is a way to override settings like this via the embedded panel query parameters or some other dynamic mechanism, though.

@tthvo
Copy link
Member Author

tthvo commented Feb 9, 2023

Been trying to find a way but seems like everything must be configured via the JSON model. I think the user can change the styling via panel editor if they are authenticated (maybe admin?). Maybe we can figure a way to embed/anchor to the editor view and somehow allow saving latest json model on top of the shipped base JSON?

@andrewazores
Copy link
Member

That's a lot more work than I think is worth doing, honestly. I think this is good to go as it is.

@andrewazores andrewazores merged commit a84e744 into cryostatio:main Feb 9, 2023
@tthvo tthvo deleted the panel-text-color branch February 9, 2023 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants