Webform Dialog Modal triggered with a link doesn't work on touch device

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

Sorry, but I have to disagree this was a good idea. Touch events are special, yes, but at least it should at least have been touchend instead of touchstart.
Using touchend ensures the event fires when the user lifts their finger, reducing accidental triggers.

Still, I think it should not be needed at all in modern browsers, according to this answer: https://stackoverflow.com/a/11398089/10031165

Should we reopen this or create a follow-up?

Which browser and device did you use @roromedia?

Maybe @roromedia could provide the affected browser? If I remind corretly, this where a Safari bug, not sure if click is still problematic in iOS Safari.

The bug occurred on iOS (when tested I used the latest stable version and versions below as well) iPads with Safari as standard browser.

Thanks @roromedia.

So I changed the task title accordingly.

Just came across this problematic behaviour again on Android. Extremely annoying when simple scrolling triggers the webform dialog.

We should try if a simple cursor: pointer; solves the problem with the click event on Safari, see:
https://stackoverflow.com/questions/24077725/mobile-safari-sometimes-doe...

I've tested with 'pointerup' instead 'click touchstart' and it seems to work well on all devices and browsers.

But someone should try this on a real Iphone, I only have the Simulator.

So should we re-open this issue or create a new one?

@jrockowitz could you reopen this one to fix it? Or would you prefer a follow-up?

@thomas.frobieter On an Iphone SE2 18.3.2, Safari:

• 'pointerup' does not trigger the modal but opens the link
• 'touchend click' does open the modal when the finger is lifted, which does not really improve the situation much

I'll attach a patch that reverts this issues patch, because we rather use a link for some safari users, instead of interfering with everybody's scrolling.

@thomas.frobieter On an Iphone SE2 18.3.2, Safari:

'pointerup' does not trigger the modal but opens the link
'touchend click' does open the modal when the finger is lifted, which does not really improve the situation much

That's strange. Safari 13.1/13.2 should support pointerup. I think we should debug this, as the pointer event sounds like the cleanest solution to me.

Can someone with a Apple touch device please test the 'pointerup click' button? => https://codepen.io/thomas-frobieter/full/qEdNoEr

I have tried on the iOS Simulator and it seems to work well.

If the real device test also works well, I will create a follow up issue.

This has caused 📌 When scrolling, the modal window opens automatically Active . Please review the fix in that issue.

@liam morland: There's nothing to review yet. We should either fix both issues over at 📌 When scrolling, the modal window opens automatically Active or here. They are very closely related, nearly the same and surely have the same fix. As this one is closed, maybe better move over?

So ==> Further work here: https://www.drupal.org/project/webform/issues/3506020 📌 When scrolling, the modal window opens automatically Active <==

Yes, please work in the other issue. There isn't a patch, but there is a prose description of the change in comment 4.