Problem/Motivation

Just prefer closing modals with the escape key.

Proposed resolution

Capture the keydown event and close the modal.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#8 close_overlay_with-2484439-8.patch875 bytesjoelpittet
#8 interdiff.txt827 bytesjoelpittet
#6 close_overlay_with-2484439-6.patch832 bytesjoelpittet
#1 close_overlay_with-2484439-1.patch814 bytesjoelpittet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
FileSize
close_overlay_with-2484439-1.patch814 bytes

Here's a patch with that working.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Active » Needs review
subhojit777’s picture

subhojit777
he/him
Primary language Bengali
CreditAttribution: subhojit777 commented

Thanks for the patch :) I was also thinking about the same.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented

Does it work for you?

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
CreditAttribution: subhojit777 as a volunteer and at Srijan | A Material+ Company commented

I have not tested the code yet. Will do it soon. Thanks!

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
FileSize
close_overlay_with-2484439-6.patch832 bytes

re-roll with latest dev changes.

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
CreditAttribution: subhojit777 as a volunteer and at Srijan | A Material+ Company commented

Thank you :) Just a small change.

+++ b/js/dc_ajax_add_cart.js
@@ -1,9 +1,16 @@
+      function closeAddToCartOverlay() {

Can you please add a function description here.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
FileSize
interdiff.txt827 bytes
close_overlay_with-2484439-8.patch875 bytes

Feel free to write what you want for the comment.

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
CreditAttribution: subhojit777 as a volunteer and at Srijan | A Material+ Company commented
Status: Needs review » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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