On Apr 24, 2013, at 1:18 AM, Gisela Decuzzi <giseladecu...@gmail.com> wrote:
> Hello! After some work I think that Smart Suggestions it's ready to have some
> real feedback to continue growing up :)
>
> What is it?
> The idea it's to have suggestions based in the text selected or the cursor
> position.
> Actually there are suggestions for:
> - Temporary/Instance/Class Variable
> - Class
> - Method (when you are in the selector)
> - Source (multiple lines)
> - Message
>
Nice!
>
> If you have some comments I will be very happy to heard about, I have removed
> the actions form nautilus menu, because it was to much visual noise (at least
> for me) if you want to add it, only uncoment the pragma in:
> #SugsMenuBuilder>>addSuggestionToSourceCodeMenu:
>
Maybe we can make a destination between two cases:
1) cursor is placed somewhere. Here people are interested in suggestions. The
context menu like it is now makes sense
(I would add one entry for the suggestions, in addition to the shortcut)
2) The user *selected* something explicitly. Here I think we should show a
modified context menu that only has those things
the make sense on the selection. Nobody want to "Debug it" a variable, or
"print it" a syntactically invalid selection...
> ps: It could appear a DNU when fetching the suggestions for a class variable
> the description and solution are in
> https://pharo.fogbugz.com/default.asp?10380#75204
I am tempted to deprecate #definesMethod: and just rename it to canUnderstand:
(and fix all senders)
This way the RB model for code would be one step closer to normal classes and
methods.
The real plan of course is to unify *all* meta models for code and just use
Ring.
The RBMethod/RBClass hierarchy is one of the candidates for that, PseudoClass
and
PseudoMethod the other. (I am sure there are eve more).
Marcus
