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

ACS-3918 - v15 modal teleport #153

Closed
wants to merge 2 commits into from
Closed

Conversation

daemoncron
Copy link
Collaborator

Description

...

Checklist:

Design:

  • Reviewed with designer and meets expectations

Cross-browser testing:

  • Chrome
  • Firefox
  • Edge
  • Safari
  • iOS
  • Android

Unit testing:

  • Sufficient unit test coverage (see unit test best practices for ideas)
  • Snapshot updates are explained with comment and reference the relevant source code change

A11y:

  • Meets WCAG AA standards

Documentation:

  • API docs created/updated
  • Examples created/updated

@daemoncron daemoncron self-assigned this Nov 3, 2023
package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "@rei/cedar",
"version": "14.1.1",
"version": "15.0.0",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Manually versioned, tags not pushed yet

@daemoncron daemoncron closed this Nov 7, 2023
'body > *',
':not(script)',
':not(style)',
':not([aria-hidden=true])',
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Prevents us from accidentally removing aria-hidden="true" from something that was set that way on page load.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Same change to all snapshots - teleport-stub added.

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