Skip to content

Touchups: Use <kbd> for specifying Keyboard Keys and Shortcuts#3760

Merged
cortinico merged 1 commit into
react:mainfrom
Pranav-yadav:Pranav-yadav/kbd-keys-formatting
Jun 19, 2023
Merged

Touchups: Use <kbd> for specifying Keyboard Keys and Shortcuts#3760
cortinico merged 1 commit into
react:mainfrom
Pranav-yadav:Pranav-yadav/kbd-keys-formatting

Conversation

@Pranav-yadav

@Pranav-yadav Pranav-yadav commented Jun 13, 2023

Copy link
Copy Markdown
Contributor

Summary

Currently, some docs are diverged from using <kbd> and rather use ` for formatting Keyboard Keys and Shortcuts.

This diff updates docs to use the <kbd> HTML tag for formatting the same and of course, adds some formatting touchups.

Changelog

[UI] Touchups: Use <kbd> for specifying Keyboard Keys and Shortcuts.


P.S.: I came across these occurrences when working on #3732 😇

@Pranav-yadav

Copy link
Copy Markdown
Contributor Author

It should be nice if we can backport the same to a few supported versions ?

@netlify

netlify Bot commented Jun 13, 2023

Copy link
Copy Markdown

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 49ebd10
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/6488ad70b9fa15000980eb85
😎 Deploy Preview https://deploy-preview-3760--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@cortinico

Copy link
Copy Markdown
Contributor

Thanks for taking a stab at this @Pranav-yadav

@Pranav-yadav

Copy link
Copy Markdown
Contributor Author

ping @Simek, if it looks good to u we may merge this so it doesn't conflicts with other PRs :)

@cortinico cortinico merged commit 6873109 into react:main Jun 19, 2023
@Pranav-yadav Pranav-yadav deleted the Pranav-yadav/kbd-keys-formatting branch June 19, 2023 15:43
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