[wp-trac] [WordPress Trac] #38697: Twenty Seventeen: Video header pause button design

[WordPress Trac]

#38697: Twenty Seventeen: Video header pause button design
-------------------------------------+----------------------------
 Reporter:  melchoyce                |       Owner:  joemcgill
     Type:  enhancement              |      Status:  assigned
 Priority:  normal                   |   Milestone:  4.7
Component:  Bundled Theme            |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:  accessibility
-------------------------------------+----------------------------
Changes (by joemcgill):

 * keywords:  has-patch dev-feedback => has-patch needs-testing


Comment:

 After clarifying direction with @melchoyce
 [https://wordpress.slack.com/archives/core-themes/p1479332920000405 via
 Slack], [attachment:38697.6.diff] slightly modifies the `.wp-custom-
 header-video-button` styles based on the following:

   '''Default:'''
   Background: #222, 50% opacity
   Border: #fff, 60% opacity
   Icon: #fff, 60% opacity

   '''Hover:'''
   Background: #222, 80% opacity
   Border: #fff, 80% opacity
   Icon: #fff, 100% opacity

 I've also removed the extra styles for dark color schemes for this button,
 at Mel's request:

 > I don’t think there should be color scheme support for this button — it
 should stay dark grey and white despite your color scheme.

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