[wp-trac] [WordPress Trac] #24024: Twenty Thirteen: Consider Header Image Sizing for HiDPI, non-HiDPI, Mobile, and IE8 Compatibility
WordPress Trac
noreply at wordpress.org
Wed Apr 10 07:46:22 UTC 2013
#24024: Twenty Thirteen: Consider Header Image Sizing for HiDPI, non-HiDPI, Mobile,
and IE8 Compatibility
------------------------------+------------------------------
Reporter: celloexpressions | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Bundled Theme | Version: trunk
Severity: normal | Resolution:
Keywords: |
------------------------------+------------------------------
Comment (by celloexpressions):
No problem! I think all of the bundled headers look fine at the large
size, so unless there's an easy fix for IE8 background size it's probably
fine as is (that is the only functional problem).
The only other related issue I see is potentially large (filesize) header
images that users upload that could waste bandwidth on mobile devices
(loading 1600px image that you see 400px of horizontally). It should be
easy to create additional versions of the cropped thumbnail that crop down
further horizontally and are used at progressive breakpoints. Well, I know
how I'd do that with a custom implementation (using
{{{WP_Image_Editor::resize()}}}), I'm not sure with the way Twenty
Thirteen does it with the core custom header features. And I may be
totally missing something here.
Also, it seems strange to only use the retina image for the default header
(assuming that a solid percentage of users will stick with it as they have
in previous default themes from my observations) but to use separate 2x
images for dotted-line.png (6 byte difference) and search-icon.png (~400
vs. 800). Remembering to change/override these if needed in child themes
is slightly annoying and easy to miss, so there is ''some'' reason not to
bother, but that's an edge case of course. When the files are that small,
does it really make much of a difference to do the non-retinas separately?
That would probably break those for IE8 though, more problematically than
for the header...
--
Ticket URL: <http://core.trac.wordpress.org/ticket/24024#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list