icons.svg can be greatly reduced in size

Bug #1113302 reported by insaner
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Wishlist
insaner

Bug Description

this might make startup much much faster.. im willing to do the cleanup, but is this something desirable? the cleanup consists of making all objects using duplicate gradients all use the same single gradient.. maybe even naming these gradients for convenience..

what say ye?

Tags: ui
su_v (suv-lp)
tags: added: ui
Changed in inkscape:
importance: Undecided → Wishlist
Revision history for this message
ScislaC (scislac) wrote :

I'm not in favor of re-using one gradient in the current icons file. There are gradients for varying base colors, and if those could be preserved where it makes sense it would be great. But in addition to that, before opening and editing the icons.svg, having the following in your preferences.xml file helps to reduce file size (per http://wiki.inkscape.org/wiki/index.php/Icons#Single_SVG_File ):

    <group
       id="svgoutput"
       usenamedcolors="0"
       numericprecision="7"
       minimumexponent="-8"
       inlineattrs="1"
       indent="0" />

Personally I'm curious if we can educe the numeric precision by a couple digits and not lose any real detail/precision.

Revision history for this message
su_v (suv-lp) wrote :

On 2013-02-11 14:50 +0100, ScislaC wrote:
> I'm not in favor of re-using one gradient in the current icons file.

Not in favor? Would you then also prefer to fork (aka duplicate) those which already are re-used (aka shared) multiple times?

Revision history for this message
ScislaC (scislac) wrote :

Sorry, was was misreading as if all objects using the same single gradient... obviously forked dupes should be removed (should be much easier to do with the new F&S dialog features and Find & Replace)

jazzynico (jazzynico)
Changed in inkscape:
status: New → Triaged
insaner (insaner)
Changed in inkscape:
assignee: nobody → insaner (insaner)
Revision history for this message
insaner (insaner) wrote :

I guess "greatly" was a severe overshoot.. but anyways, I modified the file to re-use as many common gradients as I could. If there are no objections, I will check in by Wednesday.

Revision history for this message
insaner (insaner) wrote :

committed to r14840

Changed in inkscape:
status: Triaged → Fix Committed
Revision history for this message
insaner (insaner) wrote :

Quick fix for missing radialgradient definition

Revision history for this message
insaner (insaner) wrote :

Missing radialgradient fixed in r14841

Revision history for this message
Qantas94Heavy (qantas94heavy) wrote :

Fix released with Inkscape 0.92.

Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.