?
avatar PhilETaylor
PhilETaylor
6 May 2021

Steps to reproduce the issue

One quick one for @dgrammatiko ;-)

Expected result

Open any modal in Joomla 4 admin - click ESC keyboard key - see it closes (Expected in 2021 that any modal closes on pressing of esc key)

Actual result

Now, Open any modal in Joomla 4 admin - click with the mouse anywhere INSIDE the modal - click ESC keyboard key - see it now doesn't close.

System information (as much as possible)

safari on mac. No console errors.

Additional comments

avatar PhilETaylor PhilETaylor - open - 6 May 2021
avatar joomla-cms-bot joomla-cms-bot - change - 6 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 May 2021
avatar brianteeman
brianteeman - comment - 6 May 2021

If I remember correctly the WCAG standard says that if it is a dialog then yes but if its a modal or alertdialog then no.

For the diff between a modal and a dialog this is as good as any https://www.nngroup.com/articles/modal-nonmodal-dialog/

avatar PhilETaylor
PhilETaylor - comment - 6 May 2021

LOL That article accurately describes all the reason Joomla should not be using modals for the majority of tasks that Joomla uses modals for...

avatar brianteeman
brianteeman - comment - 6 May 2021

I must have read something completely different then

avatar dgrammatiko
dgrammatiko - comment - 7 May 2021

Well, this is a problem with the dependency script, Bootstrap modal. A simple test using the batch modal and following the testing guide indicates that the script is fine with the modal that their body is not an iframe. Repeating this with let's say user selection or image selection the problem is obvious and the only difference with the previous step is that in this case, the modal content is an iframe. Probably worth opening an issue in their repo instead of trying to monkey patch things here...

avatar PhilETaylor PhilETaylor - change - 8 May 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-05-08 14:45:05
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 8 May 2021
avatar PhilETaylor
PhilETaylor - comment - 8 May 2021

Thanks for the explanation.

Add a Comment

Login with GitHub to post a comment