[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