[hugin-ptx] Re: Selecting and moving multiple images in Hugin's Image Tab.
Thomas. Apologies for the re-posting.. I did not see the orignal appear in the group? Since I'd used another emailer to send it I assumed it had been block. Yes I'd completely over-looked the fact that non-adjacent images could be selected.. The 'shuffling' of image does generate alot of 'history'. I'd been thinking about Yuv's recent post on undo history. For some reason I never use undo, not that I don't make mistakes, just Hugin provides other ways to get around mistakes... So again I never thought to test this patch for it's undo behavoiur. Looks like I do want to write methods to delete and insert images into the list. Thank-you for taking the time to review the patch.. Regards Stephen On Sep 10, 12:01 pm, T. Modes thomas.mo...@gmx.de wrote: Hi Steeve, Seems too easy, can anybody see anything I've missed? The patch needs more work: 1.) First, it creates for every image an own command in the command history, what makes is complicated to use the undo function. 2.) Your patch goes mad, if you select several non-adjacent images. In this case it moves n images following the first selected image (with n the number of selected images), but not the selected images. This is the main reason why moving several images is not so easy. You can not assume that all selected images are adjacent. If you force this, you break exisiting functionality. Thomas -- You received this message because you are subscribed to the Google Groups Hugin and other free panoramic software group. A list of frequently asked questions is available at: http://wiki.panotools.org/Hugin_FAQ To post to this group, send email to hugin-ptx@googlegroups.com To unsubscribe from this group, send email to hugin-ptx+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/hugin-ptx
Re: [hugin-ptx] Re: Selecting and moving multiple images in Hugin's Image Tab.
On September 24, 2010 01:50:22 am T. Modes wrote: That's the same patch as in http://groups.google.com/group/hugin-ptx/browse_thread/thread/dfd4cc8affbfd c39 I answered there. oh, I thought it was an improvement based on your thorough and helpful feedback. sorry Yuv signature.asc Description: This is a digitally signed message part.
[hugin-ptx] Re: Selecting and moving multiple images in Hugin's Image Tab.
That's the same patch as in http://groups.google.com/group/hugin-ptx/browse_thread/thread/dfd4cc8affbfdc39 I answered there. Thomas -- You received this message because you are subscribed to the Google Groups Hugin and other free panoramic software group. A list of frequently asked questions is available at: http://wiki.panotools.org/Hugin_FAQ To post to this group, send email to hugin-ptx@googlegroups.com To unsubscribe from this group, send email to hugin-ptx+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/hugin-ptx
[hugin-ptx] Re: Selecting and moving multiple images in Hugin's Image Tab.
Hi Steeve, Seems too easy, can anybody see anything I've missed? The patch needs more work: 1.) First, it creates for every image an own command in the command history, what makes is complicated to use the undo function. 2.) Your patch goes mad, if you select several non-adjacent images. In this case it moves n images following the first selected image (with n the number of selected images), but not the selected images. This is the main reason why moving several images is not so easy. You can not assume that all selected images are adjacent. If you force this, you break exisiting functionality. Thomas -- You received this message because you are subscribed to the Google Groups Hugin and other free panoramic software group. A list of frequently asked questions is available at: http://wiki.panotools.org/Hugin_FAQ To post to this group, send email to hugin-ptx@googlegroups.com To unsubscribe from this group, send email to hugin-ptx+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/hugin-ptx
