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

Re: Rev graph hide cross lines

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Fri, 27 May 2011 17:09:45 +0200

On 27.05.2011 10:11, Simon Large wrote:
> On 27 May 2011 08:15, Hans-Emil Skogh<Hans-Emil.Skogh_at_tritech.se>
> wrote:
>>>>> Why do we have the option to reduce cross lines? Is there any
>>>>> reason you would ever NOT want that selected? Could we just
>>>>> enable it permanently and get rid of the option?
>>> The major difference is that the graphs tend to occupy more space
>>> with the option enabled. So, we may remove the option and default
>>> to "on" but will lose some overview in certain cases.
>>
>> Maybe we should invert the option and change the wording on the
>> option to "Allow crossing lines" and the alt text to "Make the
>> graph more compact by allowing lines to cross revision numbers". It
>> should then be off by default. I think this would make more sense.
>> I have also wondered why the option is there since the current
>> wording makes it seem like it would have a negative impact to
>> disable the option.
>
> +1
>
> The reasons not to use this option seem pretty unusual. I would also
> suggest to remove the toolbar icon for this option and just leave it
> in the view menu. I never could make out what that icon picture is
> ;-)

Done in r21472.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2750393
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-05-27 17:09:57 CEST

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.