morph view hierarchy explorer

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

morph view hierarchy explorer

Steve Wirts
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.

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

ViewHierarchyExplorer.st (5K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: morph view hierarchy explorer

Stéphane Ducasse
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
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: morph view hierarchy explorer

Steve Wirts
ok,
I'll fix these things
thanks for your input


On Fri, May 22, 2009 at 11:27 AM, Stéphane Ducasse <[hidden email]> 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
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: morph view hierarchy explorer

Mariano Martinez Peck
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 <[hidden email]>
ok,
I'll fix these things
thanks for your input



On Fri, May 22, 2009 at 11:27 AM, Stéphane Ducasse <[hidden email]> 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
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: morph view hierarchy explorer

Steve Wirts
In reply to this post by Stéphane Ducasse
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 <[hidden email]> 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
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

ViewHierarchyExplorer.st (6K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: morph view hierarchy explorer

Stéphane Ducasse
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 <[hidden email]
> > 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
> > [hidden email]
> > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
> <
> ViewHierarchyExplorer
> .st>_______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: morph view hierarchy explorer

Steve Wirts
it's there now.
<img src="http://www.gstatic.com/codesite/ph/images/star_on.gif" style="cursor: pointer;" id="star" title="Vote for this issue and get email change notifications" onclick="_toggleStar(this,&#39;pharo&#39;,840); _syncStarIcons(this, &#39;star2&#39;)" width="15" height="15"> Issue 840: MorphViewHierarchyBrowser code

1 person starred this issue and may be notified of changes.




On Fri, May 22, 2009 at 2:15 PM, Stéphane Ducasse <[hidden email]> 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 <[hidden email]
> > 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
> > [hidden email]
> > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
> <
> ViewHierarchyExplorer
> .st>_______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project