>>>> 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!