Skip to content

SBK-132 - The contrast ratio between foreground and background color of “RAW” button 3.6:1 which is less than required ratio 4.5:1.#24343

Closed
gitstart wants to merge 1 commit into
storybookjs:nextfrom
gitstart:fix/raw-button-contrast
Closed

SBK-132 - The contrast ratio between foreground and background color of “RAW” button 3.6:1 which is less than required ratio 4.5:1.#24343
gitstart wants to merge 1 commit into
storybookjs:nextfrom
gitstart:fix/raw-button-contrast

Conversation

@gitstart

@gitstart gitstart commented Oct 2, 2023

Copy link
Copy Markdown
Contributor

Issue: #22473

What I did

  • change the text of RAW button to default text

How to test

Demo

Before

Screenshot 2023-10-03 at 19 48 50

After

Screenshot 2023-10-03 at 19 53 32

Co-authored-by: Mjanuary <22890925+Mjanuary@users.noreply.github.com>
@gitstart gitstart marked this pull request as ready for review October 2, 2023 14:04
@valentinpalkovic

Copy link
Copy Markdown
Contributor

cc @cdedreuille

@valentinpalkovic valentinpalkovic added ui bug ci:normal Run our default set of CI jobs (choose this for most PRs). labels Oct 2, 2023
@cdedreuille

Copy link
Copy Markdown
Contributor

The images doesn't seem to work. Can you update that @gitstart?

@cdedreuille

cdedreuille commented Nov 7, 2023

Copy link
Copy Markdown
Contributor

Thanks a lot for contributing to Storybook. I believe this issue was already resolved in this PR #24525. We fixed the color at the source in the theme making it available everywhere. I'll close this PR for now but if there's anything I missed, please add a comment or reopen it.

@cdedreuille cdedreuille closed this Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug ci:normal Run our default set of CI jobs (choose this for most PRs). ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants