Last modified: 2012-02-09 23:03:10 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 T16727, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 14727 - Upgrade to Ploticus 2.41 to fix non-conformant XHTML in EasyTimeline
Upgrade to Ploticus 2.41 to fix non-conformant XHTML in EasyTimeline
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
EasyTimeline (Other open bugs)
unspecified
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
http://validator.w3.org/check?verbose...
: patch, patch-need-review
Depends on:
Blocks: html
  Show dependency treegraph
 
Reported: 2008-07-04 10:55 UTC by Max Semenik
Modified: 2012-02-09 23:03 UTC (History)
5 users (show)

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


Attachments
Quick fix for the PHP part. (1.14 KB, patch)
2008-07-04 11:14 UTC, Platonides
Details
Quick fix for the PHP part (now with the title escaped) (1.13 KB, patch)
2008-07-04 11:17 UTC, Platonides
Details
Patch against ploticus (677 bytes, text/x-diff)
2008-07-05 15:08 UTC, Platonides
Details
Patch against ploticus (677 bytes, patch)
2008-07-05 15:08 UTC, Platonides
Details
Quick fix for the PHP part. (1.15 KB, patch)
2008-07-05 21:22 UTC, Platonides
Details

Description Max Semenik 2008-07-04 10:55:30 UTC
W3C validator complains about the lack of id attribute for <map> and alt attribute for <area> (see URL above).
Comment 1 Platonides 2008-07-04 11:14:32 UTC
Created attachment 5046 [details]
Quick fix for the PHP part.

Also complains about the img without alt tag.
Comment 2 Platonides 2008-07-04 11:17:01 UTC
Created attachment 5047 [details]
Quick fix for the PHP part (now with the title escaped)
Comment 3 Platonides 2008-07-05 15:08:39 UTC
Created attachment 5050 [details]
Patch against ploticus

This patch fixes ploticus mapfile output to be conformant with XHTML.
Sent to http://tech.groups.yahoo.com/group/ploticus/ as well.
Comment 4 Platonides 2008-07-05 15:08:43 UTC
Created attachment 5051 [details]
Patch against ploticus

This patch fixes ploticus mapfile output to be conformant with XHTML.
Sent to http://tech.groups.yahoo.com/group/ploticus/ as well.
Comment 5 Platonides 2008-07-05 21:22:30 UTC
Created attachment 5052 [details]
Quick fix for the PHP part.

If the hash begin by a number, it'd form an invalid id.
Obsoleting the duplicate patch.
Comment 6 Chad H. 2008-07-16 03:55:26 UTC
Patch implemented as part of r37735.
Comment 7 Platonides 2008-07-16 13:52:24 UTC
Good. Now we just need a custom build of ploticus or having it patched upstream and upgrade.
No answers yet for http://tech.groups.yahoo.com/group/ploticus/message/2091 though :(
Comment 8 Siebrand Mazeland 2008-08-13 11:49:26 UTC
Assigned to extension developer.
Comment 9 Siebrand Mazeland 2008-08-13 11:51:39 UTC
+patch
Comment 10 Chad H. 2008-08-13 12:10:53 UTC
Note: my patch was reverted in r37842
Comment 11 Platonides 2008-08-13 13:17:18 UTC
In answer to brion concerns at r37842
a) My patch did define global wgTitle.
b) Yes, it's lame, but even if useless makes the page validate, which was the point :)

Comment 12 Platonides 2009-03-11 20:54:44 UTC
Ploticus 2.41 was released today, with the fix for the ploticus side.
Comment 13 Mike.lifeguard 2009-06-27 22:47:01 UTC
I guess leaving this assigned to Erik doesn't make much sense as it's a shell request.
Comment 14 Sam Reed (reedy) 2010-12-01 17:59:27 UTC
Platonides, any idea where we are with this?
Comment 15 Platonides 2010-12-01 22:52:13 UTC
It should be fixed on the servers with Lucid (10.04), since it comes with ploticus 2.41-3. For Hardy (8.04), it will need to be upgraded since the package is 2.33-3.
Comment 16 Sam Reed (reedy) 2012-02-09 23:03:10 UTC
All servers are on lucid

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


Navigation
Links