Skip to content

3rd set of JS active learning updates #40272

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

Merged
merged 8 commits into from
Jul 9, 2025

Conversation

chrisdavidmills
Copy link
Contributor

Description

As part of a larger project to make the challenges/reader exercises in MDN learn more consistent, this PR:

  • Removes "Active learning" wording.
  • Makes some general wording improvements and fixes.

Motivation

Additional details

Related issues and pull requests

@chrisdavidmills chrisdavidmills requested a review from a team as a code owner July 9, 2025 10:09
@chrisdavidmills chrisdavidmills requested review from hamishwillee and removed request for a team July 9, 2025 10:09
@github-actions github-actions bot added the Content:Learn Learning area docs label Jul 9, 2025
@chrisdavidmills chrisdavidmills requested a review from bsmth July 9, 2025 10:09
@github-actions github-actions bot added the size/m [PR only] 51-500 LoC changed label Jul 9, 2025
Copy link
Contributor

github-actions bot commented Jul 9, 2025

Preview URLs (7 pages)
External URLs (5)

URL: /en-US/docs/Learn_web_development/Core/Scripting/Events
Title: Introduction to events


URL: /en-US/docs/Learn_web_development/Core/Scripting/Functions
Title: Functions — reusable blocks of code


URL: /en-US/docs/Learn_web_development/Core/Scripting/JSON
Title: Working with JSON


URL: /en-US/docs/Learn_web_development/Core/Scripting/Return_values
Title: Function return values

(comment last updated: 2025-07-09 15:46:39)

Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Thank you! Couple of small comments to consider, but nothing major. Ready when you have a look 👍🏻

chrisdavidmills and others added 7 commits July 9, 2025 16:23
…/index.md

Co-authored-by: Brian Smith <brian@smith.berlin>
Co-authored-by: Brian Smith <brian@smith.berlin>
…n_function/index.md

Co-authored-by: Brian Smith <brian@smith.berlin>
…n_function/index.md

Co-authored-by: Brian Smith <brian@smith.berlin>
…n_function/index.md

Co-authored-by: Brian Smith <brian@smith.berlin>
…n_function/index.md

Co-authored-by: Brian Smith <brian@smith.berlin>
@chrisdavidmills chrisdavidmills merged commit 6149deb into mdn:main Jul 9, 2025
8 checks passed
@chrisdavidmills chrisdavidmills deleted the js-active-learning-3 branch July 9, 2025 15:47
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.

2 participants