[hugin-ptx] Re: Button bar patch (was:Some programming tasks...)

[T. Modes]

> I think it is a huge improvement over the old new Preview without
> tabs.
>

I commit the patch to branch 2010.0 and also to trunk. The layout mode
has its own tab. So further functions can be easily added to the
toolbar.

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: Button bar patch (was:Some programming tasks...)

[Bruno Postle]

On Wed 30-Dec-2009 at 01:41 -0800, T. Modes wrote:
>So please test the patch against the 2010.0 branch. When it works, I
>will commit it to branch and port it to the trunk.

I think it is a huge improvement over the old new Preview without 
tabs.

..and it finally makes the preview very usable with the 800x640 
screen on the Asus Eee:

http://www.flickr.com/photos/36383...@n00/4236993228/
http://www.flickr.com/photos/36383...@n00/4236993230/
http://www.flickr.com/photos/36383...@n00/4236993234/
http://www.flickr.com/photos/36383...@n00/4236993236/

-- 
Bruno

-- 
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: Button bar patch (was:Some programming tasks...)

[T. Modes]

Hi Bruno,

> The labels are there on Linux, so this is probably some wxwidgets
> quirk.
>

I tried with Harrys help to overcome this problem. But there seems
some limitiation of the wxToolBar implementation on wxMac.
Therefore I redesigned the patch. The most changes are internal. There
are only some little changes in the GUI. This version works fine on
mac. The current version can found at
https://sourceforge.net/tracker/?func=detail&aid=2914002&group_id=77506&atid=550443.
It needs also the small png from the 7z archive of the previous
version.
So please test the patch against the 2010.0 branch. When it works, I
will commit it to branch and port it to the trunk.

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: Button bar patch (was:Some programming tasks...)

[Bruno Postle]

On Sun 20-Dec-2009 at 08:52 +0100, Harry van der Wolf wrote:
>
>I built the last extended patch on OSX and I certainly like it. There is one
>drawback: the icons are very small and miss the texts. I suppose that can be
>easily fixed later on, but currently you have to hover over the
>buttons/icons until the tooltip shows up to know exactly what it does.

The labels are there on Linux, so this is probably some wxwidgets 
quirk.

>See screenshots here:
>
>
>
>
>
>
>
>Another remark: The move/drag tab should be the second tab IMO.

Maybe it should be first.  This sure to change, e.g. the layout mode 
would be another tab, as would the 'old' preview.

Though I don't see this as a workflow that you would step through, 
the tabs each give you a different 'view' or way of interacting with 
the panorama, and there is space for more.

-- 
Bruno

-- 
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: Button bar patch (was:Some programming tasks...)

[Harry van der Wolf]

Hi Bruno, Thomas,


2009/12/19 Bruno Postle 

> On Sat 19-Dec-2009 at 08:48 -0800, T. Modes wrote:
> >
> >I uploaded a modified version of the toolbar patch to
> >
> https://sourceforge.net/tracker/?func=detail&aid=2914002&group_id=77506&atid=550443
> .
> >I moved most of the controls into the tab bar. I have not found a good
> >position for the fov slider in the tab. They become too small to allow
> >a fine selection of the fov. So they remain at the usual position.
>
> I think this clicking tabs is a very natural way to change the
> modes, some more screenshots here:
>
> http://www.flickr.com/photos/36383...@n00/4197750405/
> http://www.flickr.com/photos/36383...@n00/4197750407/
> http://www.flickr.com/photos/36383...@n00/4197750413/
> http://www.flickr.com/photos/36383...@n00/4197750415/
>
> It certainly frees a lot of screen space for a larger preview
> canvas.
>
>
I built the last extended patch on OSX and I certainly like it. There is one
drawback: the icons are very small and miss the texts. I suppose that can be
easily fixed later on, but currently you have to hover over the
buttons/icons until the tooltip shows up to know exactly what it does.
As experienced user you are familiar with the buttons, but without the
accompanying texts it makes you hesitate.

See screenshots here:







Another remark: The move/drag tab should be the second tab IMO. After
creating the CP's you have a "small" image (5-first_image.png), probably not
centered both horizontal and vertical and with a curved horizon (at least:
that's always the case on OSX with partial panos). The very first thing I
always do is press 3 buttons: center it, FOV estimation and straighten.


Harry

-- 
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: Button bar patch (was:Some programming tasks...)

[Bruno Postle]

On Sat 19-Dec-2009 at 08:48 -0800, T. Modes wrote:
>
>I uploaded a modified version of the toolbar patch to
>https://sourceforge.net/tracker/?func=detail&aid=2914002&group_id=77506&atid=550443.
>I moved most of the controls into the tab bar. I have not found a good
>position for the fov slider in the tab. They become too small to allow
>a fine selection of the fov. So they remain at the usual position.

I think this clicking tabs is a very natural way to change the 
modes, some more screenshots here:

http://www.flickr.com/photos/36383...@n00/4197750405/
http://www.flickr.com/photos/36383...@n00/4197750407/
http://www.flickr.com/photos/36383...@n00/4197750413/
http://www.flickr.com/photos/36383...@n00/4197750415/

It certainly frees a lot of screen space for a larger preview 
canvas.

-- 
Bruno

-- 
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: Button bar patch (was:Some programming tasks...)

[T. Modes]

> It can move too.  It would be great to move everything from the
> bottom to free the space for an expanded preview canvas.

I uploaded a modified version of the toolbar patch to
https://sourceforge.net/tracker/?func=detail&aid=2914002&group_id=77506&atid=550443.
I moved most of the controls into the tab bar. I have not found a good
position for the fov slider in the tab. They become too small to allow
a fine selection of the fov. So they remain at the usual position.
The archive contains the patch file and 3 images which should be
copied into src/hugin1/hugin/xrc/data

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: Button bar patch (was:Some programming tasks...)

[Bruno Postle]

On Thu 17-Dec-2009 at 22:40 -0800, T. Modes wrote:
>
>Some more question: what's about the advanced parameters of some
>projections? Currently they show up beside the projection box. When
>the projection is moved inside the tab, should these parameters also
>move in the same box? Or should the stay at the bottom?

It can move too.  It would be great to move everything from the 
bottom to free the space for an expanded preview canvas.

-- 
Bruno

-- 
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: Button bar patch (was:Some programming tasks...)

[T. Modes]

Hi Bruno,

> Some ideas:
>
> The tab bar eats up some screenspace, but actually it can be used to
> free up some more, the 'Projection' box can be its own tab, and the
> 'Blend mode' and 'EV' functions can move to the 'Preview' tab (not
> sure about this name though).
>
> The Centre, Fit, Autocrop and Straighten buttons don't need to be on
> the 'Preview' tab.
>
> The Field of View text entry boxes can now be moved from the
> Stitcher tab to the 'Drag' tab.  I'd also move the field of view
> sliders to this tab and free up even more space.
>
> Similarly the Crop text entry boxes can be moved from the Stitcher
> tab to the 'Crop' tab
>
> When the layout mode is merged, this can have a tab too.
>

I will try it at the weekend.

Some more question: what's about the advanced parameters of some
projections? Currently they show up beside the projection box. When
the projection is moved inside the tab, should these parameters also
move in the same box? Or should the stay at the bottom?

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: Button bar patch (was:Some programming tasks...)

[Bruno Postle]

On Wed 16-Dec-2009 at 22:34 -0800, T. Modes wrote:
>
>Thanks for feedback. I uploaded an updated version of the patch which
>uses tabs to the patch tracker 
>(https://sourceforge.net/tracker/?func=detail&aid=2914002&group_id=77506&atid=550443).
>Please give it a try.

Great, I put some screenshots here so others can comment:

http://www.flickr.com/photos/36383...@n00/4193726620/
http://www.flickr.com/photos/36383...@n00/4193726634/
http://www.flickr.com/photos/36383...@n00/4193726636/

Some ideas:

The tab bar eats up some screenspace, but actually it can be used to 
free up some more, the 'Projection' box can be its own tab, and the 
'Blend mode' and 'EV' functions can move to the 'Preview' tab (not 
sure about this name though).

The Centre, Fit, Autocrop and Straighten buttons don't need to be on 
the 'Preview' tab.

The Field of View text entry boxes can now be moved from the 
Stitcher tab to the 'Drag' tab.  I'd also move the field of view 
sliders to this tab and free up even more space.

Similarly the Crop text entry boxes can be moved from the Stitcher 
tab to the 'Crop' tab

When the layout mode is merged, this can have a tab too.

>I could also add small images to the tabs.

Good idea.

-- 
Bruno

-- 
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: Button bar patch (was:Some programming tasks...)

[T. Modes]

Hi Bruno,

> I think it is the right direction.  I know it is wxwidgets problem,
> but it isn't obvious that the first button changes the 'mode' and
> rearranges the rest of the buttons, what if it was a series of tabs
> along the top? so you could see what the other 'modes' are before
> selecting them.
>
Thanks for feedback. I uploaded an updated version of the patch which
uses tabs to the patch tracker (https://sourceforge.net/tracker/?
func=detail&aid=2914002&group_id=77506&atid=550443).
Please give it a try.

I could also add small images to the tabs.

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: Button bar patch (was:Some programming tasks...)

[Bruno Postle]

On Mon 14-Dec-2009 at 22:44 -0800, T. Modes wrote:
>
>The patch was intended for the current trunk, after merging with
>autocrop feature, when some tools were not accessible. For 2009.4 the
>toolbar should not changed. It was intended for version 2010.0 or even
>2010.2.

Ok, I'll try and release rc3 as 2009.4.0 as soon as possible.

> when I try to levelling a pano, I also need fit to accomodate for 
> the changes.  So fit should also go into levelling, but then there 
> is no big difference between levelling and composing.

For spherical panoramas, the crop and field of view stuff isn't 
relevant.

>I arranged the buttons after my workflow and grouped them after the
>effect on the pano:
>- Preview (Identify)
>- Move pano or images
>- Crop pano

I think it is the right direction.  I know it is wxwidgets problem, 
but it isn't obvious that the first button changes the 'mode' and 
rearranges the rest of the buttons, what if it was a series of tabs 
along the top? so you could see what the other 'modes' are before 
selecting them.

-- 
Bruno

-- 
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: Button bar patch (was:Some programming tasks...)

[T. Modes]

Hi Bruno,

> It is definitely an improvement, though I'm not sure this is
> something we want to introduce to 2009.4.0 - The button bar in the
> current 2009.4 branch is identical to that in the already released
> 2009.2.0, we are not making anything worse.
>

The patch was intended for the current trunk, after merging with
autocrop feature, when some tools were not accessible. For 2009.4 the
toolbar should not changed. It was intended for version 2010.0 or even
2010.2.

> What it exposes is that we have these various 'modes' for the Fast
> preview (photometrics, identify, crop, drag), but these modes don't
> really correspond with the tasks you might want to perform:
>
>- Identifying and fixing bad image relationships (show points,
>   identify, layout stuff)
> - Checking and adjusting exposure and colours (photometrics, EV
>   button, future white balance?)
>- Levelling a panorama (drag, straighten)
>- Composing a picture (drag, fit, centre, crop)

I could extend the mode selection to provide the mentioned modes.
But which modes are reasonable? E. g. do we need levelling and
composing:
when I try to levelling a pano, I also need fit to accomodate for the
changes.
So fit should also go into levelling, but then there is no big
difference between levelling
and composing.
I arranged the buttons after my workflow and grouped them after the
effect on the pano:
- Preview (Identify)
- Move pano or images
- Crop pano

But I'm open to better proposals.

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