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

Fix Clear UI with correct answers #35

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mostafanasser2000
Copy link
Contributor

case when the user answers a question incorrectly

before

before

after

after

case when the users don't answer a question

Screenshot from 2024-05-04 23-58-28

@alexeygrigorev

@alexeygrigorev
Copy link
Member

How does it work?

I'll add a bit more context to the original issue

@alexeygrigorev
Copy link
Member

alexeygrigorev commented May 13, 2024

I think this is not right behavior because it doesn't show the correct answer in Q2, and the original problem is still there - the correct answer is selected in the last picture

@alexeygrigorev
Copy link
Member

Perhaps we could just add a line saying "no answer was submitted"? or something like that

@mostafanasser2000
Copy link
Contributor Author

mostafanasser2000 commented May 13, 2024

what I understand from the context you just added is that if a student submits homework and there is a question he doesn't submit we display the correct answer as well as a "no answer was submitted" label and if he chooses the wrong answer for a question we display the correct answer as well as the wrong answer like the old behavior.

here is what I think you want

scenario1: student not answer a question

scenario3

scenario2: student does not submit a homework

scenario1

@alexeygrigorev
Copy link
Member

alexeygrigorev commented May 14, 2024

Looks great! I think for both scenarios it could be the same UI - i.e. the yellow thing with "no answer was submitted" is okay for scenario 2, I think. What do you think?

@mostafanasser2000
Copy link
Contributor Author

I think it will be ok to add it as well to homework that the student has not submitted, and what if we change the label from "No answer was submitted" to "Question not answered" for clarity

@mostafanasser2000
Copy link
Contributor Author

final version

scenario1: student not answer a question

Screenshot from 2024-05-15 14-32-21

scenario2: student does not submit a homework

Screenshot from 2024-05-15 14-29-55

@alexeygrigorev
Copy link
Member

looks great! can you please update the code? will be happy to accept it

@alexeygrigorev
Copy link
Member

okay just checked - it looks like the code is updated. is it the final version?

@alexeygrigorev
Copy link
Member

I think adding some tests will not hurt. Do they pass now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clearer UI with correct answers
2 participants