Skip to content
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

Lessons - Consider moving Take Quiz button above Complete Complete button #2739

Closed
kathrynwp opened this issue Jul 22, 2024 · 17 comments
Closed
Assignees
Labels
[Component] Learn Theme Website development issues related to the Learn theme. [Dev] Needs Design [Type] Enhancement New feature request for the Learn website.

Comments

@kathrynwp
Copy link
Collaborator

Issue

"Take Quiz" button could be overlooked.

Detail

A learner has reported that they did not notice the "Take Quiz" button, and they completed the lesson without taking the quiz. This resulted in a 0% grade for the quiz.

Suggestion

Could we move the "Take Quiz" button above the "Complete Lesson" button, or otherwise make it more obvious?

Saving_Your_Work__Create_Block_Theme___Learn_WordPress

Other

I set this to the post-launch milestone as only one learner has reported this issue so far.

@kathrynwp kathrynwp added [Type] Enhancement New feature request for the Learn website. [Component] Learn Theme Website development issues related to the Learn theme. labels Jul 22, 2024
@adamwoodnz
Copy link
Contributor

Wow I'm struggling to see how it could be more obvious in that blue! Jokes aside it is using the primary button style which is supposed to make it the most obvious next step. 'Complete lesson' is using secondary. Swapping them could be an option.

@WordPress/meta-design ?

@adamwoodnz adamwoodnz moved this to 🛑 Pending discussion in WordPress.org Jul 24, 2024
@fcoveram
Copy link

From the screenshot attached, I can't understand the user flow related to this report.

The "Take quiz" button is placed without context. It's not clear what the quiz is about, how much filling it out might take, and any additional info before clicking on it.

Missing the action could have been related to the lack of info rather than the button style, which the primary one in blue has sufficient contrast to be noticed and doesn't need to be changed.

@kathrynwp
Copy link
Collaborator Author

From the screenshot attached, I can't understand the user flow related to this report.

I believe the learner is supposed to complete the quiz before completing the lesson. @jonathanbossenger @westnz - could you please confirm?

In this learner's case, they clicked "Complete lesson" without taking the quiz, resulting in a grade of 0% for that quiz.

Since the "Complete Lesson" button is to the left of the "Quiz" button, and since English is a left-to-right language, perhaps this is part of the issue?

@westnz
Copy link
Collaborator

westnz commented Jul 24, 2024

Swapping them could be an option.

This sounds like a good solution

I believe a quiz makes sense within the context of individual lessons. We want learners to use a quiz to test their knowledge, but you can complete the lesson without completing the quiz. Some adult learners don't want to be blocked by quizzes and work through the content at their own pace. Some people might only want to complete one module as they may already understand some other content within the course. I like the idea of swapping the buttons to make it stand out more.

@adamwoodnz
Copy link
Contributor

To clarify; by 'swapping them' I meant swapping their position, rather than swapping the primary/secondary style. Is that what you understood @westnz ?

@westnz
Copy link
Collaborator

westnz commented Jul 24, 2024

Yes, that is what I understood 👍

@adamwoodnz
Copy link
Contributor

adamwoodnz commented Aug 6, 2024

What do you think about moving the primary action 'Take quiz' to the left @fcoveram?

Image

@kathrynwp
Copy link
Collaborator Author

@adamwoodnz I like what I see in your screenshot! We really want learners to take the quiz before clicking the button to complete the course (so they don't end up with a 0% grade) so moving it to the left makes a lot of sense to me.

@fcoveram
Copy link

fcoveram commented Aug 6, 2024

The change looks good 👍

@adamwoodnz adamwoodnz moved this from 🛑 Pending discussion to 📋 To do in WordPress.org Aug 6, 2024
@jasmussen
Copy link

Separate note on the screenshot here, and I think I've recorded this as feedback somewhere in the repo. Content-wise we need to update that text-highlight style (the optional exercise). That italic doesn't look great (faux italic?), and the colors and left border are part of legacy styles.

@adamwoodnz adamwoodnz self-assigned this Aug 8, 2024
@adamwoodnz
Copy link
Contributor

Separate note on the screenshot here, and I think I've recorded this as feedback somewhere in the repo. Content-wise we need to update that text-highlight style (the optional exercise). That italic doesn't look great (faux italic?), and the colors and left border are part of legacy styles.

These are using the Callout block, and need to be migrated to the new Notice block. I'm not sure the best way to communicate this, content team?

For Dev Resources we did add some styles to make callouts look more like notices, I'll take a look another look at that and see whether it could be shared or if we could find a more global solution.

@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in WordPress.org Aug 8, 2024
@jasmussen
Copy link

Whoever might take it, there are some designs here that would would work fine.

@kathrynwp
Copy link
Collaborator Author

@adamwoodnz Heya just wanted to double-check so I miss something, is there anything you need folks manually update across lessons, i.e. moving text from Callout to Notice block? Or is this something that will be migrated on the dev side?

@adamwoodnz
Copy link
Contributor

@adamwoodnz Heya just wanted to double-check so I miss something, is there anything you need folks manually update across lessons, i.e. moving text from Callout to Notice block? Or is this something that will be migrated on the dev side?

I think maybe the solution will be adding styles that update the appearance of the callout blocks, but from now on we should cease using them, and if we're updating a lesson then copying the text into a notice block would be ideal.

@westnz
Copy link
Collaborator

westnz commented Aug 8, 2024

Thanks Adam, I agree!

adamwoodnz added a commit to WordPress/wporg-parent-2021 that referenced this issue Sep 24, 2024
Add styles matching the notice block so that usage of the legacy callout blocks share the new look.

See WordPress/Learn#2739 (comment)
@adamwoodnz
Copy link
Contributor

@adamwoodnz Heya just wanted to double-check so I miss something, is there anything you need folks manually update across lessons, i.e. moving text from Callout to Notice block? Or is this something that will be migrated on the dev side?

Callout blocks are now rendered as Notice blocks, so there is no need to go back and convert older content. However for new content use Notice blocks, or if you are updating old content converting to Notice blocks is ideal.

cc @westnz @jonathanbossenger

@westnz
Copy link
Collaborator

westnz commented Oct 23, 2024

Many thanks, @adamwoodnz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Learn Theme Website development issues related to the Learn theme. [Dev] Needs Design [Type] Enhancement New feature request for the Learn website.
Projects
Status: Done
Status: Done
Development

No branches or pull requests

5 participants