[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: TortoiseMerge / TortoiseIDiff am I missing something?

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2007-06-20 14:44:56 CEST

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
Received on Wed Jun 20 14:44:50 2007

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.