Re: TortoiseMerge / TortoiseIDiff am I missing something?

On 6/20/07, Lübbe Onken <l.onken@rac.de> wrote:

> Just installed the nightly. Looks nice :) There was however a tiny bug
> introduced. When you switch the overlay mode off, the left image isn't fit
> to the window size again, but keeps its scaling factor.

I'll check that.

> Possible improvements:
> When I select "fit images together", both images are scaled to > 100%
> Wouldn't it be better to leave the scaling factor of the larger image as it
> is and only stretch the smaller one to fit the large?

Depends: if the images are bigger than the current window, they're
scaled to <100%, not >100% :)

> When I zoom in/out after the "stretch", both images are zoomed to absolute
> values again. IMHO it would be better if the zoom would be applied relative
> to the "stretch" factor. That's why I also suggested that the stretch option
> should be a toggle.

I'll change it so they zoom independently.

> Example:
> Image1=640x480
> Image2=800x600
>
> Side by side zoom factor:
> Image1=80%
> Image2=64%
>
> Overlay zoom factor:
> Image1=100%
> Image2=100%
>
> Stretch to fit zoom factor:
> Image1=133% (Current 138%) -> Image1 has a stretch factor of 1,33
> Image2=100% (Current 110%) -> Image2 has a stretch factor of 1,00
>
> Zoom in after stretch to fit:
> Image1= Stretch1*Zoom = 1,33*125% = 166,25% (Current 175%)
> Image2= Stretch2*Zoom = 1,00*125% = 125% (Current 175%)
>
> I'd like to be able to stretch -> zoom in -> pan around.

Ok, I have to object now. I definitely won't add yet another button
for yet another 'fit to ...' function. I was more thinking of dropping
the existing button and only leave the new one (well, combine those
two: in overlay mode always use the new one, in non-overlay mode use
the old one).

This is a simple diff viewer, and I'd like to keep it simple if possible.

Stefan

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org