Skip to content

Conversation

veracioux
Copy link
Contributor

@veracioux veracioux commented Oct 7, 2025

Before:
image
After:
image
Edit:
After discussion, settled on this version:
(Mac)
image
(PC)
image

Tested on:

Chrome: Version 136.0.7103.92 (Official Build) (64-bit)
Mozilla Firefox 140.0.4
(Linux)

@ZiedYousfi
Copy link

Wouldn't "Ctrl + K" be more explicit ?

@veracioux
Copy link
Contributor Author

veracioux commented Oct 7, 2025

@ZiedYousfi Yeah, I can do that. Some other variants:
image

image

@ZiedYousfi
Copy link

I feel like both of those are good but maybe the first one would be less visually overwhelming imo

@veracioux
Copy link
Contributor Author

@ZiedYousfi I agree the first one in #3021 (comment) is best.

@veracioux veracioux marked this pull request as draft October 9, 2025 18:37
@jayair
Copy link
Contributor

jayair commented Oct 11, 2025

Yeah the first version looks good, here's what it looks like on the Mac:

Screenshot 2025-10-10 at 10 20 00 PM

@veracioux
Copy link
Contributor Author

@jayair The screenshot is without my fix applied, right?

@veracioux
Copy link
Contributor Author

@jayair @ZiedYousfi Here's the newest version:
(Mac)
image
(PC)
image

@veracioux veracioux marked this pull request as ready for review October 11, 2025 10:36
Copy link

@ZiedYousfi ZiedYousfi left a comment

Choose a reason for hiding this comment

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

Nice

@jayair
Copy link
Contributor

jayair commented Oct 12, 2025

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
@veracioux
Copy link
Contributor Author

@jayair Done.
image

image

@rekram1-node rekram1-node changed the title fix: Fix Ctrl+K styling in documentation docs: fix Ctrl+K styling in documentation Oct 16, 2025
@rekram1-node
Copy link
Collaborator

gonna merge since jay said it looked good

@rekram1-node rekram1-node merged commit fb4105a into sst:dev Oct 16, 2025
3 checks passed
@jayair
Copy link
Contributor

jayair commented Oct 16, 2025

Thanks!

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.

5 participants