Page MenuHome

Inkscape broke our Teddy bear, Toys and UI Icons - How to fix the mess?
Closed, ResolvedPublic

Description

So… It appears Inkscape decided to switch from 90dpi to 96dpi (to follow recommendations they claim…). And when you open an 'old' SVG in new inkscape, it enforces some kind of conversion.

Problem is, I tried all three choices, and none work 'off hands' for us (either generation of PNG at build time crashes, or we get black or shattered icons in UI).

Did not dig that much more, but I suspect that since 90 to 96 DPI is not a simple nice round conversion, we end up with fractions of our pixels, and our icon-generation tools expect some precise, integer pixel size of icons.

In other words, it would appear that we cannot keep our current fixed-pixel size of icons (at least, not without some manual conversion of the whole SVG).

There are two issues here actually:

  1. Compatibility: Which ever solution we choose, we'll have to forbid either older inkscape, or new ones. It is clearly impossible to support/allow usage of both.
  2. Fixing the issue itself: I see at least three options, there may be more…
    1. Ignore new inkscape, and keep using older one (will be a bit annoying to keep old inkscape at hands in linux distros though, and this is not a good future-proof solution…).
    2. Accept change of our icons size (which seems necessary if we want to use some new converted SVG off-hands).
    3. Manually tweak bloody SVG file after conversion to 96dpi, to put it back in shape that we can use to generate our current icon sizes.

Details

Type
To Do

Event Timeline

Bastien Montagne (mont29) renamed this task from Inkscape broke our Teddy bear, Toys and UI - How to fix the mess? to Inkscape broke our Teddy bear, Toys and UI Icons - How to fix the mess?.
Bastien Montagne (mont29) triaged this task as Confirmed, Medium priority.

T53471 has the issue and solution. Use width and height instead of DPI setting to render.

https://hastebin.com/ekihitudoc.diff

Bastien Montagne (mont29) closed this task as Resolved.Dec 13 2017, 6:00 PM
Bastien Montagne (mont29) claimed this task.