Re: Text Mode Menu

[John Anthony <john@jo.hnanthony.com>]

On Mon, Oct 07, 2013 at 10:22:21AM -0700, Drew Adams wrote:
> 
> > > Two minor suggestions -
> > >
> > > > +      '(menu-item "Text Mode Auto Fill"
> > >
> > > `Auto Fill'         - toggle button shows that it is modal
> > >
> > > > +      '(menu-item "Paragraph Indent Minor Mode"
> > >
> > > `Indent Paragraphs' - toggle button shows that it is modal
> > 
> > Could you clarify this at all? Do you just mean clearer labels or
> > could you give an example of a menu item that shows that the option is
> > modal?
> 
> Yes, suggestions for the labels.
> 
> Your code (which I did not try) indicates that you use toggle buttons: 
> :button (:toggle . (memq 'turn-on-auto-fill text-mode-hook))
> 
> They should thus appear similar to what we see in the Options menu.
> You can tell by looking at the menu item and its checkmark (or lack
> thereof) that the item represents something with two states ("modes"),
> and you can tell what the current state is.
> 
> This means that you need not add words in the menu item label to say
> that the thing is modal: "... Minor Mode".  (Yes, one cannot tell from
> the toggle button alone that the mode is minor, but that is only minor
> info. ;-))

Yes, I can see what you mean now. I'll be honest; I find the menu
implementations of all the major and minor modes to be frankly
horribly inconsistant. I suspect this is something I should get used
to.

--
John Anthony