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

2913 focused toggle button inner border #3243

Closed
wants to merge 107 commits into from

Conversation

MI6-286
Copy link
Contributor

@MI6-286 MI6-286 commented Feb 24, 2025

Summary of the changes

Updated the inner border of focused, checked toggle button so that it is black in dark mode, and added a visual regression test for this scenario.

Related issue

#2913

Checklist

General

  • All acceptance criteria reviewed and met.

Testing

  • Relevant unit tests and visual regression tests added.
  • Visual testing against Figma component specification completed.
  • Playground stories in React Storybook up to date, with any prop changes and additions addressed.
  • Compare performance of modified components against develop using Performance addon in React Storybook.

Accessibility

  • Accessibility Insights FastPass performed.
  • A11y unit test added and yields no issues.
  • A11y plug-in on Storybook yields no issues.

GCHQ-Developer-530 and others added 30 commits February 6, 2025 08:40
Update audit dates for March
chore(root): update audit dates - v3 cherry pick
…rectly in nav group

fix: selected items with hover state are properly lined up within nav group menu
Update docs.json with empty state theme prop
Update empty state to have theme prop to make use of light/dark theme
Update tree view to prevent FOUC - set tree items visibility to hidden until loaded when inherited
props from tree view used. Removed timeouts on truncation - tree view now truncates when ready
rather than waiting 0.1 seconds to help reduce flashing.
…d items

Update tree view truncation story to have expanded items - to show truncation working on nesting
items.
Update data table empty state cypress image
Update empty state to have theme prop to make use of light/dark theme
fix(canary-web-components): update tree view to improve FOUC
deployments to npm now run from a dedicated release branch, once complete, are then merged into
develop, then main. This prevents any conflicts or needing to merge main back into develop.

2765
…ide nav item when dark

Fix double focus indicator on side nav item (light theme colour / dark foreground) when it is
selected and a slotted link is used.
Cherry pick up deployment changes
…ator-v3

fix(web-components): fix double focus indicator on selected slotted s…
…ix storybook example

Update visual regression images in IcAlert and IcTheme, fix storybook example after height change
fix in ic-alert
…themes

variables added to variables.css, container min-height now changes on theme change to compensate for
border in dark mode
uncommenting out auto dismiss toast accessibility check
…ark-mode

fix(web-components): [ic-alert] extra height in DM bug
…to-dismiss toast

adding description to ic-loading-indicator of auto-dismiss toast to prevent aria-label a11y failure
…e set programmatically

Provide a selectedItemsPerPage prop

fix #2525
…grammatically

Provide a selectedItemsPerPage prop
changed auto-dismiss toasts from role=status to alert, so they read to screenreaders
Made the auto-dismiss toasts polite so they don't interrupt navigation

2710
update the message of the Auto-dismiss toast react story to be more clear about functionality

2710
GCHQ-Developer-530 and others added 24 commits February 18, 2025 12:43
Update docs.json with disableAutoSort prop on data table
Implement disableAutoSort prop on data table so that the sort button appears and emits a sort event
but doesn't actually sort the data

. #2415
Update to Storybook v8 - .stories.mdx replaced with new format of .stories.js and .mdx
description change to say defult button will not be shown however, slotted controls will
Update to storybook v8 - with new format for stories
…g-controls-are-displayed-regardless-of-buttons-prop

docs(web-components): description change in the buttons prop
adds workflow to trigger v3 guidance site update
cherry-pick trigger of v3 site check-versions
replace references to regular develop with v3 develop
replace references to release with v3.0.0/release in the v3 release check script
fixes build by committing files which have changed
delete development release check file to unblock release
updating inner border of focused toggle button
…e button

adding visual regression test for focused, checked toggle button with dark theme
Copy link
Contributor

@MI6-286 MI6-286 marked this pull request as ready for review February 24, 2025 09:48
@MI6-255 MI6-255 force-pushed the v3.0.0/develop branch 2 times, most recently from 46015cc to 7cab72a Compare February 24, 2025 11:34
@MI6-286 MI6-286 closed this Feb 24, 2025
@MI6-286 MI6-286 deleted the 2913-focused-toggle-button-inner-border branch February 24, 2025 15:34
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.