On Tue, Sep 14, 2010 at 12:04 PM, Alexandre Bergel <alexandre@bergel.eu>wrote:
Hi Mariano,
Before I propose the test integration in system browser a few years ago, people used a lot comments like "self debug: #testFoo". Adding the command cmd-T greatly reduced these kind of comment.
Probably a kind of "send" like method will help reduce these kind of comment.
As Lukas said, I personally often select the method body. But it happens that the method body is larger than the code pane, which requires too much time to select.
At least a cmd+a and then a cmd+d anyway, I wasn't ware of cmd+d :) Maybe adding that to the menu in the case of class side methods? These will help to remove those kind of comments
Cheers, Alexandre
On 14 Sep 2010, at 11:38, Mariano Martinez Peck wrote:
HI guys. I tend to see these pattern everywhere:
openLoadAndQueryWorkspace "self openLoadAndQueryWorkspace" (Workspace new acceptContents: ' â¦â¦
Having to have that kind of thing from my point of view, reveals a weakness in our IDE. Furthermore, when we rename the method, the comment is not refactored.
Now I was thinking if it would be interesting or not, to be able to directly run class side methods. It would be nice:
- To have an entry in the context (right click) menu, only when selecting a class method, to directly evaluate it. Maybe we can call it "Evaluate it" or "Send message" or "Do it" ⦠I don't know.
- To have a shortcut for that. Thenâ¦we are inside that method for example, and with a shortcut we can evaluate itâ¦no need to go up to the name and evaluate the comment.
What do you think?
Cheers _______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
-- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project