Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add comma for better readability #3430

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

Closed
KrappRamiro wants to merge 1 commit into javascript-tutorial:master from KrappRamiro:patch-1
Closed

Add comma for better readability #3430

KrappRamiro wants to merge 1 commit into javascript-tutorial:master from KrappRamiro:patch-1

Conversation

@KrappRamiro
Copy link

@KrappRamiro KrappRamiro commented Apr 12, 2023

I think a comma (,) after every "In the code below" makes the text more readable. For example, there is a line that says:

In the code below prompt asks for width, but not for title:

It can be mistakenly interpreted as "look for the code below prompt() " , instead of "look how prompt works in the code below" Have a good day :)

I think a comma (,) after every "In the code below" makes the text more readable.
For example, there is a line that says:
In the code below `prompt` asks for `width`, but not for `title`:
It can be mistakenly interpreted as "look for the code below `prompt()` " , instead of "look how `prompt` works in the code below"
Have a good day :)
Copy link

CLAassistant commented Apr 12, 2023
edited
Loading

CLA assistant check
All committers have signed the CLA.

Copy link
Member

👍 I think the comma Is mandatory there,
but Im not a native speaker.

msnitish reacted with thumbs up emoji

@KrappRamiro KrappRamiro closed this by deleting the head repository Apr 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@joachimklug joachimklug joachimklug approved these changes

@shallow-beach shallow-beach shallow-beach approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

needs +1 One more review needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /