Skip to content

removes choiceresponse wiping after clicking Show Answer - #247

Merged
valera-rozuvan merged 1 commit into
masterfrom
alex/fix_choiceresponse
Jun 25, 2013
Merged

removes choiceresponse wiping after clicking Show Answer#247
valera-rozuvan merged 1 commit into
masterfrom
alex/fix_choiceresponse

Conversation

@auraz

@auraz auraz commented Jun 22, 2013

Copy link
Copy Markdown
Contributor

From the ticket: "If you click "show answers", then click "check", your own answers are wiped out (presumably because the radio buttons are disabled). I think this is a general issue with multiple choice questions. The only way around this is to click "show answers", navigate to another page, go back to the original page, then hit "check" again."

Support is receiving a lot of mail about this, and numerous professors have complained.

Description in https://edx-wiki.atlassian.net/browse/BLD-128, complaints in https://edx.lighthouseapp.com/projects/101932/tickets/219-Multiple-Choice-clears-answers

Sample problems (for easy testing) added to https://github.com/MITx/content-demos repo in /Testing/Testing:Choiceresponsebug/ sequence.

@sarina @arjun810 please review.

@arjun810

Copy link
Copy Markdown
Contributor

I can't access JIRA, so I can't comment there, but I don't think it's necessary. I think (but I'm not 100% sure) that the reason we disabled it before was that we used to use the checkboxes themselves to show which answers were correct, but then later were able to do so using the green check / red x.

@sarina

sarina commented Jun 24, 2013

Copy link
Copy Markdown
Contributor

@auraz you should include the text from JIRA tickets, along with the link to JIRA, in each PR. Because we are open source, but JIRA is not, all PRs should be able to stand on their own (but include the JIRA reference for those who do have access).

@sarina

sarina commented Jun 24, 2013

Copy link
Copy Markdown
Contributor

@arjun810

I think (but I'm not 100% sure) that the reason we disabled it before was that we used to use the checkboxes themselves to show which answers were correct, but then later were able to do so using the green check / red x.

I think that makes sense. However at this point, multiplechoiceresponse doesn't behave like any other problem, which is not a good thing. For example, string response, code response, formula response, and
option response, if you click "Show Answer", you are allowed to change your
answer while the answer is displayed (so yes, this allows you to click
"Show Answer" then just copy all the right answers into the boxes). However for multiple choice response, clicking "Show Answer" greys out the radio/check buttons and disallows you from changing your answer in response
to the new answer being displayed. This UX is inconsistent.

(just trying to share details of the JIRA/lighthouse ticket with you so you're up to speed on the PR)

@sarina

sarina commented Jun 24, 2013

Copy link
Copy Markdown
Contributor

👍 looks good to me, and behavior now matches behavior of other types of problems.

valera-rozuvan added a commit that referenced this pull request Jun 25, 2013
removes choiceresponse wiping after clicking Show Answer
@valera-rozuvan
valera-rozuvan merged commit 08b994b into master Jun 25, 2013
@valera-rozuvan
valera-rozuvan deleted the alex/fix_choiceresponse branch June 25, 2013 12:55
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
diegomillan referenced this pull request in eduNEXT/edx-platform Sep 14, 2016
…e-all

Make keyword substitution discoverable in studio
CrewS pushed a commit to CrewS/edx-platform-1 that referenced this pull request Jan 17, 2019
edx-secure pushed a commit that referenced this pull request Apr 15, 2022
The ony use is a GET request in admin portal so this view need not be post/put friendly right now.
It may actually get removed in an upcoming iteration, or stay readonly.

Fixes: SEC-1418
timmc-edx pushed a commit that referenced this pull request Apr 15, 2022
The ony use is a GET request in admin portal so this view need not be post/put friendly right now.
It may actually get removed in an upcoming iteration, or stay readonly.

Fixes: SEC-1418

Co-authored-by: Binod Pant <bpant@edx.org>
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
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.

4 participants