Skip to content

Update explanation of parameters and arguments#44660

Merged
chrisdavidmills merged 4 commits into
mdn:mainfrom
Lk-ux:patch-1
Jul 9, 2026
Merged

Update explanation of parameters and arguments#44660
chrisdavidmills merged 4 commits into
mdn:mainfrom
Lk-ux:patch-1

Conversation

@Lk-ux

@Lk-ux Lk-ux commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Clarify the distinction between parameters and arguments in function definitions.

Description

Corrected the note by clarifying that parameters and arguments are not same.

Motivation

Currently, the note incorrectly says that parameters are called arguments, properties, and attributes. My correction resolves this misunderstanding.

Additional details

Related issues and pull requests

Clarify the distinction between parameters and arguments in function definitions.
@Lk-ux Lk-ux requested a review from a team as a code owner July 7, 2026 18:31
@Lk-ux Lk-ux requested review from chrisdavidmills and removed request for a team July 7, 2026 18:31
@github-actions github-actions Bot added Content:Learn Learning area docs size/xs [PR only] 0-5 LoC changed labels Jul 7, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Preview URLs (2 pages)

(comment last updated: 2026-07-09 07:58:56)

@github-actions github-actions Bot added size/m [PR only] 51-500 LoC changed and removed size/xs [PR only] 0-5 LoC changed labels Jul 8, 2026

@chrisdavidmills chrisdavidmills left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, @Lk-ux, for your work on this. I agree that this needs to be called out more accurately. I decided to make a few changes to the PR for further improvements:

  1. I went through and made some general grammar improvements.
  2. I edited your note a bit and also promoted it to being part of the main content flow. I think it is too important to be hidden in a note.
  3. I changed a bunch of instances of "parameter" to "argument" as many of them were being wrongly used, as per your change.

I've approved it, but I won't merge it until you've reviewed my changes and let me know you're happy with them.

Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md Outdated
Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md Outdated
Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md Outdated
Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md
Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md Outdated
Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md Outdated
Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md Outdated
Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md Outdated
Comment thread files/en-us/learn_web_development/core/scripting/functions/index.md Outdated
@chrisdavidmills

Copy link
Copy Markdown
Contributor

Thanks, @Lk-ux. I've responded to all your comments in a new commit.

@Lk-ux

Lk-ux commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, @chrisdavidmills. I've reviewed the changes, and I'm happy with them. You can merge this now.

@chrisdavidmills chrisdavidmills merged commit 9460d58 into mdn:main Jul 9, 2026
9 checks passed
@Lk-ux Lk-ux deleted the patch-1 branch July 9, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Learn Learning area docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants