Page MenuHomePhabricator

SVG rendering maximum should base on area, not largest size?
Closed, InvalidPublic

Description

A version of [[Reference_ranges_for_blood_tests]] attempted to replace a PNG of a large panoramic graph (4436x400 pixels) with an SVG -- [[Image:Reference_ranges_for_blood_tests_-_by_mass.svg]].

In theory, this should look nice... but our SVG rasterization limit of 2048x2048 hits it, so it gets rendered to 2048x185 and scaled up in-browser... which looks terrible. :)

http://upload.wikimedia.org/wikipedia/commons/thumb/7/7c/Reference_ranges_for_blood_tests_-_by_mass.svg/2048px-Reference_ranges_for_blood_tests_-_by_mass.svg.png

We don't want to render things super-huge, but for panoramic-style images, it shouldn't be terribly expensive. The limit should probably be changed to work on area, like our existing restrictions on PNG resizing.


Version: unspecified
Severity: minor
URL: http://en.wikipedia.org/w/index.php?title=Reference_ranges_for_blood_tests&oldid=248679203#By_mass

Details

Reference
bz16416

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 10:25 PM
bzimport set Reference to bz16416.
bzimport added a subscriber: Unknown Object (MLST).
  • Bug 41373 has been marked as a duplicate of this bug. ***

This is a bug I think, not an enhancement. Just common sense and what it should be. The related ticket shows another extreme case that tripped this. These files are out there.

I don't believe this is a problem anymore, at least with Thumbor. The linked file can be rendered with a width much higher than 2048px: like 5000px for example. I can't find a size limit for SVGs other than "does librsvg take more than 60s to process it".