GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Corrected the reasoning in the 'Step into' command explanation and made the `Making bound methods with class fields` section easier to understand. by singhal-shagun · Pull Request #3972 · javascript-tutorial/en.javascript.info

Conversation

@singhal-shagun

@singhal-shagun singhal-shagun commented Jun 19, 2026

Copy link
Copy Markdown

This pull request make the following improvements in the literature:

  1. Corrected the reasoning as to why beginner-level Javascript learners can ignore the 3.1 Debugging in the browser --> “Step into”, hotkey F11 writeup.
  2. The 9.1 Class basic syntax --> Making bound methods with class fields writeup has been made easier to understand.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
singhal-shagun added 3 commits June 18, 2026 14:46
@singhal-shagun
Updated why beginners can ignore the 'Step into' command explanation
fd4c225 
Corrected a section of the writeup on 'Step into' command pertaining to beginners.

Instead of "as we don't have asynchronous calls yet", I suggest the words to be "as we haven't learnt about asynchronous calls yet".

The former suggests that Javascript lacks the ability to make asynchronous calls, which is not the case, in my opinion.
@singhal-shagun
Made the section on bound methods easier to understand.
598d566
@singhal-shagun
.
d3e1443
@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes labels Jun 19, 2026
@javascript-translate-bot
javascript-translate-bot requested a review from a team June 19, 2026 07:40
@singhal-shagun
singhal-shagun marked this pull request as ready for review June 19, 2026 07:41
@smith558

smith558 commented Jul 25, 2026

Copy link
Copy Markdown
Member

@codex review
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@smith558 smith558 self-assigned this Jul 25, 2026
chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Jul 25, 2026
View reviewed changes

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d3e1443ba7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread 1-js/09-classes/01-class/article.md
As demonstrated in the chapter <info:bind> functions in JavaScript have a dynamic `this`. It depends on the context of the call.

So if an object method is passed around and called in another context, `this` won't be a reference to its object any more.
Recall the **"losing `this`"** problem from the chapter <info:bind>. Functions in JavaScript have a dynamic `this` — it depends on how the function is called, not where it's defined. So, when an object method is passed around (e.g. as a callback to `setTimeout`, or as an event handler), `this` no longer points to the original object.

@chatgpt-codex-connector chatgpt-codex-connector Bot Jul 25, 2026

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

P2 Badge Qualify the call-site this rule

This newly added explanation is false for arrow functions: their this is lexical and therefore does depend on where they are defined. That distinction is essential here because the solution immediately below uses click = () => ... precisely to capture the instance's this; saying that JavaScript functions universally use call-site this makes the explanation contradict its own example. Please limit this rule to regular functions or methods.

Useful? React with 👍 / 👎.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@singhal-shagun

singhal-shagun commented Jul 26, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Author

Codex seems to have skipped the method vs function distinction in my writeup!

To clarify things, I clearly wrote the following in my writeup:

Making methods bound to object instances using class fields is a third solution. Such methods, called "bound methods" can be created with a very elegant syntax,

The intention is to convey to the reader that this third solution is applicable only to methods, not to functions in general.

Also, the original wirteup made no clarification as to what it meant by "bound methods". My writeup makes it explicit, making it easier for the reader to understand what the article intends to convey.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@smith558 smith558

Labels

review needed Review needed, please approve or request changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@singhal-shagun @smith558 @javascript-translate-bot
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.