Issue 5358 in pharo: FOr extension methods, only show in black my own

Status: Accepted
Owner: marianopeck
CC: [hidden email]
Labels: Type-Nautilus

New issue 5358 by marianopeck: FOr extension methods, only show in black my  
own
http://code.google.com/p/pharo/issues/detail?id=5358

When you are showing extensions to a class, I would not show the whole  
class, but intead just the extensions from the package I am browsing. Look  
attached NotShowFullClass.

Ben said:

First I thought about that too, but then I figured out that when you are  
writing an extension, it's really useful to be able to know the whole set  
of available methods in a glance :)


I answered:

Ok. I found it confusing at the beginning, but ok. Anyway, in that case,  
following to what happens in Classes, the protocolos which are not mine  
should be in gray and MINE in black. In my example, I mean to let black  
*Ast-Core and put gray to all the rest?

Do you agree?


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

Updates:
        Status: Integrated

Comment #1 on issue 5358 by [hidden email]: FOr extension methods,  
only show in black my own
http://code.google.com/p/pharo/issues/detail?id=5358

Done


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

Comment #2 on issue 5358 by marianopeck: FOr extension methods, only show  
in black my own
http://code.google.com/p/pharo/issues/detail?id=5358

Hi Ben. I loaded your last code and now it seems everything is gray, not  
even my own package. For example,if I select the package AST-Core, then I  
have the classes Behavior and CompiledMethod. If I select CompiledMethod,  
all protocols are gray, even *ast-core. Maybe you should change something  
to be case insensitive?

Thanks!


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

Comment #3 on issue 5358 by [hidden email]: FOr extension methods,  
only show in black my own
http://code.google.com/p/pharo/issues/detail?id=5358

You are right

It should be fixed now :)

Thanks for your feedback ;)


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

Comment #4 on issue 5358 by marianopeck: FOr extension methods, only show  
in black my own
http://code.google.com/p/pharo/issues/detail?id=5358

Excellent. It is fixed now. I don't want to be annoying but I was just  
thinking the following: when you are in a package XXX that has extensions  
to class YYY, and you browse XXX... when you see the class YYY, we only  
show the protocol *XXX as blck and all the rest in gray. I was thinking  
that, at least in my case, 90% of the times I want to see the *XXX  
category, that means, the black one.  So...what about sorting first those  
protocols? I know it is sroted alphabetically, but maybe we can do  2 sorts  
alphabetically, one for our own packages first (*XXX) and all the rest then.

do you undertand what I mean?


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

Comment #5 on issue 5358 by [hidden email]: FOr extension methods,  
only show in black my own
http://code.google.com/p/pharo/issues/detail?id=5358

yep it can be cool :)

I will have a look at this :)


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

Updates:
        Status: Accepted

Comment #6 on issue 5358 by marianopeck: FOr extension methods, only show  
in black my own
http://code.google.com/p/pharo/issues/detail?id=5358

(No comment was entered for this change.)


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

Updates:
        Summary: For extension methods, only show in black my own
        Status: Integrated

Comment #7 on issue 5358 by [hidden email]: For extension methods,  
only show in black my own
http://code.google.com/p/pharo/issues/detail?id=5358

Done


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

Comment #8 on issue 5358 by marianopeck: For extension methods, only show  
in black my own
http://code.google.com/p/pharo/issues/detail?id=5358

Tested and works! Incredible cool :)
Thanks a lot Ben


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