Skip to content

Add flex-wrap to buttons in All Directions story#7647

Open
llastflowers wants to merge 1 commit intomainfrom
llastflowers/14982/tooltip-all-directions-responsive-fix
Open

Add flex-wrap to buttons in All Directions story#7647
llastflowers wants to merge 1 commit intomainfrom
llastflowers/14982/tooltip-all-directions-responsive-fix

Conversation

@llastflowers
Copy link
Contributor

@llastflowers llastflowers commented Mar 9, 2026

Addresses https://github.com/github/accessibility-audits/issues/14982

Changelog

New

Added flex-wrap to buttons in All Directions Tooltip story to eliminate the need for horizontal scrolling on very small viewports (320x256)

Before:
Tooltip story buttons before changes

After:
Tooltip story buttons after changesTooltip story buttons after changes scrolled down

Rollout strategy

  • Patch release
  • Minor release
  • Major release; if selected, include a written rollout or migration plan
  • None (Storybook only)

Testing & Reviewing

Merge checklist

Copilot AI review requested due to automatic review settings March 9, 2026 17:51
@llastflowers llastflowers requested a review from a team as a code owner March 9, 2026 17:51
@llastflowers llastflowers requested a review from hectahertz March 9, 2026 17:51
@changeset-bot
Copy link

changeset-bot bot commented Mar 9, 2026

⚠️ No Changeset found

Latest commit: ae1e51a

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.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

Copilot AI reviewed Mar 9, 2026
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the accessibility and usability of the TooltipV2 Storybook feature stories at small viewport sizes (e.g. 320×256) by allowing the button rows to wrap instead of requiring horizontal scrolling.

Changes:

  • Added flex-wrap: wrap to .AllDirectionsRow and .ActionMenuRow CSS classes so buttons reflow to multiple rows on narrow viewports.

@llastflowers llastflowers added skip changeset This change does not need a changelog integration-tests: skipped manually Changes in this PR do not require an integration test labels Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@hectahertz hectahertz Awaiting requested review from hectahertz hectahertz is a code owner automatically assigned from primer/engineer-reviewers

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

integration-tests: skipped manually Changes in this PR do not require an integration test skip changeset This change does not need a changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@llastflowers