This repository has been archived by the owner on May 10, 2022. It is now read-only.
Show exercise content on feedback page #340
Assignees
Comments
navilan
added a commit
to navilan/ost
that referenced
this issue
May 19, 2014
- Add exercise content (question only) to the feedback page as a
collapsible section.
- Remove trailing whitespaces.
- Remote trailing whitespaces in `show.html.erb` as well.
Notes:
* Free text/drawing provided by the student is not shown.
* The other options in multiple selection are not shown.
* The sidebar nav has misleading text ("Show" instead of "Go to")
* Collapsible sections could use simple CSS instead of inline JS.
* By introducing a section heading the header hierarchy is disrupted.
navilan
added a commit
to navilan/ost
that referenced
this issue
May 20, 2014
- Reuse exercise blurb from assignment_exercise. - Remove redundant `if true` statements. - Add boolean flags for showing correct answer and choices. - Use defined check to provide defaults for boolean values.
navilan
added a commit
to navilan/ost
that referenced
this issue
May 20, 2014
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Sometime after a student works an exercise on an assignment, feedback (the answer, high-level solution, and detailed solution) may be released to him/her.
The feedback page is a separate page from where the student read and answered the exercise. It unfortunately doesn't show the content of the exercise, so if the student wants to go back and forth between the exercise and the solution they have to go back and forth in the browser (which is not cool).
This task is to display the exercise on the feedback page. This should be fairly easy to do as there are already partials for displaying it on other pages (see app/views/student_exercises/show.html.erb)
There is an open question as to where the exercise should be displayed. If we display it at the top of the page, we might want to display it collapsed so the user can immediately see the blurb that says if they got it right or wrong. UX people will later tell us exactly where to put the content, but for now, just pick a place that seems reasonable. If we need to move it, we should just have to move one line of code give or take.
The text was updated successfully, but these errors were encountered: