allow reordering and saving of filter order

[insaner]

it is useful when working with a lot of filters to be able to reorder these and be able to keep this order across saves. i've already found out how to allow the reordering via drag and drop while working on another bug, but its the saving this state that needs to be done.

if someone wants to give a couple tips on how to accomplish this it would really speed things up for me to get this done.

[jazzynico (jazzynico)]

The filters list order depends on the filters order in the SVG file. Thus reordering the <filter> element in the <defs> should work.
Another possibility would be to add an inkscape:filter-order attribute in the filters definition. But I'm always a bit reluctant to add new inkscape specific attributes.

[insaner (insaner)]

would this make for a good GSoC easy-fix?

[jazzynico (jazzynico)]

Not sure it is that easy. Of course it would be easier with your partial patch attached to the report.
But it still needs some understanding on how the elements are managed internally.

[insaner (insaner)]

heres part one.. the easy part. this should set the rows reorderable, such that now you can drag and drop them into the right place..

what is needed is to make this sync with inkscape's internal representation of the data.. of which i know close to nothing about.

[Martin Owens (doctormo)]

See r12394 for a nice fix, it took a while to get the right solution and I learned a bit more about the codebase. But you can see from the diff, it's quite a small fix over all.

[insaner (insaner)]

wow awesome.. thats great! yeah theres no way that i couldve come up with that solution with my current working knowledge of gtk, etc.. this will definitely move me way forward in that direction and in solving future bugs i might run into (maybe even a couple that are currently open)

oh, by the way, i found that reordering is not marking the document as "dirty", (ie, puts a little asterisk next to the name calling out "save me!") so to save any reordering, something else in the document must be modified.

thanks martin!

[su_v (suv-lp)]

Follow-up report:
- Bug #1220143 “trunk: crash in Inkscape::UI::Dialog::FilterEffectsDialog::FilterModifier::on_filter_reorder ()”
  <https://bugs.launchpad.net/inkscape/+bug/1220143>

[su_v (suv-lp)]

Follow-up report:
- Bug #1239296 “trunk: interference of XML and Filter Effects Editor (hang on dialog open and app quit) (rev >= 12394)”
  <https://bugs.launchpad.net/inkscape/+bug/1239296>

[su_v (suv-lp)]

JFYI - the changes from r12394 have been partially disabled in r13055 due to regressions reported in bug #1239296.