-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Add quiz component for FWE #12648
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
Add quiz component for FWE #12648
Conversation
|
@schultek Is this PR ready for review? |
3ac390c to
1c8c779
Compare
1af7d6d to
700efd4
Compare
|
Visit the preview URL for this PR (updated for commit 0bf4f18): https://flutter-docs-prod--pr12648-feat-quiz-component-wo094agg.web.app |
|
@parlough The initial implementation is ready and can be tested on the '/fwe' page locally. What I'm not sure yet is how the flow of answering multiple questions should be. Can you define this a bit more in detail in the project item? |
Yes of course. Check the end of the project item, I added some screenshots/mockups of the flow I was thinking. Roughly:
|
5e2c736 to
3441b60
Compare
3441b60 to
691aabc
Compare
|
@parlough All done.
Question: On "Try again" just reset the current question or the whole quiz (e.g. when answering q2 incorrect, reset to q1 or just re-answer q2)? |
|
Thanks for those updates, looks amazing.
Just the current question, so the implementation you have now looks great! Side request. Would it be challenging to add a less prominent "Previous" button to the left of the "Next question" button after the 1st question? So users can review their previous answers and the explanation. |
|
Done |
parlough
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall this looks and feels perfect @schultek. As always, thank you!
The main remaining ask I have is to convert the questions to buttons for improved accessible. Other than that, some small structural suggestions.
parlough
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it! Thanks for making those adjustments and accessibility fixes.
Adds a Quiz component for use in FWE.