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

3052 v3 upgrade minimum stenciljs version #3288

Merged

Conversation

GCHQ-Developer-530
Copy link
Contributor

Summary of the changes

Upgrade web components and canary web components minimum stencil version to 4.26.0. Update other packages based off that. And update test snapshots.

Related issue

#3052

Checklist

General

  • Changes to docs package checked and committed.
  • 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.
  • Manual screen reader testing performed using NVDA and VoiceOver.
  • Manual keyboard testing for keyboard controls and logical focus order.
  • Correct roles used and ARIA attributes used correctly where required.
  • Logical heading structure is maintained, and the HTML elements used for headings can be changed to fit within the wider page structure.

Resize/zoom behaviour

  • Page can be zoomed to 400% with no loss of content.
  • Screen magnifier used with no issues.
  • Text resized to 200% with no loss of content.
  • Text spacing increased as per the WCAG 1.4.12 success criterion with no loss of content.

System modes

  • Browser setting 'prefers reduced motion' tested. No animations or motion visible whilst this setting is on.
  • Windows High Contrast mode tested with no loss of content.
  • System light and dark mode tested with no loss of content.
  • Browser support tested (Chrome, Safari, Firefox and Edge).

Testing content extremes

  • Min/max content examples tested with no loss of content or overflow.
  • All prop combinations work without issue.
  • Tested for FOUC (Flash of Unstyled Content) in both SSR (Server-Side Rendering) and SSG (Static Site Generation) settings.
  • Controlled and uncontrolled input components tested.
  • Props/slots can be updated after initial render.

Copy link
Contributor

github-actions bot commented Mar 5, 2025

@GCHQ-Developer-530 GCHQ-Developer-530 marked this pull request as draft March 5, 2025 09:08
@GCHQ-Developer-530 GCHQ-Developer-530 force-pushed the 3052-v3-upgrade-minimum-stenciljs-version branch 5 times, most recently from bb96b83 to a0a76fd Compare March 5, 2025 13:34
@GCHQ-Developer-530 GCHQ-Developer-530 force-pushed the 3052-v3-upgrade-minimum-stenciljs-version branch 5 times, most recently from 38f0f54 to 3da8eac Compare March 17, 2025 13:38
@GCHQ-Developer-530 GCHQ-Developer-530 marked this pull request as ready for review March 17, 2025 14:13
gd2910
gd2910 previously approved these changes Mar 18, 2025
Upgrade the minimum stencil version to 4.26.0. Update test snapshots to replace mock:shadow-root
with template shadowrootmode="open"

. #3052
…nents stencil version

Update React package-lock.json with update to web components stencil version
Update docs after stencil upgrade
Add shadowrootmode to .cspell.json
…4.26.0

Upgrade the minimum stencil version to 4.26.0. Update test snapshots to replace mock:shadow-root
with template shadowrootmode="open"
…eb components stencil version

update package-lock.json with update to canary web components stencil version
update docs after stencil upgrade
@GCHQ-Developer-530 GCHQ-Developer-530 force-pushed the 3052-v3-upgrade-minimum-stenciljs-version branch from 6e67577 to aad0073 Compare March 20, 2025 07:49
@GCHQ-Developer-530 GCHQ-Developer-530 merged commit b41f475 into v3.0.0/develop Mar 24, 2025
6 checks passed
@GCHQ-Developer-530 GCHQ-Developer-530 deleted the 3052-v3-upgrade-minimum-stenciljs-version branch March 24, 2025 13:37
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.

3 participants