[Cuis-dev] Add delete action extension to PluggableListMorph

[Juan Vuletich]

Very nice contribution, thank you. Integrating it now.

This makes me wonder if it makes sense to have a deleteIndexSelector 
instance variable in PluggableListMorph, and handle it like we do with, 
for example, setIndexSelector. Or maybe, as another option, to use 
keystrokeActionSelector for that. Just saying, in case anyone needs 
this, or wants to do it.


Cheers,

-- 
Juan Vuletich
www.cuis-smalltalk.org
https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev
https://github.com/jvuletich
https://www.linkedin.com/in/juan-vuletich-75611b3
@JuanVuletich


On 8/16/2019 1:01 PM, Josef Philip Bernhart via Cuis-dev wrote:
> Hello,
>
> I made an extension to pluggable list to support the common behaviour 
> of triggering a delete action
> (e.g delete the currently selected item), when backspace or delete key 
> is pressed.
>  In addition to that I added a comment to #isBackspace and added 
> #isDelete to the keyboard event.
>
> I added that, because I'm currently playing around with a "database 
> browser" and wanted to provide
> the user besides a context menu also keyboard bindings for deleting an 
> entry in the list.
>
> By default the PluggableList flashes, as it's currently the case as 
> this is triggered by the lists
> keyboard based "search" functionality which calls flash to flash the 
> morph, when no search
> result could be found.
>
> Thanks for your time,
> Philip