| phuedx | |
| May 11 2016, 10:10 AM |
| F3994727: no-svg-support.gif | |
| May 11 2016, 10:10 AM |
When testing 288094: Allow brackets in createImgThumbnail, I discovered that if SVG isn't supported by the UA, then the thumbnail is requested but isn't rendered:
This is because the styles that position the SVG thumbnail aren't applied to the <div> fallback created by #createImgThumbnail (see resource/ext.popups.core.less).
Browser: Google Chrome (50.0.2661.94)
OS: Mac OS X (10.11.4)
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Make SVG thumbnails show for Internet Explorer | mediawiki/extensions/Popups | master | +8 -10 |
Using the same browser and OS it seems like upon hovering on the http://en.wikipedia.beta.wmflabs.org/wiki/Birdlike_dinosaur a dwell on the Archaeopteryx link yields the SVG. Able to reproduce?
@dr0ptp4kt: I should've been clearer, you need to forcefully disable SVG support or use a UA that doesn't support SVG, e.g. IE, which is explicitly blacklisted.
The SVG isn't being rendered. When SVG isn't supported, we render a <div> with a background-image style but it isn't styled correctly. The expected behaviour should be "the fallback div is rendered correctly".
So this one sucked me in.
A day later, I'm still confused. ResourceLoader doesn't support IE8 and IE9 can handle SVGs.
I was reading up on SVG rendering problems in IE9 (not to well documented in the code) and unfortunately didn't quite understand the code so ending up doing lots of code hygiene patches and found a fix for bug T68496 in the process.
I haven't got too far if someone wants to read through the chain of commits I've created to navigate through and document this code: https://gerrit.wikimedia.org/r/#/c/288552/
and pick up where I've left off...
Change 288636 had a related patch set uploaded (by Jdlrobson):
Make SVG thumbnails show for Internet Explorer
Boom. Just as I thought - we can render SVGs in IE9 so no need for a fallback. (Note in IE < 9 ResourceLoader blocks JavaScript support anyway).
This might have been fixed by 207389: Use correct selector for div's that hold PNGs when SVGs aren't supported. Verifying locally…
Verified. The original bug has now been fixed but there's still 288636: Make SVG thumbnails show for Internet Explorer to review, test, and merge.