[wp-trac] [WordPress Trac] #39349: Customizer (mobile preview) site title extra padding
WordPress Trac
noreply at wordpress.org
Tue Dec 20 22:50:54 UTC 2016
#39349: Customizer (mobile preview) site title extra padding
--------------------------+--------------------------
Reporter: halftones | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.7.1
Component: Customize | Version: 4.7
Severity: normal | Resolution:
Keywords: needs-patch | Focuses: performance
--------------------------+--------------------------
Comment (by sirbrillig):
@westonruter Oh, I didn't realize there was body padding being added.
Yeah, the shortcuts definitely shouldn't affect the page layout if at all
possible, and certainly not if they're not even present. (At the very
least I think that sort of global adjustment styling should be up to the
theme.)
Adding a class name to partial containers is a clever solution that sounds
like it would prevent this padding from being applied without a partial.
I wonder, however, if the padding is needed at all. A quick test of a few
default themes without that padding looks ok to my eye (although we'd
definitely have to do a more careful analysis to see what that was meant
to fix).
--
Ticket URL: <https://core.trac.wordpress.org/ticket/39349#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list