Skip to content

docs(menu-separator): phase 8 review - #6892

Draft
blunteshwar wants to merge 1 commit into
menu-separator-phases-2-7from
menu-separator-phase-8-review
Draft

blunteshwar wants to merge 1 commit into
menu-separator-phases-2-7from
menu-separator-phase-8-review

Conversation

@blunteshwar

Copy link
Copy Markdown
Contributor

Description

Phase 8 (review) of the swc-menu-separator migration. Stacked on #6891 (Phase 7) and targets the menu-separator-migration feature branch, so the diff includes earlier phases until they merge.

Epic: SWC-2668.

  • yarn lint:gen2 passes.
  • Ticks the review items in the migration plan.

Open items

  • Q3 (container height token update), Q4 (breaking-change tickets, 1st-gen deprecations), Q5, Q7, Q9 remain open in the plan.
  • Contrast (1.4.11), keyboard and screen reader testing, VRT, and the menu-dependent tests and stories wait on swc-menu-item and swc-menu-group.
  • The status table has no menu-separator row, so it was not updated.
  • Peer sign-off is pending.

Validation

  • yarn lint:gen2: passed
  • yarn workspace @spectrum-web-components/gen2 build: passed (Phase 7)
  • npx vitest --run --project storybook menu-separator: passed (Phase 6)

@changeset-bot

changeset-bot Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 3581e7c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@blunteshwar

Copy link
Copy Markdown
Contributor Author

No code issues. Notes: the plan now ticks "PR created" with a hard-coded PR range, which will go stale. The status table (01_status.md) still has no menu-separator row, so the "Status table updated" item rightly stays unticked, and someone needs to decide whether the table's columns apply to this component. Peer sign-off is outstanding.

@blunteshwar
blunteshwar force-pushed the menu-separator-phase-8-review branch from 8562f97 to 3581e7c Compare November 2, 2026 04:43
@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

📚 Branch Preview Links

🔍 Gen1 Visual Regression Test Results

When a visual regression test fails (or has previously failed while working on this branch), its results can be found in the following URLs:

Deployed to Azure Blob Storage: pr-6892

If the changes are expected, update the current_golden_images_cache hash in the circleci config to accept the new images. Instructions are included in that file.
If the changes are unexpected, you can investigate the cause of the differences and update the code accordingly.

@blunteshwar
blunteshwar changed the base branch from menu-separator-migration to menu-separator-phases-2-7 November 2, 2026 06:43

This branch has not been deployed

No deployments
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