[wp-trac] [WordPress Trac] #23719: Twenty Thirteen: Attachment URLs for unattached images cause redirect loop

[WordPress Trac]

#23719: Twenty Thirteen: Attachment URLs for unattached images cause redirect loop
-------------------------------------+--------------------
 Reporter:  trepmal                  |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  3.6
Component:  Bundled Theme            |     Version:  trunk
 Severity:  major                    |  Resolution:
 Keywords:  needs-testing has-patch  |
-------------------------------------+--------------------
Changes (by obenland):

 * keywords:  needs-patch needs-testing => needs-testing has-patch


Comment:

 Replying to [comment:4 lancewillett]:
 > Why does the attachment have to be an image? If I upload a PDF file, I
 might still want to link to the parent, right?

 The purpose of `twentythirteen_attachment_link()` is to reposition the
 screen on load so the user does not have to scroll down to have a
 comfortable view of the image. I don't think we need to do that for other
 attachement types since the vast majority of them will be links to a file.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/23719#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software