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

chore: Modal Button Examples Update #2904

Open
wants to merge 6 commits into
base: prerelease/major
Choose a base branch
from

Conversation

josh-bagwell
Copy link
Contributor

@josh-bagwell josh-bagwell commented Sep 3, 2024

Summary

Fixes: #2871

Updating the button order in Modal examples to show the "Primary" button as the last option or "end-aligned".

Release Category

Documentation


Where Should the Reviewer Start?

/modules/react/modal/stories/examples

Copy link

cypress bot commented Sep 3, 2024

Workday/canvas-kit    Run #7737

Run Properties:  status check passed Passed #7737  •  git commit 4d4afb4d54 ℹ️: Merge 86c636d7df1ca1449e0bb7c99b3cfa8e52e011a9 into bd882e1c4c3a8bb73ffcf74ce3da...
Project Workday/canvas-kit
Branch Review modal-button-example-update-24-08
Run status status check passed Passed #7737
Run duration 03m 29s
Commit git commit 4d4afb4d54 ℹ️: Merge 86c636d7df1ca1449e0bb7c99b3cfa8e52e011a9 into bd882e1c4c3a8bb73ffcf74ce3da...
Committer Josh
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 16
Tests that did not run due to a developer annotating a test with .skip  Pending 16
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 854
View all changes introduced in this branch ↗︎
UI Coverage  20.99%
  Untested elements 1846  
  Tested elements 488  
Accessibility  99.23%
  Failed rules  5 critical   5 serious   0 moderate   2 minor
  Failed elements 229  

@josh-bagwell josh-bagwell marked this pull request as ready for review September 4, 2024 21:21
@josh-bagwell josh-bagwell added the ready for review Code is ready for review label Sep 4, 2024
@jaclynjessup jaclynjessup linked an issue Sep 9, 2024 that may be closed by this pull request
@josh-bagwell josh-bagwell self-assigned this Sep 11, 2024
@jaclynjessup jaclynjessup added on hold PR is on hold until further notice and removed ready for review Code is ready for review labels Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on hold PR is on hold until further notice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Documenation and Examples Modal Buttons
3 participants