extensions mix selection order

Bug #182832 reported by jfb
14
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Medium
jfb

Bug Description

Some extensions depend on selection order like "pattern along path" or "interpolate" (the z-order of the output depends on it) and the new "scatter".

If you change a parameter in the extension dialog, this order is changed...
It makes it difficult to predict the result.

Extension should not depend on the selection order. The z-order would be better for instance, but I don't know an easy way to do it...

Ryan Lerch (ryanlerch)
Changed in inkscape:
importance: Undecided → Wishlist
status: New → Triaged
Revision history for this message
Tom Davidson (tjd-mit) wrote :

While the proposed solution is a feature request, the mixing up of selection order is a real bug that I've run into. jfb--could you provide step-by-step instructions that trigger the bug, starting from a new document? Thanks...

Changed in inkscape:
importance: Wishlist → Medium
status: Triaged → Incomplete
Revision history for this message
jfb (jf-barraud) wrote : Re: [Bug 182832] Re: extensions mix selection order

create 2 paths with white fill and black stroke.
place them so that one partially covers the other.
select the one in the back, then the one in the front.

efffect->interpolate.
the preview shows that the new shapes are added frome back to front.

change which ever parameter you want in the effect dialog.
the new shapes are now added from front to back...

you can do the same with pattern-along-path: which shape is considered the
pattern and which the skeleton depends on the selection order; this is
screwed up as soon as you change a parameter.

Revision history for this message
jfb (jf-barraud) wrote :

create 2 paths with white fill and black stroke.
place them so that one partially covers the other.
select the one in the back, then the one in the front.

efffect->interpolate.
the preview shows that the new shapes are added frome back to front.

change which ever parameter you want in the effect dialog.
the new shapes are now added from front to back...

you can do the same with pattern-along-path: which shape is considered the pattern and which the skeleton depends on the selection order; this is screwed up as soon as you change a parameter.

Tom Davidson (tjd-mit)
Changed in inkscape:
status: Incomplete → Confirmed
Revision history for this message
jfb (jf-barraud) wrote :

a work around is to use zOrder.instead of selection order. It is less handy, but still makes sens (at least for "interpolate").

I revisited the scripts of pathalongpath, scatter, and interpolate, and updated them to use the z-order.
As a bonus, interpolate now also supports objects like groups, clones, stars...

The files are submitted in https://bugs.launchpad.net/inkscape/+bug/182794 since the changes are also relevant for this bug.

Revision history for this message
jfb (jf-barraud) wrote :

Hi! Here is a patch containing all the changes of interpolate, path-along-path, path-scatter, path-modifier, simple-transform, and rubber-stretch.
(same as https://bugs.launchpad.net/inkscape/+bug/182794)

Revision history for this message
Tom Davidson (tjd-mit) wrote :

Could a Dev please take a look at JFB's patch and commit it if looks good?

Changed in inkscape:
assignee: nobody → jf-barraud
milestone: none → 0.46
Revision history for this message
Popolon (popolon) wrote :

path along a path, rubber stretch and related facilties are commited, but there are still some problems for interpolate and path-scatter :

In my tests, path-scatter was not visible in the menu, there are some investigation to do for that issue ???

interpolate, we could ask the Interpolate author (and some other user/dev) if he/them thinks that's a benefit.

The fact than interpolate can manage special object, is a good point, and should be committed ???

Revision history for this message
jfb (jf-barraud) wrote :

pathscatter does not showup because of an uppercase S in pathscatter.py
here is the corrected file.
The Makefile.am should also be updated in svn.

Revision history for this message
jfb (jf-barraud) wrote :
Changed in inkscape:
status: Confirmed → In Progress
Revision history for this message
Popolon (popolon) wrote :

Tested it again, with this patchs, the scatter script is now in menu, but when I execute it (selecting two simple paths), only click on live preview return me the following error in an error requester window (Inkscape has received additional data from the script executed. The script did not return an error, but this may indicate the results will not be as expected.):

  File "/tmp/ink_ext_XXXXXX.svgVJAN6T", line 1
    <?xml version="1.0" encoding="UTF-8" standalone="no"?>
    ^
SyntaxError: invalid syntax

Revision history for this message
Popolon (popolon) wrote :

For testing, the scripts of JFB are available here :
http://math.univ-lille1.fr/~barraud/Inkscape/pathdeform/

Revision history for this message
Bryce Harrington (bryce) wrote :

It's good to see the progress being made on this one, however given that it is just medium importance and seems to require more than a simple patch to fix it, this can be delayed to 0.47.

Changed in inkscape:
milestone: 0.46 → 0.47
Revision history for this message
bbyak (buliabyak) wrote :

jfb, you now have commit rights, can you please commit these fixes if they are tested enough, and close this bug?

thanks!

Revision history for this message
jfb (jf-barraud) wrote :
  • unnamed Edit (911 bytes, text/html; charset=ISO-8859-1)

Hi!

Oh, I forgot this one... Well, the only thing that is not commited yet is
about the interpolate extension:
I proposed to sort the z-order of the generated objects according the
z-order of the input, but someone said the actual situa

2008/4/14, bbyak <email address hidden>:
>
> jfb, you now have commit rights, can you please commit these fixes if
> they are tested enough, and close this bug?
>
> thanks!
>
>
> --
> extensions mix selection order
> https://bugs.launchpad.net/bugs/182832
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Revision history for this message
jfb (jf-barraud) wrote :
  • unnamed Edit (1.4 KiB, text/html; charset=ISO-8859-1)

Hi!

Oh, I forgot this one... Well, the only thing that is not commited yet is
about the interpolate extension:
I proposed to sort the z-order of the generated objects according the
z-order of the input, but I am not definitely sure it is a good thing.

However, all these were only quick work around: I maid some scripts
independant of the selection order, but the selection order is still
occasionally reverted...

I see it is marked as "work in progress". I think I messed this up a bit; I
am sorry. It was only a work-around in progress... Someone who knows
extension code better than I do will solve this more efficiently.
I think I should close the bug and reopen a new one? Is this ok?
--jfb.

2008/4/14, bbyak <email address hidden>:
>
> jfb, you now have commit rights, can you please commit these fixes if
> they are tested enough, and close this bug?
>
> thanks!
>
>
> --
> extensions mix selection order
> https://bugs.launchpad.net/bugs/182832
> You received this bug notification because you are a direct subscriber
> of the bug.
>

jfb (jf-barraud)
Changed in inkscape:
status: In Progress → Fix Released
Revision history for this message
quazgar (quazgar) wrote :

Is this actually fixed, or are there just workarounds implemented for the affected extensions? I ran into this myself when writing my own extension. Unless it's really fixed I'd like to reopen it (or write a new bug).

jazzynico (jazzynico)
tags: added: extensions-plugins
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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