About screen - Missing image and small window.

Gracias rolando por rellenar el reporte.

Thanks Rolando for fill bug.

Reproduced in Debian testing Inkscape 0.92.0 15324 and trunk Inkscape 0.92+devel 15450.
Proposed patch not tested yet: https://inkscape.org/en/paste/10624/

New patch improved by suv:
https://gist.github.com/su-v/6152a6c27b380ac2f888e1e647c9f90e

Based on tests with archived release builds:
- not reproduced with Inkscape 0.48.5, 0.91 (es_ES.UTF-8)
- reproduced with Inkscape 0.92.0 r15299 (es_ES.UTF-8)

The reported issue affects multiple interface languages in 0.92 - the about screen is no longer available in translated versions, but the translator comment in the po file was never updated accordingly, nor was made sure that Inkscape falls back to the basic version of the about screen if no translated file is found.

Patch mentioned by Jabiertxof in comment #4 (for a fallback) tested successfully with lp:inkscape/0.92.x r15345, using es_ES.UTF-8 locale.

I would recommend that the translator comment is updated nevertheless (it seems that for now there are no plans to return to translated about screens).

--
replaces (hidden) comment 5 to clarify reference to tested patch.

Please milestone for 0.92.x - affected locales in 0.92.x are be, br, es, eu, gl, id, kn, nl, pl and sl (thx @Mc). All could be fixed by adding a fallback to 'about.svg' in the source of the About dialog.

Typo:
- Please milestone for 0.92.x
+ Please milestone for 0.92.1

The attached diff (from gist) provides fallback to 'about.svg'.

TODO: update translator comments.

Partial fix committed to trunk <lp:inkscape> in rev 15453:
https://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/15453

Another possibility (if the patch is not accepted for 0.92.1) would be to update all the affected languages, and warn translators (on the list) that they should not change the original string.
I'm going to ask Bryce if it's ok to backport.

A very simple proposal for the comment.

We can also remove (or comment) that part of the code and consider that about screens no longer need to be translated.

Proposal to update the comment (comment 12) looks good to me.

Comment fixed in lp:inkscape rev. 15456.

Cumulative patch tested successfully with lp:inkscape/0.92.x r15345 (on OS X), with some of the affected locales (be, es, eu, kn), both via LC_ALL (on the command line), and the interface language selection in Inkscape's preferences:

The patched build displays the About screen as expected, whereas with the unpatched build it is missing (in all tested locales).

Ack'd for landing on 0.92.x stable branch.

Fixed in lp:inkscape/0.92.x rev. 15346.

Thanks!