Path effect icon (w suggestion)

[insaner]

It's time "Path effect" got its own icon. Instead of reinventing the wheel, I just used the same icon as
"path-effect-parameter-next" in icons.svg
without the yellow triangle (and replaced the "path-effect-parameter-next" icon with a clone to this one).

I have attached a patch. Please let me know if there are any problems or objections to the choice of icon for this purpose.

[su_v (suv-lp)]

Thanks for the patch - the diff for icons.svg seems rather large; I'll try whether I manage to create a smaller one.

[su_v (suv-lp)]

Smaller diff; proposed icon placed in the 'menu' section; icon name changed to 'dialog-path-effects'

[su_v (suv-lp)]

Updated patch (v3) includes new icon for symbolic icons as well.

[insaner (insaner)]

Patch from #5 tested working, no problems.

[jazzynico (jazzynico)]

Patch tested successfully on Windows XP (32bit) with Inkscape trunk rev. 14355.

[insaner (insaner)]

Hi su_v,
is this ready to commit and close?

[jazzynico (jazzynico)]

@insaner - Unfortunately the patch no longer applies correctly (Hunk #4 fails in icons.svg). Could you please take a look?

[Tavmjong Bah (tavmjong-free)]

Please be careful in editing the icons.svg file or it can gain a lot of cruft. It's probably better to hand edit. I can do that.

[insaner (insaner)]

I was just looking at https://bugs.launchpad.net/inkscape/+bug/1113302 and I'm thinking I can do both, if you guys don't mind that.

[su_v (suv-lp)]

Attaching updated minimal diff if patch from comment 5 against 0.91+devel r14757 - @Tav, please review and commit if ok.

[Tavmjong Bah (tavmjong-free)]

Committed in revision 14758 after purging a few unneeded properties.

[insaner (insaner)]

Fix confirmed. Thanks all!