[wp-trac] [WordPress Trac] #22058: Improve custom background properties UI
WordPress Trac
noreply at wordpress.org
Mon Sep 19 12:04:38 UTC 2016
#22058: Improve custom background properties UI
-------------------------------------+-------------------------------------
Reporter: grapplerulrich | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 4.7
Component: Customize | Version: 3.4.2
Severity: normal | Resolution:
Keywords: has-patch needs-refresh | Focuses: ui, accessibility,
has-screenshots | administration
-------------------------------------+-------------------------------------
Comment (by cdog):
@joyously: Background scroll defaults to `scroll` in core
(https://core.trac.wordpress.org/browser/trunk/src/wp-
includes/theme.php#L1694) and this patch doesn't change it. It's Twenty
Fifteen who overwrites it to `fixed`
(https://core.trac.wordpress.org/browser/trunk/src/wp-
content/themes/twentyfifteen/functions.php#L123). Maybe we should suggest
it for Twenty Seventeen not to add the same overwrite, but themes in
general should be able to define whatever defaults works best for them.
Regarding notifications, why not? Let's see where #35210 is going.
The screen icons at the bottom adjust the live preview viewport size and
are hidden on small screen sizes. They are part of the customizer being
present on all sections, not only background image. If you have some
concers about their functionality please fill in a separate ticket, I find
it broader than the current topic.
Thanks for getting involved!
--
Ticket URL: <https://core.trac.wordpress.org/ticket/22058#comment:72>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list