Thorsten Haude wrote:
> One tiny thing: I think it might be more useful to add the separator
> over an item, not under it.

I bend my brain but still it looks symmetric to me. Could you explain?
 
> >There seems also to be some special SHORT/FULL options for SGI, which
> >I can't test.
> 
> I hear you. The short/simple menu idea is not bad, but why SGI?

This is in menu.c

  /* Menu modes for SGI_CUSTOM short-menus feature */
  enum menuModes {FULL, SHORT};

and then code like

  #ifdef SGI_CUSTOM
  static void shortMenusCB(Widget w, WindowInfo *window, caddr_t callData);
  ...
  #endif
  ...
  #ifdef SGI_CUSTOM
      /*
      ** Short menu mode is a special feature for the SGI system
      ** distribution version of NEdit.
      ....


  #ifdef SGI_CUSTOM
      createMenuSeparator(menuPane, "sep4", SHORT);
      ....

I don't know anything about SGI, and I've never seen these 'short
menus', so I can't even tell whether or not they are really SGI
specific, which I only concluded from the comments in the source code.

Anyway, my patch removes these short menus.  They were not used
for the macro menu, as far as I can tell.

Jörg
--
NEdit Develop mailing list - [email protected]
http://www.nedit.org/mailman/listinfo/develop

Reply via email to