[wp-trac] [WordPress Trac] #40843: Twenty Seventeen: broken MediaElement.js player icons on hover

WordPress Trac noreply at wordpress.org
Mon Sep 18 08:36:03 UTC 2017


#40843: Twenty Seventeen: broken MediaElement.js player icons on hover
---------------------------------------+-----------------------------
 Reporter:  celloexpressions           |       Owner:
     Type:  defect (bug)               |      Status:  reopened
 Priority:  normal                     |   Milestone:  Future Release
Component:  Bundled Theme              |     Version:  4.7
 Severity:  normal                     |  Resolution:
 Keywords:  has-screenshots has-patch  |     Focuses:  ui
---------------------------------------+-----------------------------
Changes (by xkon):

 * keywords:  needs-patch has-screenshots => has-screenshots has-patch


Comment:

 '''40843.diff''' skips the `me-js` buttons from applying the custom
 background colors.

 Further information: As the button background is .svg based, by adding a
 custom color through an extra .css or Customizr was resulting on
 overwriting the .svg element. That being said there's no way to target the
 `me-js` specific .svg as installations are different on every WordPress.
 This patch uses a simple trick with `:not()` to 'skip' the `me-js` buttons
 while selecting new colors.

 It works fine on my tests but further tests on different spec pcs are
 always welcome.

 Concerning browser support on `not()`:
 https://www.w3schools.com/cssref/sel_not.asp

--
Ticket URL: <https://core.trac.wordpress.org/ticket/40843#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list