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

Re: Changelist group checking/unchecking

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2007-02-03 17:15:00 CET

Alexander Klenin wrote:
> In a recent nightly, I tested new context menu items relating to
> changelists.
> In general, they work quite well, I especially like the new "Move to
> changelist" item ;-)
> I have following further questions/suggestions:
> 1) I thing the names "Check Group" and "Uncheck Group" are slightly
> confusing, as changelists are not referred to as "groups" anywhere
> else in the interface. The fact that this therm is used in the source
> does not help user much ;-)
> I suggest "Select/deselect changelist items", also without
> capitalizing every word.

Changed in revision 8643.

> 2) Is there any reason why check/uncheck is performed by double-click,
> not single-click?

I thought it's better to use a doubleclick to avoid accidental clicks.
The group headers don't have a checkbox, which means the user can't see
immediately that a click really does something.

> 3) How about adding "Select only this changelist" item, which would
> check all items current changelist and uncheck everything else?
> This would directly correspond to "svn ci --changelist" of the command
> line client.

Use the checkbox below the file list "De/select all items", then select
the group items. I don't think another context menu entry is needed.
But I'm open to change this if people think that it's needed.

> 4) Perhaps add a "Dismantle(?) changelist" to remove all files from
> it. It is, of course, possible to select and remove all files from the
> changelist, but I am afraid it may me non-obvious to some users. Or
> may be not -- not sure here.

As long as there are changelists shown, the user can just select all
items of that changelist and use the context menu to remove those from
the changelist.
But I'm open to change this if people think that it's needed.

> 5) "Keep changelist" checkbox name is slightly confusing when more
> then one changelist is committed. Perhaps replace it with "Keep
> changelists" in that case?

Also changed in revision 8643.


   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 Sat Feb 3 17:15:16 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.