[wp-trac] [WordPress Trac] #21389: Retina theme custom headers

WordPress Trac noreply at wordpress.org
Thu Nov 5 21:00:09 UTC 2015


#21389: Retina theme custom headers
-------------------------------------------------+-------------------------
 Reporter:  nacin                                |       Owner:  joemcgill
     Type:  task (blessed)                       |      Status:  assigned
 Priority:  normal                               |   Milestone:  4.4
Component:  Customize                            |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  responsive-images has-patch 2nd-     |     Focuses:
  opinion                                        |
-------------------------------------------------+-------------------------

Comment (by azaozz):

 Replying to [comment:27 Otto42]:
 > The proposal I made 3 years ago was along the same lines...

 Right, sorry, should have linked to [#comment:16 comment 16] in my
 previous comment.

 Replying to [comment:28 joemcgill]:

 We get pretty much everything we need from `get_custom_header()`. Not sure
 if we have to go back and run most of the attachment image functions again
 (it's slow).

 How about we get the srcset and sizes attributes right in
 `get_header_image_tag()`, see
 [https://core.trac.wordpress.org/attachment/ticket/21389/21389.1.diff
 21389.1.diff]. There is some code duplication but is faster and nearly
 identical to the code most themes use now.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/21389#comment:29>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list