Skip to content

docs: fix grammatical error and inconsistent ping output in RTT definition#43655

Merged
hamishwillee merged 1 commit intomdn:mainfrom
alideniztartma:main
Apr 6, 2026
Merged

docs: fix grammatical error and inconsistent ping output in RTT definition#43655
hamishwillee merged 1 commit intomdn:mainfrom
alideniztartma:main

Conversation

@alideniztartma
Copy link
Copy Markdown
Contributor

Description

I've fixed two issues in the RTT glossary page:

Added the missing "of" in "length of time".

Fixed the inconsistency in the ping output example where the command targeted example.com but the footer showed statistics for google.com.

Motivation

These changes improve the technical accuracy and professionalism of the MDN documentation, ensuring that beginners are not confused by inconsistent code examples and grammatical errors.
Additional details

No additional resources needed; these are direct fixes to the existing content.

…ition - Changed "length time" to "length of time" for better grammar.

- Changed "length time" to "length of time" for better grammar.
- Updated the ping statistics footer from "google.com" to "example.com" to match the command.
@alideniztartma alideniztartma requested a review from a team as a code owner April 2, 2026 19:51
@alideniztartma alideniztartma requested review from hamishwillee and removed request for a team April 2, 2026 19:51
@github-actions github-actions bot added Content:Glossary Glossary entries size/xs [PR only] 0-5 LoC changed labels Apr 2, 2026
Copy link
Copy Markdown
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

Thanks @aliden1z - looks good to me.

@hamishwillee hamishwillee merged commit f796ce2 into mdn:main Apr 6, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Glossary Glossary entries size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants