Re: [PATCH] Cosmetic enhancement to old-whitespace-mode
* Nelson Ferreira <nelson.ferreira(a)ieee.org> writes:
The old-whitespace-mode when old-whitespace-install-submenu is set
to
t installs it under Apps for which it becomes the only submenu, at
least in my install.
Yep, there's been no "Apps" menu since 21.1.
In order to allow the user greater control on where to place that
submenu I devised this patch.
Let me know if this is the wrong forum and if I should send it to
someone in particular.
Send your patches to xemacs-patches(a)xemacs.org.
The maintainer for the text-tools package is listed as
xemacs-bet(a)xemacs.org so I am emailing here.
Yep, it's an orphan. Wanna adopt it?
Index: whitespace-mode.el
===================================================================
RCS file:
/pack/xemacscvs/XEmacs/packages/xemacs-packages/text-modes/whitespace-mode.el,v
retrieving revision 1.6
diff -u -u -b -r1.6 whitespace-mode.el
--- whitespace-mode.el 2004/11/08 02:21:00 1.6
+++ whitespace-mode.el 2004/12/14 04:19:23
＠＠ -504,6 +504,10 ＠＠
(defvar old-whitespace-menu nil
"A menu for the whitespace minor mode.")
+(defvar old-whitespace-menu-name "Apps"
+ "The menu where the whitespace minor mode menu is installed.")
+
+
(setq old-whitespace-menu
'("Whitespace Menu"
["Highlight Whitespaces"
＠＠ -533,7 +537,7 ＠＠
))
(if (and (boundp 'old-whitespace-install-submenu)
old-whitespace-install-submenu)
- (add-submenu '("Apps") old-whitespace-menu))
+ (add-submenu (list old-whitespace-menu-name) old-whitespace-menu))
;;; Toolbar icon for the XEmacs
I don't think this will fly. If I were doing it, I'd put it in the
"Tools" menu. But... I know that Ben is trying to convince packages to
_not_ use the "Tools" menu.
Ben, where should stuff like this go? It seems too insignificant to be
in a menu all to its own.
--
|---<Steve Youngs>---------------<GnuPG KeyID: A94B3003>---|
| XEmacs - The only _______ you'll ever need. |
| Fill in the blank, yes, it's THAT good! |
|----------------------------------<steve(a)youngs.au.com>---|
Attachments:
- attachment.sig (application/pgp-signature — 256 bytes)