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

Added screenreader text on close button #1672

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

EldRoos
Copy link
Contributor

@EldRoos EldRoos commented Oct 15, 2024

Added screen reader text on close button.
Connected to this issue: #706 & #665

Please check if you also get visual error on AlertRibbon close button on colored backgrounds in React:
https://storybook.seb.io/latest/react/?path=/docs/components-alertribbon--docs

Locally it looks like this line is not triggered anymore:
green\libs\chlorophyll\scss\components\alert-ribbon_mixins.scss
line 139 - background-color: var(--color);

Also green\libs\react\src\lib\card\card.stories.tsx file shows up as red/error?

Slider storybook for react now shows label and sublabel, along with default value and a big number example.
@green-design-system-bot
Copy link
Contributor

green-design-system-bot bot commented Oct 15, 2024

🦋 Changeset status

Based on the included changeset, the following releases will be published as a result of this PR:

  • @sebgroup/chlorophyll: 3.6.7 → 3.6.8 (patch bump)
  • @sebgroup/green-angular: 4.6.4 → 4.6.5 (patch bump)
  • @sebgroup/green-react: 3.19.0 → 3.19.1 (patch bump)

Note: If unpublished changesets where merged to main after the first commit of this PR, this
list will include those changes in addition to the ones specific to this PR.

@green-design-system-bot
Copy link
Contributor

green-design-system-bot bot commented Oct 16, 2024

👋 Thanks for creating a pull request!

🚀 Checkout the storybooks we've created for it:

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.

1 participant