Make WordPress Core

Opened 2 years ago

Closed 16 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 @helen2 years ago

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

comment:2 @SergeyBiryukov2 years ago

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

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


comment:4 @ocean902 years ago

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

comment:5 @helen2 years ago

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

comment: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.

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

comment:8 @pat@…2 years ago

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

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

comment:10 @wonderboymusic16 months ago

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

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