[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