Page MenuHomePhabricator

The cancel event handler closes the ULS even if a descendant of the trigger is clicked
Closed, ResolvedPublic

Description

In the cancel event handler, the ULS window will be closed if any element other than the ULS trigger is clicked, even if that element is a descendant of the ULS trigger. This is a problem when the ULS trigger is an OOjs UI button.

Event Timeline

Victorbarbu claimed this task.
Victorbarbu raised the priority of this task from to Medium.
Victorbarbu updated the task description. (Show Details)
Victorbarbu added subscribers: Victorbarbu, TTO.

Change 261810 had a related patch set uploaded (by Victorbarbu):
Fix the cancel event function to prevent erroneous close

https://gerrit.wikimedia.org/r/261810

Change 261810 merged by jenkins-bot:
Fix the cancel event function to prevent erroneous close

https://gerrit.wikimedia.org/r/261810

Nemo_bis subscribed.

Ahem @TTO, thanks for looking into this but this should not have been merged in gerrit first. This is an upstream library, the patch needs to be in https://github.com/wikimedia/jquery.uls too.

Whoops, didn't even think to check that :(

Thanks Santhosh for the merge, I assume this is fixed.