Skip to content

Add max-width to kbd styles#3012

Merged
francinelucca merged 2 commits intomainfrom
francinelucca-patch-1
Mar 16, 2026
Merged

Add max-width to kbd styles#3012
francinelucca merged 2 commits intomainfrom
francinelucca-patch-1

Conversation

@francinelucca
Copy link
Member

Closes https://github.com/github/primer/issues/6368

What are you trying to accomplish?

Prevent kbd elements from overflowing.

What approach did you choose and why?

What should reviewers focus on?

Can these changes ship as is?

  • Yes, this PR does not depend on additional changes. 🚢

Copilot AI review requested due to automatic review settings March 16, 2026 19:48
@francinelucca francinelucca requested review from a team as code owners March 16, 2026 19:48
@francinelucca francinelucca requested a review from jonrohan March 16, 2026 19:48
@changeset-bot
Copy link

changeset-bot bot commented Mar 16, 2026

🦋 Changeset detected

Latest commit: 4bba8d4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/css Patch

Not sure what this means? Click here to learn what changesets are.

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

Add max-width styling to kbd elements in CSS.
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

Adds a constraint to kbd styling to avoid content spilling outside its container, and ships the change as a patch release via changeset.

Changes:

  • Add max-width: 100% to kbd base styles
  • Add a patch changeset for @primer/css

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/base/kbd.scss Constrains kbd sizing to its container width
.changeset/lazy-tigers-happen.md Announces patch release for the style adjustment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@francinelucca francinelucca merged commit 31df244 into main Mar 16, 2026
17 checks passed
@francinelucca francinelucca deleted the francinelucca-patch-1 branch March 16, 2026 20:02
@primer primer bot mentioned this pull request Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants