[wp-trac] [WordPress Trac] #31195: Add a user-friendly way to preview site responsiveness in the Customizer

WordPress Trac noreply at wordpress.org
Mon Feb 15 18:06:47 UTC 2016


#31195: Add a user-friendly way to preview site responsiveness in the Customizer
-------------------------------------+-------------------------------------
 Reporter:  celloexpressions         |       Owner:  westonruter
     Type:  feature request          |      Status:  accepted
 Priority:  normal                   |   Milestone:  4.5
Component:  Customize                |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-screenshots has-     |     Focuses:  ui, accessibility,
  patch commit                       |  administration
-------------------------------------+-------------------------------------

Comment (by celloexpressions):

 Is there a particular reason to make the hover and focus styles different?
 I'd be hesitant to make further changes to those at this point after the
 extensive back and forth that we already went through to agree on the
 current styles. That being said, I'd prefer to have the border for both
 and drop the background change from the focus styles. I'd also say that a
 higher contrast change would call for a longer transition, if anything,
 since no transition at all is more jarring.

 I also just noticed - should we be running `api.trigger` or
 `api.preview.send` instead of only `api.set` when the device value is
 changed? Not sure whether themes can hook into the value change binding
 from the preview.

 Replying to [comment:97 ocean90]:
 > [attachment:31195.10.diff] and [attachment:31195.9.diff] are looking
 good. I'm only curious why the word "also" was removed in
 [attachment:31195.2.diff]?
 I was looking at something else, must've forgotten to remove it from the
 patch.

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


More information about the wp-trac mailing list