Skip to content

Comments

Removing 'this'#14463

Merged
teoli2003 merged 1 commit intomdn:mainfrom
Alexandru-O:patch-4
Mar 30, 2022
Merged

Removing 'this'#14463
teoli2003 merged 1 commit intomdn:mainfrom
Alexandru-O:patch-4

Conversation

@Alexandru-O
Copy link
Contributor

Arrow functions does not have their own 'this' context.
Replacing 'this' with 'volumeSlider' to access value property OR we can replace the arrow function with 'function ()' and use 'this' to access value.

Summary

Motivation

Supporting details

Related issues

Metadata

  • Adds a new document
  • Rewrites (or significantly expands) a document
  • Fixes a typo, bug, or other error

Arrow functions does not have their own 'this' context. 
Replacing 'this' with 'volumeSlider' to access value property or we can replace the arrow function with 'function ()' and use 'this' to access value.
@Alexandru-O Alexandru-O requested a review from a team as a code owner March 30, 2022 08:43
@Alexandru-O Alexandru-O requested review from teoli2003 and removed request for a team March 30, 2022 08:43
@github-actions github-actions bot added the Content:Learn Learning area docs label Mar 30, 2022
@github-actions
Copy link
Contributor

Preview URLs

Flaws

URL: /en-US/docs/Learn/JavaScript/Client-side_web_APIs/Introduction
Title: Introduction to web APIs
on GitHub
Flaw count: 1

  • broken_links:
    • Link points to the page it's already on

External URLs

URL: /en-US/docs/Learn/JavaScript/Client-side_web_APIs/Introduction
Title: Introduction to web APIs
on GitHub

No new external URLs

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

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

Good catch!

@teoli2003 teoli2003 merged commit 78dfe9d into mdn:main Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Learn Learning area docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants