-
Notifications
You must be signed in to change notification settings - Fork 2.1k
docs: fix Ctrl+K styling in documentation #3021
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Wouldn't "Ctrl + K" be more explicit ? |
@ZiedYousfi Yeah, I can do that. Some other variants: ![]() |
I feel like both of those are good but maybe the first one would be less visually overwhelming imo |
@ZiedYousfi I agree the first one in #3021 (comment) is best. |
@jayair The screenshot is without my fix applied, right? |
@jayair @ZiedYousfi Here's the newest version: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice
Yeah this looks good, but let's drop the border on the individual characters. Thanks! |
Remove border and padding from kbd elements, adjust Cmd symbol line-height
@jayair Done. ![]() |
gonna merge since jay said it looked good |
Thanks! |
Before:




After:
Edit:
After discussion, settled on this version:
(Mac)
(PC)
Tested on:
Chrome: Version 136.0.7103.92 (Official Build) (64-bit)
Mozilla Firefox 140.0.4
(Linux)