hugin-devs team mailing list archive

[rew <679919@xxxxxxxxxxxxxxxxxx>]

Hi,

Your patch contains sometimes differences to the original in
tabs/spaces.

     XRCCTRL(*this, "cp_editor_celeste_button", wxButton)->Disable();
     XRCCTRL(*this, "cp_editor_choice_zoom", wxChoice)->Disable();
     XRCCTRL(*this, "cp_editor_previous_img", wxButton)->Disable();
+	XRCCTRL(*this, "cp_editor_swap_img", wxButton)->Disable();
     XRCCTRL(*this, "cp_editor_next_img", wxButton)->Disable();

You probably used a tab, while the original uses spaces (and less than
8). To get your code accepted please go through your patch and copy
the tabs/spaces from the lines above or below the line you patched.

Thanks! Looks good!

        Roger.

-- 
** R.E.Wolff@xxxxxxxxxxxx ** http://www.BitWizard.nl/ ** +31-15-2600998 **
**    Delftechpark 26 2628 XH  Delft, The Netherlands. KVK: 27239233    **
*-- BitWizard writes Linux device drivers for any device you may have! --*
Q: It doesn't work. A: Look buddy, doesn't work is an ambiguous statement. 
Does it sit on the couch all day? Is it unemployed? Please be specific! 
Define 'it' and what it isn't doing. --------- Adapted from lxrbot FAQ

-- 
You received this bug notification because you are a member of Hugin
Developers, which is subscribed to Hugin.
https://bugs.launchpad.net/bugs/679919

Title:
  Swap images in controlpoints window.

Status in Hugin - Panorama Tools GUI:
  New

Bug description:
  If the two images in the control point tab align, but have been taken
  right-to-left, it is easier to work on the middle area, than to work
  on the left side of the left image, and on the right side of the right
  image. So I usually swap them. It might be useful to have a button do
  this instead of having to change both image selectors.

To manage notifications about this bug go to:
https://bugs.launchpad.net/hugin/+bug/679919/+subscriptions