Skip to content

Conversation

@chrisdavidmills
Copy link
Contributor

Description

I got some feedback that the ordering of the "Test your skills" articles has become confusing since the last update. Specifically, in some of the tests, the "solution" live sample appears before the "starting state" live sample, so it is unclear which one to click on to start the test.

To remedy this, I have decided to go through all the "Test your skills" articles and make the structure as consistent as possible throughout, fixing the above issue in the process.

This PR in particular updates the structure of the Dynamic scripting with JavaScript "Test your skills" articles.

Note: Not all of the tests show the finished state of the example. I have elected not to show it in cases where the before and after rendering look no different, where the final rendering gives away the answer, and where the question is freeform so there is no single answer.

Motivation

Additional details

Related issues and pull requests

@chrisdavidmills chrisdavidmills requested a review from a team as a code owner January 29, 2026 09:16
@chrisdavidmills chrisdavidmills requested review from hamishwillee and removed request for a team January 29, 2026 09:16
@github-actions github-actions bot added the Content:Learn Learning area docs label Jan 29, 2026
@github-actions github-actions bot added the size/l [PR only] 501-1000 LoC changed label Jan 29, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 29, 2026

Preview URLs (10 pages)

Flaws (4)

Note! 6 documents with no flaws that don't need to be listed. 🎉

Found an unexpected or unresolvable flaw? Please report it here.

URL: /en-US/docs/Learn_web_development/Core/Scripting/Test_your_skills/Arrays
Title: Test your skills: Arrays
Flaw count: 1

  • macros:
    • Macro ? produces link /en-US/curriculum/ which doesn't resolve

URL: /en-US/docs/Learn_web_development/Core/Scripting/Test_your_skills/Conditionals
Title: Test your skills: Conditionals
Flaw count: 1

  • macros:
    • Macro ? produces link /en-US/curriculum/ which doesn't resolve

URL: /en-US/docs/Learn_web_development/Core/Scripting/Test_your_skills/Events
Title: Test your skills: Events
Flaw count: 1

  • macros:
    • Macro ? produces link /en-US/curriculum/ which doesn't resolve

URL: /en-US/docs/Learn_web_development/Core/Scripting/Test_your_skills/Math
Title: Test your skills: Math
Flaw count: 1

  • macros:
    • Macro ? produces link /en-US/curriculum/ which doesn't resolve

(comment last updated: 2026-01-30 16:00:07)

@chrisdavidmills chrisdavidmills marked this pull request as draft January 29, 2026 18:45
@chrisdavidmills
Copy link
Contributor Author

Converting to draft until I've looked through the review at #42953.

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/l [PR only] 501-1000 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant