Skip to content

Global object #3415

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

bogdanbacosca
Copy link
Contributor

line 8 - grammar fix
line 51 - removed unnecessary space
line 64, 72 - small improvements
newline 65

@@ -5,7 +5,7 @@ The global object provides variables and functions that are available anywhere.

In a browser it is named `window`, for Node.js it is `global`, for other environments it may have another name.

Recently, `globalThis` was added to the language, as a standardized name for a global object, that should be supported across all environments. It's supported in all major browsers.
Recently, `globalThis` was added to the language as a standardized name for a global object, that should be supported across all environments. It's supported in all major browsers.
Copy link
Member

Choose a reason for hiding this comment

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

  • I'm not a native eng speaker -

I tthink it needs to fix the hanging "that" too

omiting second comma
...name for a global object that should be supported...

adding something
...name for a global object, name that should be supported...

replacing it
...name for a global object, intended to be supported...
...name for a global object; it should be supported...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I replaced that should with intended to

@javascript-translate-bot javascript-translate-bot added the needs +1 One more review needed label Aug 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs +1 One more review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants