>>>> 3. (Improvement / New Feature)
>>>>> XXX.keyTyped() methods which select the next item matching the keypress
>>>>> should probably
>>>>> - Move in the opposite direction when SHIFT is pressed
>>> 
>>> Just as before, it is a continuation of the SHIFT key reversing direction
>> 
>> That's what I figured, but is this a common convention? I have never heard
>> of it before.
>> 
>> 
> I couldn't tell you of a single place off the top of my head where this
> does occur, but it was something that seemed natural to my (clearly)
> perverted way of thinking ;)

I can actually see the utility in it - but if it is not a widely known 
convention, users may not even think to try it, so not worth the effort of 
coding/testing.

I'd rather see some effort go into the wrapping functionality, since this is 
actually a common convention that we don't currently support.

>>>> 4. (Improvement / New Feature)
>>>> Pressing SPACE in TerraListViewSkin & TerraTreeViewSkin will toggle
>> check
>>>> mark state only when selection mode is SelectMode.SINGLE
>> 

> My use case might be that I have a list of items which I am navigating with
> the keyboard.
> I use SHIFT+arrow keys to define my selection and then want to toggle all at
> once to a common state,
> I will then perform some action on those items based on the checkbok state.
> Being able to toggle the checkbox state of potentially the entire list would
> speed things up immensely.
> 
> I see your point though and think that perhaps it is best to leave it to the
> application.
> If we were to implement it, my algorithm would be as follows
> 
> If selection contains more than a single checkbox state {
>  Toggle unchecked to checked
> } else {
>  Toggle entire selection
> }
> 
> (If mixed states are supported then go from unchecked -> mixed -> checked)
> 
> 
>>>>> 5. (Bug)
>> 
>> 
> Good stuff.  You are certainly on a roll today.  Closing out tickets like
> there is no tomorrow!

Well, there is a tomorrow, but I will be busy with other stuff then.  :-)

> 6. (Question)
>>>>> In TerraFileBrowserSkin, the ENTER key only works if
>>>>> keyboardFolderTraversalEnabled has been set, but
>>>>> DELETE & BACKSPACE work regardless.
>> 
>> I'm OK with dropping the style. The default value is true, and I can't find
>> any examples that set it to false. Do you want to make this change?
>> 
> I am happy to but it will have to wait until Tuesday as I am away for the
> weekend.

Sounds good. Have fun!

Reply via email to