Feedback randomly generated :)

- sixxContentOn: aStream indent: level context: dictionary ??
Do we want to rely on SIXX? No!

- Please add one comment per class!

- run SmallLint
        
        initialize categoryContainer ifNil: [ categoryContainer := 
KeymapManager default ]
        => should do a super initialize

- move deprecated method to 'deprecated' category
        KMBuilder

- Cant you remove KMCmd and KMMeta
Looks to me that you could do the same with a sharedPool or moving the class 
methods somewhere else.

- can you have morph/text specific bindings?


I commited some simple changes
Stef

Reply via email to