Last modified: 2010-05-15 15:33:21 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T3124, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 1124 - <gallery> syntax is not valid xhtml 1.0
<gallery> syntax is not valid xhtml 1.0
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch
Depends on:
Blocks: html
  Show dependency treegraph
Reported: 2004-12-18 02:40 UTC by bdk
Modified: 2010-05-15 15:33 UTC (History)
1 user (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Patches ImageGallery.php for HEAD (1.66 KB, patch)
2004-12-27 16:59 UTC, Zigger
Patches ImageGallery.php for REL1_4 (1.67 KB, patch)
2004-12-27 17:00 UTC, Zigger
Proposed fix to ImageGallery.php (3.65 KB, text/plain)
2004-12-28 06:44 UTC, ejsanders

Description bdk 2004-12-18 02:40:43 UTC
gallery syntax is parsed in a wrong way, the problem exists for every gallery on 
commons at the moment; compare these validation results
check? for example
Comment 1 Brion Vibber 2004-12-18 02:41:24 UTC
Added blocking on XHTML tracking bug 209
Comment 2 Zigger 2004-12-27 16:59:23 UTC
Created attachment 183 [details]
Patches ImageGallery.php for HEAD
Comment 3 Zigger 2004-12-27 17:00:00 UTC
Created attachment 184 [details]
Patches ImageGallery.php for REL1_4
Comment 4 ejsanders 2004-12-28 06:44:39 UTC
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,
not Strict.

Sorry, but I don't know how to do diffs :(.
Comment 5 Patrick-Emil Zörner 2005-01-04 15:22:07 UTC
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. --~~~~
Comment 6 Kai F. Lahmann 2005-01-04 15:47:05 UTC
<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
Comment 7 JeLuF 2005-01-05 22:14:20 UTC
Fixed in CVS HEAD, based on patch provided by Ed Sanders.

ImageGallery.php, revision 1.15, and lots of CSS files. 
Comment 8 JeLuF 2005-01-06 23:03:15 UTC
Fixed in REL1_4

Note You need to log in before you can comment on or make changes to this bug.