Issue 6208 in pharo: Directly browse single matched methods on implementors / senders search

Status: Accepted
Owner: ----
Labels: Milestone-2.0 Type-Feature

New issue 6208 by [hidden email]: Directly browse single matched  
methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

Current when doing an implementors or senders search a method list is shown  
even if there is only one matched method.

Instead we should directly browse!


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

Comment #1 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

I don't like that actually. Because opening a browser can be *slow*, while  
the the list is *fast*.

I am always annoyed when a browser is opened for a single sender.


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

Comment #2 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

I am with Mr. Bruni.

If there is only one choice it is really silly to present a pick list with  
one item.

Now, opening a browser should be fast ;-)

Nautilus is really cool, but sometimes it could open faster, I guess that  
is a price that we pay for all those features.


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

Comment #3 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

@Cami: +1 for opening a browser eventually, but maybe we leave this issue  
for now and let the pick list be a workaround for Nautilus being slow until  
Benjamin can address the situation? It'd be silly for us devs to suffer  
from a slowdown, and e have quite a bit of time until the 2.0 release. This  
is not critical, requiring an immediate solution...


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

Comment #4 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

Opening the browser might be slow, however consider:
- using the mouse to click is slow too (human interaction response delay  
lies around 500ms)
- the current message list does not select anything
- Nautilus is mostly slow on our beast classes Morphic and Object
- Nautilus caches most stuff so a second open is fast enough
- Most methods on these classes (specially Object) are reimplemented  
elsewhere => multiple matches for implementors / senders => open list

all in all it's a win.
Furthermore unless we use Nautilus heavily there is no need to push  
optimization. If we don't push optimization we won't like Nautilus, deadend.

besides:
we already that feature in 1.4 and it got accidentally overridden!




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

Updates:
        Status: FixReviewNeeded

Comment #5 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

slice in the inbox


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

Comment #6 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

Slice in the inbox

SLICE-Issue-6208-Directly-browse-single-matched-methods-on-implementors--senders-search-CamilloBruni.1


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

Comment #7 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

But of course I can just add an option to the settings and people can  
deactivate this behavior


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

Comment #8 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

To please everybody, that would be a great option.

Now we can start argueing what should be the default ;-)


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

Updates:
        Status: MonkeyIsChecking

Comment #9 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208#c9

The Monkey is currently checking this issue. Please don't change it!


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

Updates:
        Status: FixToInclude

Comment #10 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208#c10

This Issue has been checked by Ulysse the Monkey
       
5193 tests passed in 00:00:10s:
===============================
        CollectionsTests-Arrayed (562)
        CollectionsTests-Atomic (12)
        CollectionsTests-Sequenceable (907)
        CollectionsTests-SplitJoin (27)
        CollectionsTests-Stack (16)
        CollectionsTests-Streams (37)
        CollectionsTests-Strings (592)
        CollectionsTests-Support (12)
        CollectionsTests-Text (45)
        CollectionsTests-Unordered (1951)
        CollectionsTests-Weak (739)
        CompilerTests (179)
        SUnit-Core-Extensions (3)
        SUnit-Core-Utilities (3)
        SUnit-Tests-Core (78)
        ToolsTest-Base (2)
        ToolsTest-Debugger (10)
        ToolsTest-Finder (9)
        ToolsTest-Inspector (1)
        ToolsTest-MessageTally (5)
        ToolsTest-PointerFinder (3)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-6208-Directly-browse-single-matched-methods-on-implementors--senders-search-CamilloBruni.1  
from http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20172-a on CoInterpreter  
VMMaker-oscog-EstebanLorenzano.160 uuid:  
bec8cdf0-4e06-4975-8c02-e882fadf4df3 Jun 22 2012,  
StackToRegisterMappingCogit VMMaker-oscog-EstebanLorenzano.160 uuid:  
bec8cdf0-4e06-4975-8c02-e882fadf4df3 Jun 22 2012,  
https://git.gitorious.org/cogvm/blessed.git Commit:  
744bfe905c78a1a5d408680a8780367ea77e0549 Date: Fri Jun 1 15:17:41 2012  
+0200 By: Esteban Lorenzano <[hidden email]>


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

Updates:
        Status: Integrated

Comment #11 on issue 6208 by [hidden email]: Directly browse single  
matched methods on implementors / senders search
http://code.google.com/p/pharo/issues/detail?id=6208

in 2.0 173


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