On Wed, Aug 07, 2019 at 05:26:51PM -0500, Chris Muller wrote:
> +1 for adding #openHelpBrowser, but -1 for overloading #help.  Please don't.
> 

Point well taken. I put #openHelpBrowser into trunk and moved my inbox
submission to treated.


> #openHelpBrowser is perfectly descriptive.  JavaDoc was never very useful,
> we don't need to try to save typing for this feature.  If you
> disagree, please consider integrating somewhere into the IDE (class menu?)
> but still use #openHelpBrowser.  The core library should not poach #help
> from the namespace.
> 

Interestingly and somewhat ironically, the current implementers of #help
do nothing of direct interest to the hypothetical DynaBook user. They might
better be renamed to e.g. #helpString.

If the selector #help were to be implemented in some way that might be
directly meaningful to users, then what should it do?

Dave



> Best,
>   Chris
> 
> 
> 
> On Tue, Aug 6, 2019 at 7:02 PM <commits at source.squeak.org> wrote:
> 
> > David T. Lewis uploaded a new version of HelpSystem-Core to project The
> > Inbox:
> > http://source.squeak.org/inbox/HelpSystem-Core-dtl.110.mcz
> >
> > ==================== Summary ====================
> >
> > Name: HelpSystem-Core-dtl.110
> > Author: dtl
> > Time: 6 August 2019, 8:02:02.4221 pm
> > UUID: 1d5757f7-f282-4072-bee9-cacb63c3f24d
> > Ancestors: HelpSystem-Core-kks.109
> >
> > Let any Behavior open a help browser on itself.
> >
> > =============== Diff against HelpSystem-Core-kks.109 ===============
> >
> > Item was added:
> > + ----- Method: Behavior>>help (in category '*HelpSystem-Core') -----
> > + help
> > +       "Open a help browser. For classes that implement #help for some
> > other purpose,
> > +       use #openHelpBrowser instead."
> > +
> > +       "Array help"
> > +       "ObjectTracer help"
> > +       "SMLoader openHelpBrowser"
> > +
> > +       self openHelpBrowser!
> >
> > Item was added:
> > + ----- Method: Behavior>>openHelpBrowser (in category '*HelpSystem-Core')
> > -----
> > + openHelpBrowser
> > +       HelpBrowser openOn: self asHelpTopic!
> >
> >
> >

>