Issue 3954 in pharo: SystemNavigation use BrowserEnvironment as Refactoring and Nautilus

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

Issue 3954 in pharo: SystemNavigation use BrowserEnvironment as Refactoring and Nautilus

pharo
Status: FixProposed
Owner: [hidden email]
Labels: Nautilus

New issue 3954 by [hidden email]: SystemNavigation use  
BrowserEnvironment as Refactoring and Nautilus
http://code.google.com/p/pharo/issues/detail?id=3954

When a new environment is set, it just ask the BrowserEnvironment for  
classes, and select corresponding bindings

Attachments:
        SystemNavigationUseBrowserEnvironment.2.cs  4.4 KB


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3954 in pharo: SystemNavigation use BrowserEnvironment as Refactoring and Nautilus

pharo

Comment #1 on issue 3954 by [hidden email]: SystemNavigation use  
BrowserEnvironment as Refactoring and Nautilus
http://code.google.com/p/pharo/issues/detail?id=3954

I do not like your change :). We cannot do that because we should bind  
SystemNavigation with BrowserEnvironment.
What should be done is that when the SystemNavigation is used we should  
pass an enviroment
when you need it in Nautilus just use on: BrowserEnvironment new.
Does it make sense?


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3954 in pharo: SystemNavigation use BrowserEnvironment as Refactoring and Nautilus

pharo

Comment #2 on issue 3954 by [hidden email]: SystemNavigation use  
BrowserEnvironment as Refactoring and Nautilus
http://code.google.com/p/pharo/issues/detail?id=3954

OK, let's move it into Nautilus


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3954 in pharo: SystemNavigation use BrowserEnvironment as Refactoring and Nautilus

pharo
Updates:
        Status: WontFix

Comment #3 on issue 3954 by [hidden email]: SystemNavigation use  
BrowserEnvironment as Refactoring and Nautilus
http://code.google.com/p/pharo/issues/detail?id=3954

(No comment was entered for this change.)