Hi hernan

some feedback/questions while waiting for my kids...

=============
"protocol: *SpecUIAddOns"
includesNameBeginsWith: aString
    " Answer whether the receiver begins with aString. "

    ^ self asLowercase = aString first asLowercase


does not look like doing what it says.


=====================================
I do not get

specModalParent
    " Receiver cannot be modal parent, answer a default (morphic world) "

    ^ [ self currentWorld ]
    on: Error
    do: [ : ex | UIManager default currentWorld ]


=====================================

In searchableTree

either use tree or self tree

"protocol: *SpecUIAddOns"
autoMultiSelection: aBoolean

    self tree autoMultiSelection: aBoolean



It was interested to see that you added a lot of methods

methods
    self xxxmodel methods

I was against them.
Now the point is

if we ban such idiom

remove: anItem
    self listModel remove: anItem

then (beside exposing the model which would be ok for me if we consider it as a port)

we have the problem of encapsulating more complex behavior
such

removeAllItems

    self listModel removeAll.
    self updateList

So probably that

remove: anItem
    self listModel remove: anItem

is better


Stef



Reply via email to