negative shift for Tile Clones

Bug #170978 reported by Norbert Nemec
4
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Wishlist
Unassigned

Bug Description

Tile clone shift should work for values less than -100%
as well. There are several uses where it may come in
handy and no obvious reason why it should be prohibited.

Tags: clones ui
Revision history for this message
Hystrix (hystrix-) wrote :

It allows negtive values in Inkscape 0.45.1, built Mar 21 2007

Changed in inkscape:
importance: Undecided → Wishlist
status: New → Fix Released
Revision history for this message
Florent Becker (florent-becker-ens-lyon) wrote :

Negative values should reach further than -100%; in 0.45.1 and svn as of feb 7 2007, they don't. Here is a patch, which also sets the max absolute value to 10000%, as 1000 can be puny at times (large tilings of small objects).

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

Thanks for the patch! I'll milestone this so that it will get reviewed by one of the devs for inclusion.

(Facundo--the request is for negative values less than -100%, not just negative values. In current SVN rev 17256, this limit is still in place)

Changed in inkscape:
milestone: none → 0.46
status: Fix Released → Fix Committed
status: Fix Committed → Confirmed
Revision history for this message
Bryce Harrington (bryce) wrote :

This looks like a good feature, although we'd need to test it to ensure expanding the bounds can't lead to out of bounds errors anywhere. It's a bit too late in the release cycle for it, so I'm dropping the milestone.

Changed in inkscape:
milestone: 0.46 → none
Revision history for this message
Florent Becker (florent-becker-ens-lyon) wrote :

Is there a hope to see this patch in 0.46.1, anything I can do to help including it ? I've been testing it for a while, and I haven't experienced any problems. It really is a feature i've been missing, and I don't want to wait until 0.47.

Revision history for this message
bbyak (buliabyak) wrote :

Thanks for the patch, committed to svn trunk

I'm not too happy about increasing these limits indiscriminately - it makes it too easy to get a weird result accidentally, and harder to reset the values. Perhaps we need to add some way to quickly reset the values to 0, 100, and -100. One approach is to use the same spinbutton right-click menu as in the controls bar, with entries for these common values.

Changed in inkscape:
status: Confirmed → Fix Released
Revision history for this message
bbyak (buliabyak) wrote :

Bryce: I approve this patch for 0.46.1, it's not critical but safe

Changed in inkscape:
assignee: nobody → bryceharrington
milestone: none → 0.46.1
status: Fix Released → Confirmed
Revision history for this message
Bryce Harrington (bryce) wrote :

It's not necessary to assign 0.46.x bugs to me - just milestone them or target them to 0.46.1 and I'll catch them when I'm doing release works.

Changed in inkscape:
assignee: bryceharrington → nobody
Revision history for this message
Florent Becker (florent-becker-ens-lyon) wrote :

As for increasing the limits, is it possible to have the spinbutton be limited to say [-1000,1000], and allow arbitrary values to be entered manually ?

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

Fixed in SVN head; sounds like we won't be doing a 0.46.1 after all so no need for this to still be open.

Changed in inkscape:
status: Confirmed → Fix Released
su_v (suv-lp)
tags: added: clones ui
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.