Skip to content

feat: Add Sana Canvas Menu Updates#4052

Draft
sheelah wants to merge 17 commits into
Workday:prerelease/majorfrom
sheelah:sana-canvas-menu
Draft

feat: Add Sana Canvas Menu Updates#4052
sheelah wants to merge 17 commits into
Workday:prerelease/majorfrom
sheelah:sana-canvas-menu

Conversation

@sheelah

@sheelah sheelah commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes: #3987.

Release Category

Components

Release Note

Optional release note message. Changelog and release summaries will contain a pull request title. This section will add additional notes under that title. This section is not a summary, but something extra to point out in release notes. An example might be calling out breaking changes in a labs component or minor visual changes that need visual regression updates. Remove this section if no additional release notes are required.


Checklist

For the Reviewer

  • PR title is short and descriptive
  • PR summary describes the change (Fixes/Resolves linked correctly)
  • PR Release Notes describes additional information useful to call out in a release message or removed if not applicable
  • Breaking Changes provides useful information to upgrade to this code or removed if not applicable

Where Should the Reviewer Start?

Areas for Feedback? (optional)

  • Code
  • Documentation
  • Testing
  • Codemods

Testing Manually

Screenshots or GIFs (if applicable)

Thank You Gif (optional)

@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: aa3a7550-47be-4cc5-a5ee-5499b3976027

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cypress

cypress Bot commented Jul 8, 2026

Copy link
Copy Markdown

Workday/canvas-kit    Run #11216

Run Properties:  status check passed Passed #11216  •  git commit 4779a4c87d ℹ️: Merge 52b5eda48c24115619c8944da048c2aa4bb792ff into df9a69acd1329b7bc14a225dfbe4...
Project Workday/canvas-kit
Branch Review sana-canvas-menu
Run status status check passed Passed #11216
Run duration 02m 23s
Commit git commit 4779a4c87d ℹ️: Merge 52b5eda48c24115619c8944da048c2aa4bb792ff into df9a69acd1329b7bc14a225dfbe4...
Committer Sheelah Brennan
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 17
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 809
View all changes introduced in this branch ↗︎
UI Coverage  19.68%
  Untested elements 1524  
  Tested elements 371  
Accessibility  99.41%
  Failed rules  5 critical   5 serious   0 moderate   2 minor
  Failed elements 72  

@sheelah sheelah changed the title feat: Add Sana Canvas Menu feat: Add Sana Canvas Menu Updates Jul 8, 2026
@sheelah sheelah added the needs-a11y Extra attention from accessibility is needed label Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-a11y Extra attention from accessibility is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant