(comment last updated: 2025-08-12 16:23:41)

Thanks for the feedback, @hamishwillee. I'm taking this all as positive — your feelings match my intent for the changes.

I've run out of Mozilla hours this month, but I'll be back working on this in August. This'll also give @bsmth and others time to review and provide feedback if they have anything else to add.

They are positive. I have reservations, but they are purely selfish because I am not the target audience :-).

• There should be no orphan pages.

I've added a link to the "Test your skills" index page on the main "Accessibility" index page, just so it won't be an orphan page. I think any other solution/reorganization will make things a bit too complicated.

They need to be highlighted, but why not just a Test: prefix

@hamishwillee I've tried doing that for all the "Test your skills" article short titles, and I think it looks fine. Updated in my next commit.

Looking good. Couple of nits. If I approve and merge this, is the "Experiment" over? I guess I'm asking whether you want me to leave this open for more feedback or if you are happy to continue rolling out the model. IMO it is better, so as long as we continue to the end of rolling this out everywhere I'm happy to merge.

Looking good. Couple of nits. If I approve and merge this, is the "Experiment" over? I guess I'm asking whether you want me to leave this open for more feedback or if you are happy to continue rolling out the model. IMO it is better, so as long as we continue to the end of rolling this out everywhere I'm happy to merge.

@hamishwillee I've removed the "Experiment" label ;-)

I'm happy to proceed with this and merge it, having had your feedback.

As you'll see, I'm moving ahead nicely with getting it done in all places. I'll probably have them done by the end of the week.

Hi @chrisdavidmills, I've taken a look now. I like that the tests are now part of the learning flow - this is great!

That said, I'm missing the visual cue in the sidebar. To me, the previous drop-down icon used for grouping the tests helped the tests stand out in the sidebar.
Thinking about how we could visually convey the interspersed tests in the "new sidebar" - I wonder if we can have a short title along these lines (or if you any other suggestions for a better emoji):
short-title: "🎯 Test: CSS/JS a11y"

True, like Hamish said, we're not the audience, so we're making assumptions about what might work better in their learning process. We do get learners from time to time on Discord. Maybe we could ask a few where they prefer to see the tests - in the flow or at the end. (we could even share this change in our next community call and see what folks think.)

I like the addition of "Test your skills" to the module landing page.

Hi @chrisdavidmills, I've taken a look now. I like that the tests are now part of the learning flow - this is great!

@dipikabh Yay!

That said, I'm missing the visual cue in the sidebar. To me, the previous drop-down icon used for grouping the tests helped the tests stand out in the sidebar. Thinking about how we could visually convey the interspersed tests in the "new sidebar" - I wonder if we can have a short title along these lines (or if you any other suggestions for a better emoji): short-title: "🎯 Test: CSS/JS a11y"

This is a really good point — I was also worried about them getting lost in the sidebar. In terms of what emoji to use, I can see the logic of using the target emoji to indicate a test or challenge, but I tried it, and I was a bit concerned that it is too hard to tell what it is, especially in the sidebar.

To move forward on this, my latest commit adds a different emoji to each "Test your skills" short title, so you can have a look and see if you like any of these better than the target:

• 🚩: indicates challenge, goal, or end of stage; maybe also a bit hard to see, especially in dark mode.
• 💡: indicates challenge, "your turn", or "thought required"
• ⭐: indicates challenge or reward; maybe not as obvious as the other options, but it stands out very clearly.

True, like Hamish said, we're not the audience, so we're making assumptions about what might work better in their learning process. We do get learners from time to time on Discord. Maybe we could ask a few where they prefer to see the tests - in the flow or at the end. (we could even share this change in our next community call and see what folks think.)

I honestly can't see how the majority of learners would have a more effective learning experience by having all the challenges bunched up at the end, rather than interspersed throughout the learning articles. And, even if they say they prefer them at the end, I think the temptation is much higher to skip them rather than work your way through them all.

I surveyed several learning resources, and they overwhelmingly have their tests interspersed in small chunks.

Also, the way I've got it in these test PRs, we have the index page linked at the end anyway, so you can revisit them in a group if you want to.

I like the addition of "Test your skills" to the module landing page.

Cool!

Thanks. Changes you made since my last look are good.

FWIW I much prefer the target followed closely by the star. The other two don't do it for me.

I'd like to get some other opinions here. I think that the target is conceptually good, but I am still concerned that it is hard to see what it is, especially in dark mode.

I like the flag 🚩 - it symbolizes a milestone.
It's a good enough marker (works for me at least) in dark mode.

I can bring up the options for a team vote on Monday and get back to you. Since Ruth and Vadim have been working on the new design system, they might have a better idea of what'll work in the new scheme or maybe even other suggestions.

I can bring up the options for a team vote on Monday and get back to you. Since Ruth and Vadim have been working on the new design system, they might have a better idea of what'll work in the new scheme or maybe even other suggestions.

That would be wonderful, thanks @dipikabh!

@chrisdavidmills, I have an update for you from our discussion in the team meeting.

The long-term solution would be to use an icon in the sidebar instead of a hardcoded emoji (the icon will be added to the end of the text, not the beginning like I was suggesting).
And since the test pages already have a page type, learn-module-assessment, that'll make it easier to add the chosen icon to the sidebar for all such pages in one go once the new front end is released.

We're using Lucide icons in the new front end. There was a suggestion to use book-open-check for the test pages.

So for now, we can probably skip adding any emoji, land the pages with the new navigation as-is, and follow up after the new front end is officially released, it shouldn't be long now.
(If you'd like, I imagine you could already open an issue to add an icon for the test pages – with the caveat of following up after fred is launched. OR we can wait and follow up with the engineering team after the launch.)

@chrisdavidmills, I have an update for you from our discussion in the team meeting.

The long-term solution would be to use an icon in the sidebar instead of a hardcoded emoji (the icon will be added to the end of the text, not the beginning like I was suggesting). And since the test pages already have a page type, learn-module-assessment, that'll make it easier to add the chosen icon to the sidebar for all such pages in one go once the new front end is released.

We're using Lucide icons in the new front end. There was a suggestion to use book-open-check for the test pages.

So for now, we can probably skip adding any emoji, land the pages with the new navigation as-is, and follow up after the new front end is officially released, it shouldn't be long now. (If you'd like, I imagine you could already open an issue to add an icon for the test pages – with the caveat of following up after fred is launched. OR we can wait and follow up with the engineering team after the launch.)

Great stuff, thanks @dipikabh! I have removed the emojis, and will merge these PRs for now. I'll then file an issue to request that the devs add the relevant lucide icon to those article short titles and titles after the fred launch is done.

I think book-open-check is a good choice too.

@dipikabh fred issue filed here: mdn/mdn#710