Skip to content

FE-1732: Add default max width to menu component lists - #9753

Merged
alex-e-leon merged 1 commit into
mainfrom
FE-1732-default-menu-width
Sep 16, 2026
Merged

alex-e-leon merged 1 commit into
mainfrom
FE-1732-default-menu-width

Conversation

@alex-e-leon

Copy link
Copy Markdown
Contributor

🌟 What is the purpose of this PR?

See title

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

@vercel

vercel Bot commented Sep 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
hash Ready Ready Preview Sep 16, 2026 8:10am UTC
hashdotdesign-tokens Ready Ready Preview Sep 16, 2026 8:10am UTC
petrinaut Ready Ready Preview Sep 16, 2026 8:10am UTC
petrinaut-docs Ready Ready Preview Sep 16, 2026 8:10am UTC

Request Review

@cursor

cursor Bot commented Sep 16, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
CSS-only layout constraints in design-system recipes with no auth, data, or runtime logic changes.

Overview
Menu-style SelectableList popovers now get a default maxWidth of min(500px, 70vw) on their content slot, so long option lists don’t stretch unbounded on wide viewports.

The Filter sort menu keeps its tighter 300px cap by moving that rule under a && selector, so it reliably overrides the new shared menu default when specificity would otherwise tie on extraction order.

Reviewed by Cursor Bugbot for commit b9077c4. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions github-actions Bot added area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team area/apps > hash.design Affects the `hash.design` design site (app) labels Sep 16, 2026
@alex-e-leon
alex-e-leon added this pull request to the merge queue Sep 16, 2026
Merged via the queue into main with commit c3c24bd Sep 16, 2026
64 checks passed
@alex-e-leon
alex-e-leon deleted the FE-1732-default-menu-width branch September 16, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/apps > hash.design Affects the `hash.design` design site (app) area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team

Development

Successfully merging this pull request may close these issues.

2 participants