Make WordPress Core

Opened 2 years ago

Closed 17 months ago

#25097 closed defect (bug) (worksforme)

Webkit and Moz transition events un-neccessary

Reported by: pat@… Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.6
Component: Administration Keywords: needs-patch
Focuses: ui Cc:


In recent versions of Chrome and Firefox, the vendor prefixes for transitions, animations, and transforms have been removed. This is fine for CSS, but in javascript the vendor-prefixed events fire in addition to the non-prefixed events. This results in transitionend events firing twice in Chrome/Firefox. I'm not sure where this would have an impact in the WP back-end, but I was looking in media-views.js and noticed where the prefixed events are used and thought to myself that it would likely cause an issue somewhere.

Change History (11)

#1 @helen
2 years ago

What versions of those browsers, and what might we need to continue to support for, say, Kindle Fire/Silk?

#2 @SergeyBiryukov
2 years ago

  • Component changed from General to Administration
  • Keywords ui-focus added

#3 @jorbin
2 years ago

the webkit versions need to stay for safari.

the moz versions could possibly go. Support for the prefix free version was added in ff16 (24 is current).


#4 @ocean90
2 years ago

Version 0, edited 2 years ago by ocean90 (next)

#5 @helen
2 years ago

If nothing uses -moz- anymore, seems like we could probably drop that. Not -webkit-, though.

#6 @pat@…
2 years ago

Hm, seems we might need to find a solution then to the multiple events firing in Chrome. If the webkit prefix needs to stay, there should be a detect of some sort to ensure that listeners aren't responding to cloned/prefixed events.

#7 @helen
2 years ago

I guess as far as the JS (jQuery-specific?) part goes, I'm not convinced that's a WP bug. Seems like that would affect a lot of sites and apps that support a broader level of browser compat.

#8 @pat@…
2 years ago

I agree, although WP may very well be affected by this behavior, perhaps a patch?

#9 @helen
2 years ago

Not sure what we should patch in core - is there an actual issue here? I don't think we do (or would do) JS on a transition event.

I'm leaving this open for now because we should probably re-evaluate the -moz- prefix usage.

#10 @wonderboymusic
17 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release

#11 @helen
17 months ago

  • Milestone Future Release deleted
  • Resolution set to worksforme
  • Status changed from new to closed

Actually, I'm closing. I don't know what needs to actually be done here - reopen if we find out.

Note: See TracTickets for help on using tickets.