[Pharo-project] morph view hierarchy explorer
I've enhanced a few things to make use of existing tools as per an early thread. Is there any chance this simple enhancement can get integrated into the main stream of developement? It really is quite usefull in understanding a complex morph composition.
Hi steven at the minimum you should comment the class and have some examples methods that illustrate its purpose. ;) and it would be to be able to invoke it from the inspector/explorer? Then it would be good that the comments tell the user a little scenario so that we can learn and improve from it so far I had to read the cs file to see that you changed the debug menu and But for example I do not see how I can get one particular object to flash On May 22, 2009, at 5:14 PM, Steve Wirts wrote:
I've enhanced a few things to make use of existing tools as per an early thread.
Is there any chance this simple enhancement can get integrated into the main stream of developement? It really is quite usefull in understanding a complex morph composition. < ViewHierarchyExplorer .st>_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
ok, I'll fix these things thanks for your input On Fri, May 22, 2009 at 11:27 AM, Stéphane Ducasse < stephane.ducasse@inria.fr> wrote:
Hi steven
at the minimum you should comment the class and have some examples methods that illustrate its purpose. ;) and it would be to be able to invoke it from the inspector/explorer? Then it would be good that the comments tell the user a little scenario so that we can learn and improve from it so far I had to read the cs file to see that you changed the debug menu and But for example I do not see how I can get one particular object to flash
On May 22, 2009, at 5:14 PM, Steve Wirts wrote:
I've enhanced a few things to make use of existing tools as per an early thread.
Is there any chance this simple enhancement can get integrated into the main stream of developement? It really is quite usefull in understanding a complex morph composition. < ViewHierarchyExplorer .st>_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
I think than more than examples, what is better is to have unit tests. They have a lot of advantages and they even work as examples. Cheers, Mariano 2009/5/22 Steve Wirts <stevewirts@gmail.com>
ok, I'll fix these things thanks for your input
On Fri, May 22, 2009 at 11:27 AM, Stéphane Ducasse < stephane.ducasse@inria.fr> wrote:
Hi steven
at the minimum you should comment the class and have some examples methods that illustrate its purpose. ;) and it would be to be able to invoke it from the inspector/explorer? Then it would be good that the comments tell the user a little scenario so that we can learn and improve from it so far I had to read the cs file to see that you changed the debug menu and But for example I do not see how I can get one particular object to flash
On May 22, 2009, at 5:14 PM, Steve Wirts wrote:
I've enhanced a few things to make use of existing tools as per an early thread.
Is there any chance this simple enhancement can get integrated into the main stream of developement? It really is quite usefull in understanding a complex morph composition. < ViewHierarchyExplorer .st>_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Stef, I've renamed a few methods for clarity, utilized Morph>>flashBounds method for flashing, added a class comment, and 2 examples. is it at least a C- instead of D+ now? :) let me know what else I should do? On Fri, May 22, 2009 at 11:27 AM, Stéphane Ducasse < stephane.ducasse@inria.fr> wrote:
Hi steven
at the minimum you should comment the class and have some examples methods that illustrate its purpose. ;) and it would be to be able to invoke it from the inspector/explorer? Then it would be good that the comments tell the user a little scenario so that we can learn and improve from it so far I had to read the cs file to see that you changed the debug menu and But for example I do not see how I can get one particular object to flash
On May 22, 2009, at 5:14 PM, Steve Wirts wrote:
I've enhanced a few things to make use of existing tools as per an early thread.
Is there any chance this simple enhancement can get integrated into the main stream of developement? It really is quite usefull in understanding a complex morph composition. < ViewHierarchyExplorer .st>_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Good! :) But really one day we will have to fix all the classes without comments. can you please add it to the bugtracker. I will harvest it and put it in the same package as the inspector. Stef On May 22, 2009, at 6:44 PM, Steve Wirts wrote:
Stef,
I've renamed a few methods for clarity, utilized Morph>>flashBounds method for flashing, added a class comment, and 2 examples. is it at least a C- instead of D+ now? :)
let me know what else I should do?
On Fri, May 22, 2009 at 11:27 AM, Stéphane Ducasse <stephane.ducasse@inria.fr
wrote: Hi steven
at the minimum you should comment the class and have some examples methods that illustrate its purpose. ;) and it would be to be able to invoke it from the inspector/explorer? Then it would be good that the comments tell the user a little scenario so that we can learn and improve from it so far I had to read the cs file to see that you changed the debug menu and But for example I do not see how I can get one particular object to flash
On May 22, 2009, at 5:14 PM, Steve Wirts wrote:
I've enhanced a few things to make use of existing tools as per an early thread.
Is there any chance this simple enhancement can get integrated into the main stream of developement? It really is quite usefull in understanding a complex morph composition. < ViewHierarchyExplorer .st>_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
< ViewHierarchyExplorer .st>_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
it's there now. Issue 840 <http://code.google.com/p/pharo/issues/detail?id=840>: MorphViewHierarchyBrowser code ⹠Prev<http://code.google.com/p/pharo/issues/detail?can=6&colspec=ID%20Type%20Status%20Summary%20Milestone&id=839>93 of 93 1 person starred this issue and may be notified of changes. On Fri, May 22, 2009 at 2:15 PM, Stéphane Ducasse <stephane.ducasse@inria.fr
wrote:
Good! :) But really one day we will have to fix all the classes without comments.
can you please add it to the bugtracker. I will harvest it and put it in the same package as the inspector.
Stef
On May 22, 2009, at 6:44 PM, Steve Wirts wrote:
Stef,
I've renamed a few methods for clarity, utilized Morph>>flashBounds method for flashing, added a class comment, and 2 examples. is it at least a C- instead of D+ now? :)
let me know what else I should do?
On Fri, May 22, 2009 at 11:27 AM, Stéphane Ducasse < stephane.ducasse@inria.fr
wrote: Hi steven
at the minimum you should comment the class and have some examples methods that illustrate its purpose. ;) and it would be to be able to invoke it from the inspector/explorer? Then it would be good that the comments tell the user a little scenario so that we can learn and improve from it so far I had to read the cs file to see that you changed the debug menu and But for example I do not see how I can get one particular object to flash
On May 22, 2009, at 5:14 PM, Steve Wirts wrote:
I've enhanced a few things to make use of existing tools as per an early thread.
Is there any chance this simple enhancement can get integrated into the main stream of developement? It really is quite usefull in understanding a complex morph composition. < ViewHierarchyExplorer .st>_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
< ViewHierarchyExplorer .st>_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
participants (3)
-
Mariano Martinez Peck -
Steve Wirts -
Stéphane Ducasse