From e60535aff1941ae22f2f2c13504f790a17a34a98 Mon Sep 17 00:00:00 2001 From: Lance Willett Date: Tue, 12 Mar 2013 22:07:27 +0000 Subject: [PATCH] Twenty Thirteen: revert addition of `twentythirteen_attachment_link()` in r23523 -- as discussed in the following ticket comments, the tradeoff means simpler code instead of a minor UX enhancement. Closes #23543 and #23719. git-svn-id: https://develop.svn.wordpress.org/trunk@23677 602fd350-edb4-49c9-b593-d223f7449a82 --- .../themes/twentythirteen/functions.php | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/wp-content/themes/twentythirteen/functions.php b/wp-content/themes/twentythirteen/functions.php index cb353f727c..801866147b 100644 --- a/wp-content/themes/twentythirteen/functions.php +++ b/wp-content/themes/twentythirteen/functions.php @@ -601,25 +601,6 @@ function twentythirteen_aside_date( $content ) { } add_filter( 'the_content', 'twentythirteen_aside_date', 8 ); // After embeds, before everything else. -/** - * Usability improvement for better viewing of images on attachment pages. - * - * Moves the focus down to the main content area to by appending an in-page - * anchor to attachment link URLs. - * - * @since Twenty Thirteen 1.0 - * - * @param string $url The image attachment URL - * @return string URL with extra anchor appended. - */ -function twentythirteen_attachment_link( $url ) { - if ( wp_attachment_is_image() ) - $url .= '#main'; - - return $url; -} -add_filter( 'attachment_link', 'twentythirteen_attachment_link' ); - /** * Add postMessage support for site title and description for the Customizer. *