openlab-at-city-tech / webworkqa

WeBWorK integration for WordPress and BuddyPress
GNU General Public License v2.0
4 stars 2 forks source link

after asking a question, reply field is in preview mode #34

Closed moui72 closed 7 years ago

moui72 commented 8 years ago

after I asked a question about a problem, if I go to reply to myself (admittedly this may be an edge use case) the reply field is grayed out because I am in preview mode. I have to click 'edit' to enter text. This is a bit unintuitive.

response preview

WeBWorK url: http://mathww.citytech.cuny.edu/webwork2/WW-Dev/AddRationalExpressions2/50331648/?user=tpeckenpaugh&key=yTNvez9EZ3wp5TWXvb1Gd1t6H7gmNOZb&effectiveUser=tpeckenpaugh

OpenLab Dev url: http://openlabdev.org/webwork-playground/#/problem/local/AddRationalExpressions/diff-binomial-med.pg/question-11073

boonebgorges commented 8 years ago

Good catch! Should be fixed by 52264f9. Ready for testing.

moui72 commented 8 years ago

Yes, looks good!

I noticed a related issue, though: if I submit a reply from preview mode, the field gets cleared but stays in preview mode, so I then have to click edit to reply again. I think it should go back to edit mode. Admittedly this is not likely to matter most of the time, but I just think it looks better :eyeglasses:

let me know if you would prefer a new issue for this

submitreplyinpreview

boonebgorges commented 8 years ago

I think it should go back to edit mode.

Good catch! I hadn't thought of that. I've made the change.

moui72 commented 8 years ago

looks good now

moui72 commented 7 years ago

this issue has come back. when I asked a question from http://openlabdev.org/webwork-playground/#:problemId=local/CoordinatePlaneTrig/ratio-inequality-xyr.pg:questionId=11392 it posted and scrolled as expected, but the reply field was in 'preview' mode such that I needed to click 'edit' in order to enter text

q-r-edit

boonebgorges commented 7 years ago

Thanks for pointing this out, @moui72. This was caused by some refactoring related to editable fields. It should now be resolved.

moui72 commented 7 years ago

yep!