Backwards navigation does not seem to be supported but the checkmark is there.

Comments

seanbfuller’s picture

/bump

Still an issue after http://drupal.org/node/85983

webchick’s picture

For now, let's rm the checkbox, and then change this to a feature instead.

nicholasthompson’s picture

Nah - its a piece of piss to do.

When you shift the question out of the array when you move to the next one, instead of overwriting the current question by overwriting it, push it into a "previous questions" array.

When you hit back, push the current quesion onto the "Next Question" array and pop the previous question array into the current question.

Its all about use of Stacks :-)

seanbfuller’s picture

Was there also an issue with backwards navigation and feedback after each question?

Usually you would not want this kind of setup, but I can also see where you might. Imagine a kids site where you want the child to "keep trying" until he/she gets the question right.

If we do implement this, we would need a "try again" or "go back and try again" link, yes?

nicholasthompson’s picture

If it were me - I'd implement it, but if enabled you dont get feedback after each question. Maybe you could have an "I'm done now" type button near the top or something (once you submit the last question, maybe go back to the first?) That way you can go through and check your answers as you would in a real sit-down exam (like degree, etc).

I agree - you certainly dont want backtracking and feedback enabled at the same time!

awong’s picture

The backwards navigation checkbox had been already removed in the HEAD version. Is this bug still an issue?

awong’s picture

Status: Active » Needs review
StatusFileSize
new1.61 KB
new5.44 KB

I just create a patch that make the backwards navigation working.
I added the backwards navigation checkbox back and make the functionality work. Please review the code. Thanks

westwesterson’s picture

Status: Needs review » Needs work
StatusFileSize
new7.17 KB

I keep getting this error when i 'try again' too many times:
An illegal choice has been detected. Please contact the site administrator.
Also, i think that the placement of try again is bad, should be the other way around.

I also changed the wording from 'try again' to 'last question' i'm thinking about changing it to 'back' instead, but try again for me implies the current question, and because of the placement.

I have played around with this a bit, and i think its a step in the right direction, but this issue is far from fixed.

westwesterson’s picture

StatusFileSize
new7.17 KB

fixed code standards issues, otherwise this patch is the same.

awong’s picture

Status: Needs work » Needs review
StatusFileSize
new7.08 KB

Changed the wording of the button to "Back".
fixed the "An illegal choice has been detected. Please contact the site administrator." error.
Please let me know if there have anything else is not working right.

westwesterson’s picture

Status: Needs review » Fixed

Much cleaner interface, works as expected, I'm going to green light this one. Congrats.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

  • Commit e643550 on 5.x-2.x, 6.x-2.x, 6.x-3.x, 6.x-4.x, 6.x-5.x, 6.x-6.x, 7.x-4.x, master, 7.x-5.x by westwesterson:
    Backwards navigation is now supported Issue #84863 by awong
    
    

  • Commit e643550 on 5.x-2.x, 6.x-2.x, 6.x-3.x, 6.x-4.x, 6.x-5.x, 6.x-6.x, 7.x-4.x, master, 7.x-5.x, quiz-pages by westwesterson:
    Backwards navigation is now supported Issue #84863 by awong
    
    

  • Commit e643550 on 5.x-2.x, 6.x-2.x, 6.x-3.x, 6.x-4.x, 6.x-5.x, 6.x-6.x, 7.x-4.x, master, 7.x-5.x, quiz-pages, 2269219 by westwesterson:
    Backwards navigation is now supported Issue #84863 by awong
    
    

  • Commit e643550 on 5.x-2.x, 6.x-2.x, 6.x-3.x, 6.x-4.x, 6.x-5.x, 6.x-6.x, 7.x-4.x, master, 7.x-5.x, 2269219 by westwesterson:
    Backwards navigation is now supported Issue #84863 by awong