Skip to content

Use descriptive text links and add keyboard markup in 3.getting-started.md #57

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

Lamwaunga
Copy link
Contributor

Hi, I wondered to the get started page wanting to find out more and thought some little things on that page could be better if you don't mind these little changes:

  • Replaced "click here" type of links with descriptive text.
  • Use Markdown keyboard mark-up.
  • Add periods to end of sentences.

Copy link

netlify bot commented May 8, 2025

Deploy Preview for emulatorjs-org ready!

Name Link
🔨 Latest commit 94a54a6
🔍 Latest deploy log https://app.netlify.com/sites/emulatorjs-org/deploys/681cd9e2efd0720008dfcc7e
😎 Deploy Preview https://deploy-preview-57--emulatorjs-org.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 configuration.

Copy link
Member

@ethanaobrien ethanaobrien 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

@ethanaobrien
Copy link
Member

@allancoding or @michael-j-green please review and merge

Copy link
Member

@allancoding allancoding left a comment

Choose a reason for hiding this comment

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

Looks good

@allancoding allancoding merged commit a3f1277 into EmulatorJS:main May 8, 2025
8 checks passed
@Lamwaunga Lamwaunga deleted the links-and-keyboard-in-3-getting-started-md branch May 10, 2025 08:46
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.

3 participants