r91430 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r91429‎ | r91430 | r91431 >
Date:23:43, 4 July 2011
Author:kaldari
Status:reverted (Comments)
Tags:
Comment:
using proper margin heights: 14 + 68 * 2 = 150
Modified paths:
  • /trunk/phase3/tests/parser/parserTests.txt (modified) (history)

Diff [purge]

Index: trunk/phase3/tests/parser/parserTests.txt
——@@ -7465,14 +7465,14 @@
74667466 !! result
74677467 <ul class="gallery">
74687468 <li class="gallerybox" style="width: 155px"><div style="width: 155px">
7469 - <div class="thumb" style="width: 150px;"><div style="margin:66px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="galleryalt" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
 7469+ <div class="thumb" style="width: 150px;"><div style="margin:68px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="galleryalt" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
74707470 <div class="gallerytext">
74717471 <p><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image" title="desc"><img alt="inneralt" src="http://example.com/images/3/3a/Foobar.jpg" width="20" height="2" /></a>
74727472 </p>
74737473 </div>
74747474 </div></li>
74757475 <li class="gallerybox" style="width: 155px"><div style="width: 155px">
7476 - <div class="thumb" style="width: 150px;"><div style="margin:66px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="galleryalt" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
 7476+ <div class="thumb" style="width: 150px;"><div style="margin:68px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="galleryalt" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
74777477 <div class="gallerytext">
74787478 <p>This is a test template
74797479 </p>
——@@ -7509,7 +7509,7 @@
75107510 </div>
75117511 </div></li>
75127512 <li class="gallerybox" style="width: 155px"><div style="width: 155px">
7513 - <div class="thumb" style="width: 150px;"><div style="margin:66px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
 7513+ <div class="thumb" style="width: 150px;"><div style="margin:68px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
75147514 <div class="gallerytext">
75157515 <p><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" title="File:Foobar.jpg">Foobar.jpg</a><br />
75167516 some <b>caption</b> <a href="https://www.mediawiki.org/wiki/Main_Page" title="Main Page">Main Page</a>
——@@ -7517,7 +7517,7 @@
75187518 </div>
75197519 </div></li>
75207520 <li class="gallerybox" style="width: 155px"><div style="width: 155px">
7521 - <div class="thumb" style="width: 150px;"><div style="margin:66px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="Foobar.jpg" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
 7521+ <div class="thumb" style="width: 150px;"><div style="margin:68px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="Foobar.jpg" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
75227522 <div class="gallerytext">
75237523 <p><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" title="File:Foobar.jpg">Foobar.jpg</a><br />
75247524 </p>
——@@ -7549,12 +7549,12 @@
75507550 </div>
75517551 </div></li>
75527552 <li class="gallerybox" style="width: 155px"><div style="width: 155px">
7553 - <div class="thumb" style="width: 150px;"><div style="margin:66px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="Foobar.jpg" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
 7553+ <div class="thumb" style="width: 150px;"><div style="margin:68px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="Foobar.jpg" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
75547554 <div class="gallerytext">
75557555 </div>
75567556 </div></li>
75577557 <li class="gallerybox" style="width: 155px"><div style="width: 155px">
7558 - <div class="thumb" style="width: 150px;"><div style="margin:66px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="Foobar.jpg" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
 7558+ <div class="thumb" style="width: 150px;"><div style="margin:68px auto;"><a href="https://www.mediawiki.org/wiki/File:Foobar.jpg" class="image"><img alt="Foobar.jpg" src="http://example.com/images/3/3a/Foobar.jpg" width="120" height="14" /></a></div></div>
75597559 <div class="gallerytext">
75607560 </div>
75617561 </div></li>

Follow-up revisions

RevisionCommit summaryAuthorDate
r91557Revert r91426 and followups r91427, r91430: Breaks Gallery-related parser testsdemon16:23, 6 July 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r91426fixing rounding problem, per comment at r82309kaldari21:05, 4 July 2011
r91427better fix for bug 27338 - doesnt rely on line-heightkaldari21:30, 4 July 2011

Comments

#Comment by Kaldari (talk | contribs)   23:47, 4 July 2011

The rev that corrected this value on the MediaWiki side is r91426.

#Comment by Kaldari (talk | contribs)   23:51, 4 July 2011

... plus r91427. Basically, it no longer tries to do a guess at the line-height and instead now uses the actual height of the thumbnail.

Status & tagging log