Skip to content

Styles: create pill style - #553

Merged
stsdc merged 4 commits into
mainfrom
danirabbit/styles-pill
Sep 15, 2026
Merged

stsdc merged 4 commits into
mainfrom
danirabbit/styles-pill

Conversation

@danirabbit

Copy link
Copy Markdown
Member

DRY several styles by creating a pill class

@danirabbit
danirabbit requested a review from a team September 15, 2026 16:10
@stsdc

stsdc commented Sep 15, 2026

Copy link
Copy Markdown
Member
image image

Hmm, it's weird I remember critical state having full red background. main branch is the same, so we must have lost it somewhere during the porting (?)
Test: https://www.xbitlabs.com/pl/cpu-stress-test/

May I ask you to make critical state same as the warning state, but red?

@danirabbit

Copy link
Copy Markdown
Member Author

@stsdc should be fixed now! Nice catch. Was missing the color variables

Screenshot from 2026-09-15 10 45 00

@stsdc stsdc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! they're blinking again!

@stsdc
stsdc merged commit c2325c1 into main Sep 15, 2026
3 of 7 checks passed
@stsdc
stsdc deleted the danirabbit/styles-pill branch September 15, 2026 17:50
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