Last modified: 2014-04-22 14:10:44 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 T65590, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 63590 - Hovercards: "Unexpected value 200 parsing viewBox attribute." in console
Hovercards: "Unexpected value 200 parsing viewBox attribute." in console
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Popups (Other open bugs)
master
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2014-04-06 09:22 UTC by spage
Modified: 2014-04-22 14:10 UTC (History)
3 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description spage 2014-04-06 09:22:43 UTC 
In Firefox 28 roll over any link with an image, see
   Unexpected value 200 parsing viewBox attribute.
in Browser console/Firebug. In Chromium developer console see
   Warning: Problem parsing viewBox="200" 

The logic in line 169 is faulty. As a result the viewBox is set to 200 (if the image is wide not tall) instead of a string of four numbers like "0 0 300 200", and the former is invalid SVG.
Comment 1 Gerrit Notification Bot 2014-04-07 09:31:15 UTC 
Change 124296 had a related patch set uploaded by Prtksxna:
Remove viewBox from SVG, not needed.

https://gerrit.wikimedia.org/r/124296
Comment 2 Gerrit Notification Bot 2014-04-22 11:16:48 UTC 
Change 124296 merged by jenkins-bot:
Remove viewBox from SVG, not needed.

https://gerrit.wikimedia.org/r/124296
Comment 3 Bartosz Dziewoński 2014-04-22 14:10:44 UTC 
Looks fixed to me.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links