With the fix made in issue #2913831: Confirmation alert we now have a working confirmation box again. However if you have enabled backwards navigation you'll also receive the warning when pressing the back button, which considering the nature of the warning leads to confusion.

CommentFileSizeAuthor
#4 quiz-no_confirmation_on_back_button-2953548-4.patch742 bytesDa_Cloud
#2 quiz-no_confirmation_on_back_button-2953548-2.patch722 bytesDa_Cloud
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Da_Cloud created an issue. See original summary.

Da_Cloud’s picture

Da_Cloud CreditAttribution: Da_Cloud at INDICIA commented
Status: Active » Needs review
FileSize
quiz-no_confirmation_on_back_button-2953548-2.patch722 bytes

My JavaScript skills aren't that great, but I've created a working fix in the patch below.

Status: Needs review » Needs work

The last submitted patch, 2: quiz-no_confirmation_on_back_button-2953548-2.patch, failed testing. View results

Da_Cloud’s picture

Da_Cloud CreditAttribution: Da_Cloud at INDICIA commented
FileSize
quiz-no_confirmation_on_back_button-2953548-4.patch742 bytes

Made the click function more specific so that it only triggers on a form-action button press.

Da_Cloud’s picture

Da_Cloud CreditAttribution: Da_Cloud at INDICIA commented
Status: Needs work » Needs review

  • djdevin committed 003bb3b on 7.x-5.x authored by Da_Cloud 
    Issue #2953548 by Da_Cloud: Don't show the confimation alert message on...

  • djdevin committed f9f0de3 on 7.x-5.x 
    Issue #2953548: target skip button too
djdevin’s picture

djdevin
Location Philadelphia
CreditAttribution: djdevin at Cadmium commented
Status: Needs review » Fixed

Thanks, fixed.

I made a change to target it inclusively rather than exclusively and used the jQuery event instead of checking the ID.

Status: Fixed » Closed (fixed)

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