<div dir="ltr">I think we can remove 'browse class' from both Inspector and ObjectExplorer if they just do the same thing as 'browse full'<div><br></div><div>And we should probably stick with one way of activating features like this. </div><div>I got really confused when I looked at the different approaches. </div><div><br></div><div>Best,</div><div>Karl</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 10, 2018 at 7:26 PM, tim Rowledge <span dir="ltr"><<a href="mailto:tim@rowledge.org" target="_blank">tim@rowledge.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If you look in the code for ObjectExplorer (which is not a terribly pleasant experience) you can find implementations for #browseFull, #classHierarchy & #browseFullProtocol but not #browseClass <br>
<br>
Just within the three methods actually there we have three totally different ways of opening a new browser; via ToolSet, via systemNavigation and by direct testing of Smalltalk isMorphic. Sigh. Nice thing to clean up for a simple job.<br>
<br>
The practical problem is that ToolSet appears to have no access to make a single-class browser (which we do have, Browser newOnClass: works perfectly well) and in any case ToolSet goes via yet another damn AppRegistry (SystemBrowser). So we'd have to add methods to ObjectExplorer, ToolSet, SystemBrowser and probably another dozen currently hidden places. But in principle, implement ObjectExplorer>browseClass and ToolSet><wbr>browseSomethingOrOtherButNotCl<wbr>assBecauseThatIsAlreadyMisused<wbr>ForBrowseFull and SystemBrowser>somethingOrOther<br>
<br>
Or delete it from the ObejctExplorer menu.<br>
<br>
tim<br>
--<br>
tim Rowledge; <a href="mailto:tim@rowledge.org">tim@rowledge.org</a>; <a href="http://www.rowledge.org/tim" rel="noreferrer" target="_blank">http://www.rowledge.org/tim</a><br>
Useful random insult:- Overdue for deincarnation.<br>
<br>
<br>
<br>
</blockquote></div><br></div>