Last modified: 2010-05-15 15:33:21 UTC
gallery syntax is parsed in a wrong way, the problem exists for every gallery on
commons at the moment; compare these validation results http://validator.w3.org/
check?uri=http%3A%2F%2Fcommons.wikimedia.org%2Fwiki%2FAcropolis for example
Added blocking on XHTML tracking bug 209
Created attachment 183 [details]
Patches ImageGallery.php for HEAD
Created attachment 184 [details]
Patches ImageGallery.php for REL1_4
Created attachment 185 [details]
Proposed fix to ImageGallery.php
Image galleries are now valid xhtml, and look a bit more like the current image
boxes. Positioning is done with calculated padding to achieve vertical
alignment as XHTML/CSS can't do vertical alignment (beyond aligning inline
images against their text, it really is impossible). The other fixes still use
the height property of TD , which is only accepted in XHTML 1.0 Transistional,
Sorry, but I don't know how to do diffs :(.
commons] there is already a short discussion about this. First the attribute
valign="center" does not do anything at all (you could write foo="FU" instead)!
But the attribute "height" has an effect all major brwosers. This is a disaster
and the reason why I reset the priority. --~~~~
<table height="XXpx"> is nonsence. What the browsers like is <table height="XX">
(without unit!!!). And what both, browsers AND HTML Spec like, is <table
Fixed in CVS HEAD, based on patch provided by Ed Sanders.
ImageGallery.php, revision 1.15, and lots of CSS files.
Fixed in REL1_4