From: Leo Koppelkamm Date: Sat, 23 Apr 2011 07:27:45 +0000 (+0000) Subject: Add ability to use add alt texts for images in galleries ( Bug 18682 ). Patch by... X-Git-Tag: 1.31.0-rc.0~30619 X-Git-Url: http://git.cyclocoop.org/data/Fool?a=commitdiff_plain;h=f3e6866d1abea4536ca8f42923fb81d7f2c38511;p=lhc%2Fweb%2Fwiklou.git Add ability to use add alt texts for images in galleries ( Bug 18682 ). Patch by Jan Paul Posma. Also cleaned up some comments and var names --- diff --git a/includes/ImageGallery.php b/includes/ImageGallery.php index f490a381ee..9a372020a4 100644 --- a/includes/ImageGallery.php +++ b/includes/ImageGallery.php @@ -136,14 +136,15 @@ class ImageGallery * Add an image to the gallery. * * @param $title Title object of the image that is added to the gallery - * @param $html String: additional HTML text to be shown. The name and size of the image are always shown. + * @param $html String: Additional HTML text to be shown. The name and size of the image are always shown. + * @param $alt String: Alt text for the image */ - function add( $title, $html='' ) { + function add( $title, $html = '', $alt = '' ) { if ( $title instanceof File ) { // Old calling convention $title = $title->getTitle(); } - $this->mImages[] = array( $title, $html ); + $this->mImages[] = array( $title, $html, $alt ); wfDebug( "ImageGallery::add " . $title->getText() . "\n" ); } @@ -151,14 +152,15 @@ class ImageGallery * Add an image at the beginning of the gallery. * * @param $title Title object of the image that is added to the gallery - * @param $html String: Additional HTML text to be shown. The name and size of the image are always shown. + * @param $html String: Additional HTML text to be shown. The name and size of the image are always shown. + * @param $alt String: Alt text for the image */ - function insert( $title, $html='' ) { + function insert( $title, $html='', $alt='' ) { if ( $title instanceof File ) { // Old calling convention $title = $title->getTitle(); } - array_unshift( $this->mImages, array( &$title, $html ) ); + array_unshift( $this->mImages, array( &$title, $html, $alt ) ); } @@ -218,15 +220,16 @@ class ImageGallery if ( $this->mPerRow > 0 ) { $maxwidth = $this->mPerRow * ( $this->mWidths + self::THUMB_PADDING + self::GB_PADDING + self::GB_BORDERS ); $oldStyle = isset( $this->mAttribs['style'] ) ? $this->mAttribs['style'] : ""; + # _width is ignored by any sane browser. IE6 doesn't know max-width so it uses _width instead $this->mAttribs['style'] = "max-width: {$maxwidth}px;_width: {$maxwidth}px;" . $oldStyle; } $attribs = Sanitizer::mergeAttributes( array( 'class' => 'gallery' ), $this->mAttribs ); - $s = Xml::openElement( 'ul', $attribs ); + $output = Xml::openElement( 'ul', $attribs ); if ( $this->mCaption ) { - $s .= "\n\t
  • {$this->mCaption}
    • "; + $output .= "\n\t
  • {$this->mCaption}
    • "; } $params = array( 'width' => $this->mWidths, 'height' => $this->mHeights ); @@ -234,6 +237,7 @@ class ImageGallery foreach ( $this->mImages as $pair ) { $nt = $pair[0]; $text = $pair[1]; # "text" means "caption" here + $alt = $pair[2]; $descQuery = false; if ( $nt->getNamespace() == NS_FILE ) { @@ -272,18 +276,19 @@ class ImageGallery $thumbhtml = "\n\t\t\t".'
    ' . htmlspecialchars( $img->getLastError() ) . '
    '; } else { - //We get layout problems with the margin, if the image is smaller - //than the line-height, so we less margin in these cases. + # We get layout problems with the margin, if the image is smaller + # than the line-height (17), so we add less margin in these cases. $minThumbHeight = $thumb->height > 17 ? $thumb->height : 17; $vpad = floor(( self::THUMB_PADDING + $this->mHeights - $minThumbHeight ) /2); $imageParameters = array( 'desc-link' => true, - 'desc-query' => $descQuery + 'desc-query' => $descQuery, + 'alt' => $alt, ); - # In the absence of a caption, fall back on providing screen readers with the filename as alt text - if ( $text == '' ) { + # In the absence of both alt text and caption, fall back on providing screen readers with the filename as alt text + if ( $alt == '' && $text == '' ) { $imageParameters['alt'] = $nt->getText(); } @@ -308,14 +313,14 @@ class ImageGallery if( $this->mShowBytes ) { if( $img ) { - $nb = wfMsgExt( 'nbytes', array( 'parsemag', 'escape'), + $fileSize = wfMsgExt( 'nbytes', array( 'parsemag', 'escape'), $wgLang->formatNum( $img->getSize() ) ); } else { - $nb = wfMsgHtml( 'filemissing' ); + $fileSize = wfMsgHtml( 'filemissing' ); } - $nb = "$nb
    \n"; + $fileSize = "$fileSize
    \n"; } else { - $nb = ''; + $fileSize = ''; } $textlink = $this->mShowFilename ? @@ -332,20 +337,20 @@ class ImageGallery # in version 4.8.6 generated crackpot html in its absence, see: # http://bugzilla.wikimedia.org/show_bug.cgi?id=1765 -Ævar - # Weird double wrapping in div needed due to FF2 bug + # Weird double wrapping (the extra div inside the li) needed due to FF2 bug # Can be safely removed if FF2 falls completely out of existance - $s .= + $output .= "\n\t\t" . '
  • ' . '
    ' . $thumbhtml . "\n\t\t\t" . '
    ' . "\n" - . $textlink . $text . $nb + . $textlink . $text . $fileSize . "\n\t\t\t
    " . "\n\t\t
    • "; } - $s .= "\n"; + $output .= "\n"; - return $s; + return $output; } /** diff --git a/includes/parser/Parser.php b/includes/parser/Parser.php index ad4ba77b9e..3dcfe4292c 100644 --- a/includes/parser/Parser.php +++ b/includes/parser/Parser.php @@ -4733,21 +4733,37 @@ class Parser { if ( strpos( $matches[0], '%' ) !== false ) { $matches[1] = rawurldecode( $matches[1] ); } - $tp = Title::newFromText( $matches[1], NS_FILE ); - $nt =& $tp; - if ( is_null( $nt ) ) { + $title = Title::newFromText( $matches[1], NS_FILE ); + if ( is_null( $title ) ) { # Bogus title. Ignore these so we don't bomb out later. continue; } + + $label = ''; + $alt = ''; if ( isset( $matches[3] ) ) { - $label = $matches[3]; - } else { - $label = ''; + // look for an |alt= definition while trying not to break existing + // captions with multiple pipes (|) in it, until a more sensible grammar + // is defined for images in galleries + + $altmatches = StringUtils::explode('|', $matches[3]); + + foreach ( $altmatches as $altmatch ) { + if ( substr( $altmatch, 0, 4 ) === 'alt=' ) { + $alt = $this->stripAltText( trim( substr( $altmatch, 4 ) ), false ); + } + else { + // concatenate all other pipes + $label .= '|' . $altmatch; + } + } + // remove the first pipe + $label = substr( $label, 1 ); } $html = $this->recursiveTagParse( trim( $label ) ); - $ig->add( $nt, $html ); + $ig->add( $title, $html, $alt ); } return $ig->toHTML(); } diff --git a/tests/parser/parserTests.txt b/tests/parser/parserTests.txt index 27e90b63f0..97a65f48e4 100644 --- a/tests/parser/parserTests.txt +++ b/tests/parser/parserTests.txt @@ -7312,6 +7312,7 @@ File:Nonexistant.jpg|caption File:Nonexistant.jpg image:foobar.jpg|some '''caption''' [[Main Page]] image:foobar.jpg +image:foobar.jpg|Blabla|alt= This is a foo-bar.|blabla. !! result !! end