Make WordPress Core

Opened 20 months ago

Closed 9 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)

comment:1 @helen20 months ago

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

comment:2 @SergeyBiryukov20 months ago

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

comment:3 @jorbin20 months 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).


comment:4 @ocean9020 months ago

Version 0, edited 20 months ago by ocean90 (next)

comment:5 @helen20 months ago

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

comment:6 @pat@…20 months 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.

comment:7 @helen20 months 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.

comment:8 @pat@…20 months ago

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

comment:9 @helen20 months 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.

comment:10 @wonderboymusic9 months ago

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

comment:11 @helen9 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.