Hans-Emil Skogh wrote:
>>> Maybe we should do that differently? Only uncheck items when "None" is
>>> clicked, and 'add' the checkmarks for all others? I.e., a click on
>>> "Deleted" will check all deleted items, another click on "Added" will
>>> leave the deleted items checked but also check all added items?
>>> If we do it that way, then disabling the links shouldn't be necessary
> This makes sense to me as well.
>> My first idea on how to do this is as follows.
>> Toggle: Added, Deleted, Modified
> And what would happen if you manually uncheck half of the added files
> and then click added? Or unchecks all of them?
I wouldn't implement this as a toggle function. A click on the link
would only check the corresponding items and leave all not-affected
items as they were: if they were checked, they would still be checked,
if they were not checked they would still be unchecked.
Having a link behave as a toggle button is bad - either we would have to
use a real toggle button (with a 'pressed' and 'normal' state) or not
toggle but only check.
But toggle buttons look ugly (my opinion - others might disagree).
>> However, if there are no "Added" items in the commit dialog, then I do
> not believe
>> that the "Added" selection pattern link should be disabled. I think
> that by clicking
>> "Added" when there are no added items, it should end up doing nothing
> at all
>> except iterating every item in the commit dialog to search for items
> with the "Added" state.
> I still thinks that options that doesn't do anything should be grayed
> out. If you save a document, the save option gets grayed out until you
> make a change. If you press undo, the button gets grayed out when there
> are no more things to undo. They doesn't just stay enabled wating for a
> user to press it and then silently do nothing.
> Graying out an option does also provide additional cues to the user,
> like that the document has not been modified since the last save in case
> of a save button, or the presence of added files in a list of files in our.
> I just can't see why we should make an exception from a perfectly good
> GUI defacto standard here, without getting a single benefit from doing so!
It sure would be possible to gray such links. But it's not as easy as
you might think. That will take me a while...
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest Interface to (Sub)Version Control
/_/ \_\ http://tortoisesvn.net
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2009-05-28 18:47:07 CEST