Part of the JavaScript selectors clean-up effort.
#1574470: Selectors clean-up
#1415788: Javascript winter clean-up

CommentFileSizeAuthor
#4 1751382-clean-up-file-js.patch2.85 KBdroplet
#3 core-js-1751382-3-file-selectors-clean-up.patch4.31 KBjelle_s

Comments

nod_’s picture

nod_
Primary language French
Location Lille
commented

use .on and .off
can probably be clean-up some more as well

nod_’s picture

nod_
Primary language French
Location Lille
commented
Issue tags: +Needs JavaScript testing

tag

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented
Status: Active » Needs review
StatusFileSize
new core-js-1751382-3-file-selectors-clean-up.patch4.31 KB

This issue has been around long enough... Seems like a pretty straight-forward task at first sight, so here we go ;-)

droplet’s picture

droplet commented
Issue summary: View changes
StatusFileSize
new 1751382-clean-up-file-js.patch2.85 KB

Re-reroll patches & Clean up a bit more. No interdiff will be attached, please review whole patch. Thanks.

nod_’s picture

nod_
Primary language French
Location Lille
commented
Status: Needs review » Reviewed & tested by the community

That works, thanks!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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