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 CSS layout "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 28, 2026 16:51
@chrisdavidmills chrisdavidmills requested review from hamishwillee and removed request for a team January 28, 2026 16:51
@github-actions github-actions bot added Content:Learn Learning area docs size/m [PR only] 51-500 LoC changed labels Jan 28, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 28, 2026

Preview URLs (4 pages)

Flaws (4)

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

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

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

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

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

URL: /en-US/docs/Learn_web_development/Core/CSS_layout/Test_your_skills/Grid
Title: Test your skills: CSS grids
Flaw count: 1

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

URL: /en-US/docs/Learn_web_development/Core/CSS_layout/Test_your_skills/Position
Title: Test your skills: Positioning
Flaw count: 1

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

(comment last updated: 2026-01-30 14:09:48)

Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

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

Thanks @chrisdavidmills.
That's a good strategy to avoid confusion and makes better sense - adding the starting point rendering before the expected output. I have a small suggestion to reorder the frames within the starting blocks: show the rendered iframe first and then the corresponding code snippets.
I've also left a few small edits in nearby text.

Leaving a +1

chrisdavidmills and others added 12 commits January 30, 2026 13:22
…ills/flexbox/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/flexbox/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/flexbox/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/floats/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/grid/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/position/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/position/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/position/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/position/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/grid/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/grid/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/grid/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
chrisdavidmills and others added 4 commits January 30, 2026 13:31
…ills/grid/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/floats/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/floats/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…ills/grid/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@chrisdavidmills chrisdavidmills merged commit 143f734 into mdn:main Jan 30, 2026
7 checks passed
@chrisdavidmills chrisdavidmills deleted the reorder-test-your-skills-3 branch January 30, 2026 14:11
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