Skip to content

feat: Add styling for keyboard keys#2478

Merged
szaganek merged 3 commits intomasterfrom
add-keyboard-key-component
Apr 30, 2026
Merged

feat: Add styling for keyboard keys#2478
szaganek merged 3 commits intomasterfrom
add-keyboard-key-component

Conversation

@szaganek
Copy link
Copy Markdown
Contributor

@szaganek szaganek commented Apr 29, 2026

Adds styling to the <kbd> element in Markdown.

Preview

https://pr-2478.preview.docs.apify.com/platform/integrations/mcp#client-configuration

In light mode:
Screenshot 2026-04-29 at 11 22 51

In dark mode:
Screenshot 2026-04-29 at 11 22 34

@szaganek szaganek requested review from TC-MO and barjin April 29, 2026 12:44
@szaganek szaganek self-assigned this Apr 29, 2026
@szaganek szaganek requested a review from marcel-rbro as a code owner April 29, 2026 12:44
@szaganek szaganek added the t-docs Issues owned by technical writing team. label Apr 29, 2026
@szaganek szaganek linked an issue Apr 29, 2026 that may be closed by this pull request
@apify-service-account
Copy link
Copy Markdown

Preview for this PR was built for commit b91b1666 and is ready at https://pr-2478.preview.docs.apify.com!

Copy link
Copy Markdown
Member

@barjin barjin left a comment

Choose a reason for hiding this comment

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

Thank you @szaganek !

I am a fan of semantic HTML (but I admit I don't know much about styling / React best practices). Is there a reason not to use the original kbd element?

It's well supported in browsers and with the same styles applied, it looks nearly indistinguishable from the proposed solutions ⬇️

Image

@B4nan
Copy link
Copy Markdown
Member

B4nan commented Apr 29, 2026

Agreed, we already use <kbd> for the search component btw.

image

@szaganek
Copy link
Copy Markdown
Contributor Author

Makes sense, I'll rewrite it. Thanks!

@szaganek szaganek changed the title feat: Add component for styling keyboard keys feat: Add styling for keyboard keys Apr 30, 2026
@szaganek szaganek requested a review from barjin April 30, 2026 07:05
@apify-service-account
Copy link
Copy Markdown

Preview for this PR was built for commit c32a8ff6 and is ready at https://pr-2478.preview.docs.apify.com!

Copy link
Copy Markdown
Member

@barjin barjin left a comment

Choose a reason for hiding this comment

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

lgtm, thank you @szaganek !

@szaganek szaganek merged commit 6fe46a8 into master Apr 30, 2026
17 checks passed
@szaganek szaganek deleted the add-keyboard-key-component branch April 30, 2026 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-docs Issues owned by technical writing team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add styling for keyboard keys

4 participants