Skip to content

[Typo]- In chapter Arrays in the section of loops we have to changed number to index#3949

Open
sajwan05 wants to merge 3 commits intojavascript-tutorial:masterfrom
sajwan05:fix-index
Open

[Typo]- In chapter Arrays in the section of loops we have to changed number to index#3949
sajwan05 wants to merge 3 commits intojavascript-tutorial:masterfrom
sajwan05:fix-index

Conversation

@sajwan05
Copy link

@sajwan05 sajwan05 commented Mar 22, 2026

In chapter Arrays in the section Loops we have for...of loop and there we changed number to index for more accurate language.

Issue: #3940

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Mar 22, 2026
@javascript-translate-bot javascript-translate-bot requested a review from a team March 22, 2026 19:01
@joaquinelio
Copy link
Member

It looks like this branch was created from another branch, so it includes those commits as well.

No big deal.

The index fix is 👍
The "capitalized" fix still lacks the final d.

@sajwan05
Copy link
Author

sajwan05 commented Mar 24, 2026

Thanks for the suggestion but
I think capitalize is right word here instead of capitalized.
Because capitalize is base form of verb, which can be used for some action which can happen now or in the future.
But if you still thinks "d" needs to be added let me know, I will make the changes.

@joaquinelio
Copy link
Member

oh, I see.

I'm not a native English speaker, so I asked chatgpt :

«
I see what you meant: using capitalize as a sort of label, like saying “in uppercase.”
The problem is that English doesn’t use capitalize that way. If we want the neutral/label form, the idiomatic option is still capitalized.
So even though your intention makes sense, capitalized is the correct and natural choice here.
»

@sajwan05
Copy link
Author

« I see what you meant: using capitalize as a sort of label, like saying “in uppercase.” The problem is that English doesn’t use capitalize that way. If we want the neutral/label form, the idiomatic option is still capitalized. So even though your intention makes sense, capitalized is the correct and natural choice here. »

Oh I see, thanks for this, I have changed capitalize to capitalized.

@sajwan05
Copy link
Author

@joaquinelio Hey I did the requested changes. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review needed Review needed, please approve or request changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants