* startup.el (command-line): Call menu-bar-mode with 1 instead of t.
[bpt/emacs.git] / lisp / menu-bar.el
index ab390f4..596241c 100644 (file)
@@ -1,10 +1,10 @@
-;;; menu-bar.el --- define a default menu bar.
+;;; menu-bar.el --- define a default menu bar
 
-;; Copyright (C) 1993, 1994, 1995 Free Software Foundation, Inc.
+;; Copyright (C) 1993, 1994, 1995, 2000, 2001, 2002 Free Software Foundation, Inc.
 
 ;; Author: RMS
 ;; Maintainer: FSF
-;; Keywords: internal
+;; Keywords: internal, mouse
 
 ;; This file is part of GNU Emacs.
 
@@ -25,6 +25,8 @@
 
 ;; Avishai Yacobi suggested some menu rearrangements.
 
+;;; Commentary:
+
 ;;; Code:
 
 ;;; User options:
@@ -50,127 +52,182 @@ A large number or nil slows down menu responsiveness."
 (setq menu-bar-final-items '(help-menu))
 
 (define-key global-map [menu-bar help-menu] (cons "Help" menu-bar-help-menu))
-(defvar menu-bar-search-menu (make-sparse-keymap "Search"))
-(define-key global-map [menu-bar search] (cons "Search" menu-bar-search-menu))
-(defvar menu-bar-edit-menu (make-sparse-keymap "Edit"))
-(define-key global-map [menu-bar edit] (cons "Edit" menu-bar-edit-menu))
 (defvar menu-bar-tools-menu (make-sparse-keymap "Tools"))
 (define-key global-map [menu-bar tools] (cons "Tools" menu-bar-tools-menu))
-(defvar menu-bar-files-menu (make-sparse-keymap "Files"))
-(define-key global-map [menu-bar files] (cons "Files" menu-bar-files-menu))
+;; This definition is just to show what this looks like.
+;; It gets overridden below when menu-bar-update-buffers is called.
+(define-key global-map [menu-bar buffer]
+  (cons "Buffers" (make-sparse-keymap "Buffers")))
+(defvar menu-bar-options-menu (make-sparse-keymap "Options"))
+(define-key global-map [menu-bar options]
+  (cons "Options" menu-bar-options-menu))
+(defvar menu-bar-edit-menu (make-sparse-keymap "Edit"))
+(define-key global-map [menu-bar edit] (cons "Edit" menu-bar-edit-menu))
+(defvar menu-bar-files-menu (make-sparse-keymap "File"))
+(define-key global-map [menu-bar files] (cons "File" menu-bar-files-menu))
 
 ;; This alias is for compatibility with 19.28 and before.
 (defvar menu-bar-file-menu menu-bar-files-menu)
-\f
-(defvar vc-menu-map (make-sparse-keymap "Version Control"))
-
-(define-key menu-bar-tools-menu [gdb] '("Debugger..." . gdb))
-(define-key menu-bar-tools-menu [compile] '("Compile..." . compile))
-(define-key menu-bar-tools-menu [grep] '("Search Files..." . grep))
-
-(define-key menu-bar-tools-menu [separator-1]
-  '("--"))
-
-(define-key menu-bar-tools-menu [calendar] '("Display Calendar" . calendar))
-(define-key menu-bar-tools-menu [speedbar]
-  '("Display Speedbar" . speedbar-frame-mode))
-(define-key menu-bar-tools-menu [compose-mail] '("Send Mail" . compose-mail))
-(define-key menu-bar-tools-menu [rmail] '("Read Mail" . rmail))
-(define-key menu-bar-tools-menu [gnus] '("Read Net News" . gnus))
-
-(define-key menu-bar-tools-menu [separator-vc]
-  '("--"))
-
-(define-key menu-bar-tools-menu [vc]
-  (cons "Version Control" vc-menu-map))
-
-(define-key menu-bar-tools-menu [separator-compare]
-  '("--"))
-
-(define-key menu-bar-tools-menu [ediff-misc]
-  '("Ediff Miscellanea" . menu-bar-ediff-misc-menu))
-(define-key menu-bar-tools-menu [epatch]
-  '("Apply Patch" . menu-bar-epatch-menu))
-(define-key menu-bar-tools-menu [ediff-merge]
-  '("Merge" . menu-bar-ediff-merge-menu))
-(define-key menu-bar-tools-menu [compare]
-  '("Compare" . menu-bar-ediff-menu))
-
-(define-key menu-bar-tools-menu [separator-print]
-  '("--"))
-
-(defvar menu-bar-print-menu (make-sparse-keymap "Print"))
-
-(define-key menu-bar-print-menu [ps-print-region]
-  '("Postscript Print Region" . ps-print-region-with-faces))
-(define-key menu-bar-print-menu [ps-print-buffer]
-  '("Postscript Print Buffer" . ps-print-buffer-with-faces))
-(define-key menu-bar-print-menu [separator-ps-print]
-  '("--"))
-(define-key menu-bar-print-menu [print-region]
-  '("Print Region" . print-region))
-(define-key menu-bar-print-menu [print-buffer]
-  '("Print Buffer" . print-buffer))
 
-(define-key menu-bar-tools-menu [print]
-  (cons "Print" menu-bar-print-menu))
+;; This is referenced by some code below; it is defined in uniquify.el
+(defvar uniquify-buffer-name-style)
 
-(put 'print-region 'menu-enable 'mark-active)
-(put 'ps-print-region-with-faces 'menu-enable 'mark-active)
 \f
+;; The "File" menu items
 (define-key menu-bar-files-menu [exit-emacs]
-  '("Exit Emacs" . save-buffers-kill-emacs))
+  '(menu-item "Exit Emacs" save-buffers-kill-emacs
+             :help "Save unsaved buffers, then exit"))
 
 (define-key menu-bar-files-menu [separator-exit]
   '("--"))
 
+;; Don't use delete-frame as event name because that is a special
+;; event.
+(define-key menu-bar-files-menu [delete-this-frame]
+  '(menu-item "Delete Frame" delete-frame
+             :visible (fboundp 'delete-frame)
+             :enable (delete-frame-enabled-p)
+             :help "Delete currently selected frame"))
+(define-key menu-bar-files-menu [make-frame-on-display]
+  '(menu-item "New Frame on Display..." make-frame-on-display
+             :visible (fboundp 'make-frame-on-display)
+             :help "Open a new frame on another display"))
+(define-key menu-bar-files-menu [make-frame]
+  '(menu-item "New Frame" make-frame-command
+             :visible (fboundp 'make-frame-command)
+             :help "Open a new frame"))
+
 (define-key menu-bar-files-menu [one-window]
-  '("One Window" . delete-other-windows))
+  '(menu-item "Unsplit Windows" delete-other-windows
+             :enable (not (one-window-p t nil))
+             :help "Make selected window fill its frame"))
 
 (define-key menu-bar-files-menu [split-window]
-  '("Split Window" . split-window-vertically))
-
-(if (fboundp 'delete-frame)
-    (progn
-      ;; Don't use delete-frame as event name
-      ;; because that is a special event.
-      (define-key menu-bar-files-menu [delete-this-frame]
-       '("Delete Frame" . delete-frame))
-      (define-key menu-bar-files-menu [make-frame-on-display]
-       '("Open New Display..." . make-frame-on-display))
-      (define-key menu-bar-files-menu [make-frame]
-       '("Make New Frame" . make-frame-command))))
-
-(define-key menu-bar-files-menu [separator-buffers]
-  '("--"))
+  '(menu-item "Split Window" split-window-vertically
+             :help "Split selected window in two"))
+
+(define-key menu-bar-files-menu [separator-window]
+  '(menu-item "--"))
+
+(define-key menu-bar-files-menu [ps-print-region]
+  '(menu-item "Postscript Print Region (B+W)" ps-print-region
+             :enable mark-active
+             :help "Pretty-print marked region in black and white to PostScript printer"))
+(define-key menu-bar-files-menu [ps-print-buffer]
+  '(menu-item "Postscript Print Buffer (B+W)" ps-print-buffer
+             :help "Pretty-print current buffer in black and white to PostScript printer"))
+(define-key menu-bar-files-menu [ps-print-region-faces]
+  '(menu-item "Postscript Print Region" ps-print-region-with-faces
+             :enable mark-active
+             :help "Pretty-print marked region to PostScript printer"))
+(define-key menu-bar-files-menu [ps-print-buffer-faces]
+  '(menu-item "Postscript Print Buffer" ps-print-buffer-with-faces
+             :help "Pretty-print current buffer to PostScript printer"))
+(define-key menu-bar-files-menu [print-region]
+  '(menu-item "Print Region" print-region
+             :enable mark-active
+             :help "Print region between mark and current position"))
+(define-key menu-bar-files-menu [print-buffer]
+  '(menu-item "Print Buffer" print-buffer
+             :help "Print current buffer with page headings"))
+
+(define-key menu-bar-files-menu [separator-print]
+  '(menu-item "--"))
 
-(define-key menu-bar-files-menu [kill-buffer]
-  '("Kill Current Buffer" . kill-this-buffer))
-(define-key menu-bar-files-menu [insert-file]
-  '("Insert File..." . insert-file))
 (define-key menu-bar-files-menu [recover-session]
-  '("Recover Session..." . recover-session))
+  '(menu-item "Recover Crashed Session..." recover-session
+             :enable (and auto-save-list-file-prefix
+                          (file-directory-p
+                            (file-name-directory auto-save-list-file-prefix))
+                           (directory-files
+                           (file-name-directory auto-save-list-file-prefix)
+                           nil
+                           (concat "\\`"
+                                   (regexp-quote
+                                    (file-name-nondirectory
+                                     auto-save-list-file-prefix)))
+                           t))
+             :help "Recover edits from a crashed session"))
 (define-key menu-bar-files-menu [revert-buffer]
-  '("Revert Buffer" . revert-buffer))
+  '(menu-item "Revert Buffer" revert-buffer
+             :enable (or revert-buffer-function
+                         revert-buffer-insert-file-contents-function
+                         (and buffer-file-number
+                              (or (buffer-modified-p)
+                                  (not (verify-visited-file-modtime
+                                        (current-buffer))))))
+             :help "Re-read current buffer from its file"))
 (define-key menu-bar-files-menu [write-file]
-  '("Save Buffer As..." . write-file))
-(define-key menu-bar-files-menu [save-buffer] '("Save Buffer" . save-buffer))
-(define-key menu-bar-files-menu [dired] '("Open Directory..." . dired))
-(define-key menu-bar-files-menu [open-file] '("Open File..." . find-file))
-
-(put 'recover-session 'menu-enable
-     '(and auto-save-list-file-prefix
-          (directory-files
-           (file-name-directory auto-save-list-file-prefix)
-           nil
-           (concat "\\`"
-                   (regexp-quote (file-name-nondirectory
-                                  auto-save-list-file-prefix)))
-           t)))
+  '(menu-item "Save Buffer As..." write-file
+             :enable (not (window-minibuffer-p
+                           (frame-selected-window menu-updating-frame)))
+             :help "Write current buffer to another file"))
+(define-key menu-bar-files-menu [save-buffer]
+  '(menu-item "Save (current buffer)" save-buffer
+             :enable (and (buffer-modified-p)
+                          (buffer-file-name)
+                          (not (window-minibuffer-p
+                                (frame-selected-window menu-updating-frame))))
+             :help "Save current buffer to its file"))
+
+(define-key menu-bar-files-menu [separator-save]
+  '(menu-item "--"))
+
+(define-key menu-bar-files-menu [kill-buffer]
+  '(menu-item "Close (current buffer)" kill-this-buffer
+             :enable (kill-this-buffer-enabled-p)
+             :help "Discard current buffer"))
+(define-key menu-bar-files-menu [insert-file]
+  '(menu-item "Insert File..." insert-file
+             :enable (not (window-minibuffer-p
+                           (frame-selected-window menu-updating-frame)))
+             :help "Insert another file into current buffer"))
+(define-key menu-bar-files-menu [dired]
+  '(menu-item "Open Directory..." dired
+             :help "Read a directory, operate on its files"))
+(define-key menu-bar-files-menu [open-file]
+  '(menu-item "Open File..." find-file
+             :enable (not (window-minibuffer-p
+                           (frame-selected-window menu-updating-frame)))
+             :help "Read a file into an Emacs buffer"))
+
 \f
+;; The "Edit" menu items
+
+;; The "Edit->Search" submenu
+(defvar menu-bar-last-search-type nil
+  "Type of last non-incremental search command called from the menu.")
+
+(defun nonincremental-repeat-search-forward ()
+  "Search forward for the previous search string or regexp."
+  (interactive)
+  (cond
+   ((and (eq menu-bar-last-search-type 'string)
+        search-ring)
+    (search-forward (car search-ring)))
+   ((and (eq menu-bar-last-search-type 'regexp)
+        regexp-search-ring)
+    (re-search-forward (car regexp-search-ring)))
+   (t
+    (error "No previous search"))))
+
+(defun nonincremental-repeat-search-backward ()
+  "Search backward for the previous search string or regexp."
+  (interactive)
+  (cond
+   ((and (eq menu-bar-last-search-type 'string)
+        search-ring)
+    (search-backward (car search-ring)))
+   ((and (eq menu-bar-last-search-type 'regexp)
+        regexp-search-ring)
+    (re-search-backward (car regexp-search-ring)))
+   (t
+    (error "No previous search"))))
+
 (defun nonincremental-search-forward (string)
   "Read a string and search for it nonincrementally."
   (interactive "sSearch for string: ")
+  (setq menu-bar-last-search-type 'string)
   (if (equal string "")
       (search-forward (car search-ring))
     (isearch-update-ring string nil)
@@ -179,6 +236,7 @@ A large number or nil slows down menu responsiveness."
 (defun nonincremental-search-backward (string)
   "Read a string and search backward for it nonincrementally."
   (interactive "sSearch for string: ")
+  (setq menu-bar-last-search-type 'string)
   (if (equal string "")
       (search-backward (car search-ring))
     (isearch-update-ring string nil)
@@ -187,6 +245,7 @@ A large number or nil slows down menu responsiveness."
 (defun nonincremental-re-search-forward (string)
   "Read a regular expression and search for it nonincrementally."
   (interactive "sSearch for regexp: ")
+  (setq menu-bar-last-search-type 'regexp)
   (if (equal string "")
       (re-search-forward (car regexp-search-ring))
     (isearch-update-ring string t)
@@ -195,89 +254,229 @@ A large number or nil slows down menu responsiveness."
 (defun nonincremental-re-search-backward (string)
   "Read a regular expression and search backward for it nonincrementally."
   (interactive "sSearch for regexp: ")
+  (setq menu-bar-last-search-type 'regexp)
   (if (equal string "")
       (re-search-backward (car regexp-search-ring))
     (isearch-update-ring string t)
     (re-search-backward string)))
 
-(defun nonincremental-repeat-search-forward ()
-  "Search forward for the previous search string."
-  (interactive)
-  (if (null search-ring)
-      (error "No previous search"))
-  (search-forward (car search-ring)))
-
-(defun nonincremental-repeat-search-backward ()
-  "Search backward for the previous search string."
-  (interactive)
-  (if (null search-ring)
-      (error "No previous search"))
-  (search-backward (car search-ring)))
-
-(defun nonincremental-repeat-re-search-forward ()
-  "Search forward for the previous regular expression."
-  (interactive)
-  (if (null regexp-search-ring)
-      (error "No previous search"))
-  (re-search-forward (car regexp-search-ring)))
+(defvar menu-bar-search-menu (make-sparse-keymap "Search"))
 
-(defun nonincremental-repeat-re-search-backward ()
-  "Search backward for the previous regular expression."
-  (interactive)
-  (if (null regexp-search-ring)
-      (error "No previous search"))
-  (re-search-backward (car regexp-search-ring)))
-
-(define-key menu-bar-search-menu [query-replace-regexp]
-  '("Query Replace Regexp..." . query-replace-regexp))
-(define-key menu-bar-search-menu [query-replace]
-  '("Query Replace..." . query-replace))
-(define-key menu-bar-search-menu [find-tag]
-  '("Find Tag..." . find-tag))
-(define-key menu-bar-search-menu [bookmark]
-  '("Bookmarks" . menu-bar-bookmark-map))
-
-(define-key menu-bar-search-menu [separator-search]
-  '("--"))
+;; The Edit->Search->Incremental Search menu
+(defvar menu-bar-i-search-menu
+  (make-sparse-keymap "Incremental Search"))
+
+(define-key menu-bar-i-search-menu [isearch-backward-regexp]
+  '(menu-item "Backward Regexp..." isearch-backward-regexp
+             :help "Search backwards for a regular expression as you type it"))
+(define-key menu-bar-i-search-menu [isearch-forward-regexp]
+  '(menu-item "Forward Regexp..." isearch-forward-regexp
+             :help "Search forward for a regular expression as you type it"))
+(define-key menu-bar-i-search-menu [isearch-backward]
+  '(menu-item "Backward String..." isearch-backward
+             :help "Search backwards for a string as you type it"))
+(define-key menu-bar-i-search-menu [isearch-forward]
+  '(menu-item "Forward String..." isearch-forward
+             :help "Search forward for a string as you type it"))
+
+
+(define-key menu-bar-search-menu [i-search]
+  (list 'menu-item "Incremental Search" menu-bar-i-search-menu
+             :help "Incremental Search finds partial matches while you type the search string.\nIt is most convenient from the keyboard.  Try it!"))
+(define-key menu-bar-search-menu [separator-tag-isearch]
+  '(menu-item "--"))
+
+(define-key menu-bar-search-menu [tags-continue]
+  '(menu-item "Continue Tags Search" tags-loop-continue
+             :help "Continue last tags search operation"))
+(define-key menu-bar-search-menu [tags-srch]
+  '(menu-item "Search tagged files" tags-search
+             :help "Search for a regexp in all tagged files"))
+(define-key menu-bar-search-menu [separator-tag-search]
+  '(menu-item "--"))
 
-(define-key menu-bar-search-menu [repeat-regexp-back]
-  '("Repeat Regexp Backwards" . nonincremental-repeat-re-search-backward))
 (define-key menu-bar-search-menu [repeat-search-back]
-  '("Repeat Backwards" . nonincremental-repeat-search-backward))
-(define-key menu-bar-search-menu [repeat-regexp-fwd]
-  '("Repeat Regexp" . nonincremental-repeat-re-search-forward))
+  '(menu-item "Repeat Backwards" nonincremental-repeat-search-backward
+             :enable (or (and (eq menu-bar-last-search-type 'string)
+                              search-ring)
+                         (and (eq menu-bar-last-search-type 'regexp)
+                              regexp-search-ring))
+             :help "Repeat last search backwards"))
 (define-key menu-bar-search-menu [repeat-search-fwd]
-  '("Repeat Search" . nonincremental-repeat-search-forward))
-
-(define-key menu-bar-search-menu [separator-repeat]
-  '("--"))
+  '(menu-item "Repeat Forward" nonincremental-repeat-search-forward
+             :enable (or (and (eq menu-bar-last-search-type 'string)
+                              search-ring)
+                         (and (eq menu-bar-last-search-type 'regexp)
+                              regexp-search-ring))
+             :help "Repeat last search forward"))
+(define-key menu-bar-search-menu [separator-repeat-search]
+  '(menu-item "--"))
 
 (define-key menu-bar-search-menu [re-search-backward]
-  '("Regexp Search Backwards..." . nonincremental-re-search-backward))
-(define-key menu-bar-search-menu [search-backward]
-  '("Search Backwards..." . nonincremental-search-backward))
+  '(menu-item "Regexp Backwards..." nonincremental-re-search-backward
+             :help "Search backwards for a regular expression"))
 (define-key menu-bar-search-menu [re-search-forward]
-  '("Regexp Search..." . nonincremental-re-search-forward))
-(define-key menu-bar-search-menu [search-forward]
-  '("Search..." . nonincremental-search-forward))
-\f
-(if (fboundp 'start-process)
-    (define-key menu-bar-edit-menu [spell] '("Spell" . ispell-menu-map)))
-(define-key menu-bar-edit-menu [fill] '("Fill" . fill-region))
-(define-key menu-bar-edit-menu [props] '("Text Properties" . facemenu-menu))
-
-(define-key menu-bar-edit-menu [separator-edit]
-  '("--"))
+  '(menu-item "Regexp Forward..." nonincremental-re-search-forward
+             :help "Search forward for a regular expression"))
 
-(define-key menu-bar-edit-menu [clear] '("Clear" . delete-region))
+(define-key menu-bar-search-menu [search-backward]
+  '(menu-item "String Backwards..." nonincremental-search-backward
+             :help "Search backwards for a string"))
+(define-key menu-bar-search-menu [search-forward]
+  '(menu-item "String Forward..." nonincremental-search-forward
+             :help "Search forward for a string"))
+
+;; The Edit->Replace submenu
+
+(defvar menu-bar-replace-menu (make-sparse-keymap "Replace"))
+
+(define-key menu-bar-replace-menu [tags-repl-continue]
+  '(menu-item "Continue Replace" tags-loop-continue
+             :help "Continue last tags replace operation"))
+(define-key menu-bar-replace-menu [tags-repl]
+  '(menu-item "Replace in tagged files" tags-query-replace
+             :help "Interactively replace a regexp in all tagged files"))
+(define-key menu-bar-replace-menu [separator-replace-tags]
+  '(menu-item "--"))
+
+(define-key menu-bar-replace-menu [query-replace-regexp]
+  '(menu-item "Replace Regexp..." query-replace-regexp
+             :enable (not buffer-read-only)
+             :help "Replace regular expression interactively, ask about each occurrence"))
+(define-key menu-bar-replace-menu [query-replace]
+  '(menu-item "Replace String..." query-replace
+             :enable (not buffer-read-only)
+             :help "Replace string interactively, ask about each occurrence"))
+
+;;; Assemble the top-level Edit menu items.
+(define-key menu-bar-edit-menu [props]
+  '(menu-item "Text Properties" facemenu-menu
+             :help "Change properties of text in region"))
+
+(define-key menu-bar-edit-menu [fill]
+  '(menu-item "Fill" fill-region
+             :enable (and mark-active (not buffer-read-only))
+             :help
+             "Fill text in region to fit between left and right margin"))
+
+(define-key menu-bar-edit-menu [separator-bookmark]
+  '(menu-item "--"))
+
+(define-key menu-bar-edit-menu [bookmark]
+  '(menu-item "Bookmarks" menu-bar-bookmark-map
+             :help "Record positions and jump between them"))
+
+(defvar menu-bar-goto-menu (make-sparse-keymap "Go To"))
+
+(define-key menu-bar-goto-menu [set-tags-name]
+  '(menu-item "Set Tags File Name" visit-tags-table
+             :help "Tell Tags commands which tag table file to use"))
+
+(define-key menu-bar-goto-menu [separator-tag-file]
+  '(menu-item "--"))
+
+(define-key menu-bar-goto-menu [apropos-tags]
+  '(menu-item "Tags Apropos" tags-apropos
+             :help "Find function/variables whose names match regexp"))
+(define-key menu-bar-goto-menu [next-tag-otherw]
+  '(menu-item "Next Tag in Other Window"
+             menu-bar-next-tag-other-window
+             :enable (and (boundp 'tags-location-ring)
+                          (not (ring-empty-p tags-location-ring)))
+             :help "Find next function/variable matching last tag name in another window"))
+
+(defun menu-bar-next-tag-other-window ()
+  "Find the next definition of the tag already specified."
+  (interactive)
+  (find-tag-other-window nil t))
 
+(defun menu-bar-next-tag ()
+  "Find the next definition of the tag already specified."
+  (interactive)
+  (find-tag nil t))
+
+(define-key menu-bar-goto-menu [next-tag]
+  '(menu-item "Find Next Tag"
+             menu-bar-next-tag
+             :enable (and (boundp 'tags-location-ring)
+                          (not (ring-empty-p tags-location-ring)))
+             :help "Find next function/variable matching last tag name"))
+(define-key menu-bar-goto-menu [find-tag-otherw]
+  '(menu-item "Find Tag in Other Window..." find-tag-other-window
+             :help "Find function/variable definition in another window"))
+(define-key menu-bar-goto-menu [find-tag]
+  '(menu-item "Find Tag..." find-tag
+             :help "Find definition of function or variable"))
+
+(define-key menu-bar-goto-menu [separator-tags]
+  '(menu-item "--"))
+
+(define-key menu-bar-goto-menu [end-of-buf]
+  '(menu-item "Goto End of Buffer" end-of-buffer))
+(define-key menu-bar-goto-menu [beg-of-buf]
+  '(menu-item "Goto Beginning of Buffer" beginning-of-buffer))
+(define-key menu-bar-goto-menu [go-to-pos]
+  '(menu-item "Goto Buffer Position..." goto-char
+             :help "Read a number N and go to buffer position N"))
+(define-key menu-bar-goto-menu [go-to-line]
+  '(menu-item "Goto Line..." goto-line
+             :help "Read a line number and go to that line"))
+
+(define-key menu-bar-edit-menu [goto]
+  (list 'menu-item "Go To" menu-bar-goto-menu))
+
+(define-key menu-bar-edit-menu [replace]
+  (list 'menu-item "Replace" menu-bar-replace-menu))
+
+(define-key menu-bar-edit-menu [search]
+  (list 'menu-item "Search" menu-bar-search-menu))
+
+(define-key menu-bar-edit-menu [separator-search]
+  '(menu-item "--"))
+
+(define-key menu-bar-edit-menu [mark-whole-buffer]
+  '(menu-item "Select All" mark-whole-buffer
+             :help "Mark the whole buffer for a subsequent cut/copy."))
+(define-key menu-bar-edit-menu [clear]
+  '(menu-item "Clear" delete-region
+             :enable (and mark-active
+                          (not buffer-read-only)
+                          (not (mouse-region-match)))
+             :help
+             "Delete the text in region between mark and current position"))
 (defvar yank-menu (cons "Select Yank" nil))
 (fset 'yank-menu (cons 'keymap yank-menu))
-(define-key menu-bar-edit-menu [select-paste] '("Select and Paste" . yank-menu))
-(define-key menu-bar-edit-menu [paste] '("Paste" . yank))
-(define-key menu-bar-edit-menu [copy] '("Copy" . menu-bar-kill-ring-save))
-(define-key menu-bar-edit-menu [cut] '("Cut" . kill-region))
-(define-key menu-bar-edit-menu [undo] '("Undo" . undo))
+(define-key menu-bar-edit-menu [select-paste]
+  '(menu-item "Select and Paste" yank-menu
+             :enable (and (cdr yank-menu) (not buffer-read-only))
+             :help "Paste (yank) text cut or copied earlier"))
+(define-key menu-bar-edit-menu [paste]
+  '(menu-item "Paste" yank
+             :enable (and
+                      ;; Emacs compiled --without-x doesn't have
+                      ;; x-selection-exists-p.
+                      (fboundp 'x-selection-exists-p)
+                      (x-selection-exists-p) (not buffer-read-only))
+             :help "Paste (yank) text most recently cut/copied"))
+(define-key menu-bar-edit-menu [copy]
+  '(menu-item "Copy" menu-bar-kill-ring-save
+             :enable mark-active
+             :help "Copy text in region between mark and current position"
+             :keys "\\[kill-ring-save]"))
+(define-key menu-bar-edit-menu [cut]
+  '(menu-item "Cut" kill-region
+             :enable (and mark-active (not buffer-read-only))
+             :help
+             "Cut (kill) text in region between mark and current position"))
+(define-key menu-bar-edit-menu [undo]
+  '(menu-item "Undo" undo
+             :enable (and (not buffer-read-only)
+                          (not (eq t buffer-undo-list))
+                          (if (eq last-command 'undo)
+                              pending-undo-list
+                            (consp buffer-undo-list)))
+             :help "Undo last operation"))
+
 
 (defun menu-bar-kill-ring-save (beg end)
   (interactive "r")
@@ -285,21 +484,6 @@ A large number or nil slows down menu responsiveness."
       (message "Selecting a region with the mouse does `copy' automatically")
     (kill-ring-save beg end)))
 
-(put 'fill-region 'menu-enable '(and mark-active (not buffer-read-only)))
-(put 'kill-region 'menu-enable '(and mark-active (not buffer-read-only)))
-(put 'menu-bar-kill-ring-save 'menu-enable 'mark-active)
-(put 'yank 'menu-enable '(and (x-selection-exists-p) (not buffer-read-only)))
-(put 'yank-menu 'menu-enable '(and (cdr yank-menu) (not buffer-read-only)))
-(put 'delete-region 'menu-enable '(and mark-active
-                                      (not buffer-read-only)
-                                      (not (mouse-region-match))))
-(put 'undo 'menu-enable '(and (not buffer-read-only)
-                             (if (eq last-command 'undo)
-                                 pending-undo-list
-                               (consp buffer-undo-list))))
-(put 'query-replace 'menu-enable '(not buffer-read-only))
-(put 'query-replace-regexp 'menu-enable '(not buffer-read-only))
-
 (autoload 'ispell-menu-map "ispell" nil t 'keymap)
 
 ;; These are alternative definitions for the cut, paste and copy
@@ -308,10 +492,11 @@ A large number or nil slows down menu responsiveness."
 (put 'clipboard-kill-region 'menu-enable 'mark-active)
 (put 'clipboard-kill-ring-save 'menu-enable 'mark-active)
 (put 'clipboard-yank 'menu-enable
-     '(or (x-selection-exists-p) (x-selection-exists-p 'CLIPBOARD)))
+     '(or (and (fboundp 'x-selection-exists-p) (x-selection-exists-p))
+         (x-selection-exists-p 'CLIPBOARD)))
 
 (defun clipboard-yank ()
-  "Reinsert the last stretch of killed text, or the clipboard contents."
+  "Insert the clipboard contents, or the last stretch of killed text."
   (interactive)
   (let ((x-select-enable-clipboard t))
     (yank)))
@@ -335,11 +520,13 @@ Do the same for the keys of the same name."
   ;; We can't use constant list structure here because it becomes pure,
   ;; and because it gets modified with cache data.
   (define-key menu-bar-edit-menu [paste]
-    (cons "Paste" 'clipboard-yank))
+    (cons "Paste" (cons "Paste text from clipboard" 'clipboard-yank)))
   (define-key menu-bar-edit-menu [copy]
-    (cons "Copy" 'clipboard-kill-ring-save))
+    (cons "Copy" (cons "Copy text in region to the clipboard"
+                      'clipboard-kill-ring-save)))
   (define-key menu-bar-edit-menu [cut]
-    (cons "Cut" 'clipboard-kill-region))
+    (cons "Cut" (cons "Delete text in region and copy it to the clipboard"
+                     'clipboard-kill-region)))
 
   (define-key global-map [f20] 'clipboard-kill-region)
   (define-key global-map [f16] 'clipboard-kill-ring-save)
@@ -349,167 +536,782 @@ Do the same for the keys of the same name."
   (define-key global-map [copy] 'clipboard-kill-ring-save)
   (define-key global-map [paste] 'clipboard-yank))
 \f
-
-;;; Menu support
+;; The "Options" menu items
 
 (defvar menu-bar-custom-menu (make-sparse-keymap "Customize"))
 
 (define-key menu-bar-custom-menu [customize-apropos-groups]
-  '("Apropos Groups..." . customize-apropos-groups))
+  '(menu-item "Groups Matching Regexp..." customize-apropos-groups
+             :help "Browse groups whose names match regexp"))
 (define-key menu-bar-custom-menu [customize-apropos-faces]
-  '("Apropos Faces..." . customize-apropos-faces))
+  '(menu-item "Faces Matching Regexp..." customize-apropos-faces
+             :help "Browse faces whose names match regexp"))
 (define-key menu-bar-custom-menu [customize-apropos-options]
-  '("Apropos Options..." . customize-apropos-options))
+  '(menu-item "Options Matching Regexp..." customize-apropos-options
+             :help "Browse options whose names match regexp"))
 (define-key menu-bar-custom-menu [customize-apropos]
-  '("Apropos..." . customize-apropos))
-(define-key menu-bar-custom-menu [separator-2]
+  '(menu-item "Settings Matching Regexp..." customize-apropos
+             :help "Browse customizable settings whose names match regexp"))
+(define-key menu-bar-custom-menu [separator-1]
   '("--"))
 (define-key menu-bar-custom-menu [customize-group]
-  '("Specific Group..." . customize-group))
+  '(menu-item "Specific Group..." customize-group
+             :help "Customize settings of specific group"))
 (define-key menu-bar-custom-menu [customize-face]
-  '("Specific Face..." . customize-face))
+  '(menu-item "Specific Face..." customize-face
+             :help "Customize attributes of specific face"))
 (define-key menu-bar-custom-menu [customize-option]
-  '("Specific Option..." . customize-option))
+  '(menu-item "Specific Option..." customize-option
+             :help "Customize value of specific option"))
+(define-key menu-bar-custom-menu [separator-2]
+  '("--"))
 (define-key menu-bar-custom-menu [customize-changed-options]
-  '("Changed Options..." . customize-changed-options))
+  '(menu-item "New Options..." customize-changed-options
+             :help "Options added or changed in recent Emacs versions"))
+(define-key menu-bar-custom-menu [customize-saved]
+  '(menu-item "Saved Options" customize-saved
+             :help "Customize previously saved options"))
 (define-key menu-bar-custom-menu [separator-3]
   '("--"))
 (define-key menu-bar-custom-menu [customize-browse]
-  '("Browse Customization Groups" . customize-browse))
+  '(menu-item "Browse Customization Groups" customize-browse
+             :help "Browse all customization groups"))
 (define-key menu-bar-custom-menu [customize]
-  '("Top-level Customization Group" . customize))
-
-;; Options menu
-(defvar menu-bar-options-menu (make-sparse-keymap "Global Options"))
-
-(defmacro menu-bar-make-toggle (name variable doc message &rest body)
+  '(menu-item "Top-level Customization Group" customize
+             :help "The master group called `Emacs'"))
+
+;(defvar menu-bar-preferences-menu (make-sparse-keymap "Preferences"))
+
+(defmacro menu-bar-make-mm-toggle (fname doc help &optional props)
+  "Make a menu-item for a global minor mode toggle.
+FNAME is the minor mode's name (variable and function).
+DOC is the text to use the menu entry.
+HELP is the text to use for the tooltip.
+PROPS are additional properties."
+  `'(menu-item ,doc ,fname
+     ,@(if props props)
+     :help ,help
+     :button (:toggle . (and (default-boundp ',fname)
+                            (default-value ',fname)))))
+
+(defmacro menu-bar-make-toggle (name variable doc message help &rest body)
   `(progn
      (defun ,name ()
-       ,(concat doc ".")
+       ,(concat "Toggle whether to " (downcase (substring help 0 1))
+               (substring help 1) ".")
        (interactive)
        (if ,(if body `(progn . ,body)
-             `(setq ,variable (not ,variable)))
+             `(progn
+                (custom-load-symbol ',variable)
+                (let ((set (or (get ',variable 'custom-set) 'set-default))
+                      (get (or (get ',variable 'custom-get) 'default-value)))
+                  (funcall set ',variable (not (funcall get ',variable))))))
           (message ,message "enabled")
-        (message ,message "disabled")))
-     '(menu-item ,doc ,name .
-                 (:button (:toggle . (and (boundp ',variable) ,variable))))))
+        (message ,message "disabled"))
+       ;; The function `customize-mark-as-set' must only be called when
+       ;; a variable is set interactively, as the purpose is to mark it as
+       ;; a candidate for "Save Options", and we do not want to save options
+       ;; the user have already set explicitly in his init file.
+       (if (interactive-p) (customize-mark-as-set ',variable)))
+     '(menu-item ,doc ,name
+                :help ,help
+                 :button (:toggle . (and (default-boundp ',variable)
+                                        (default-value ',variable))))))
+
+;;; Assemble all the top-level items of the "Options" menu
+(define-key menu-bar-options-menu [customize]
+  (list 'menu-item "Customize Emacs" menu-bar-custom-menu
+       :help "Full customization of every Emacs feature"))
+
+(defun menu-bar-options-save ()
+  "Save current values of Options menu items using Custom."
+  (interactive)
+  (let ((need-save nil))
+    ;; These are set with `customize-set-variable'.
+    (dolist (elt '(line-number-mode column-number-mode scroll-bar-mode
+                  debug-on-quit debug-on-error menu-bar-mode tool-bar-mode
+                  save-place uniquify-buffer-name-style fringe-mode
+                  case-fold-search cua-mode show-paren-mode
+                  transient-mark-mode global-font-lock-mode
+                  display-time-mode auto-compression-mode
+                  current-language-environment default-input-method
+                  ;; Saving `text-mode-hook' is somewhat questionable,
+                  ;; as we might get more than we bargain for, if
+                  ;; other code may has added hooks as well.
+                  ;; Nonetheless, not saving it would like be confuse
+                  ;; more often.
+                  ;; -- Per Abrahamsen <abraham@dina.kvl.dk> 2002-02-11.
+                  text-mode-hook))
+      (and (get elt 'customized-value)
+          (customize-mark-to-save elt)
+          (setq need-save t)))
+    ;; Save if we changed anything.
+    (when need-save
+      (custom-save-all))))
+
+(define-key menu-bar-options-menu [save]
+  '(menu-item "Save Options" menu-bar-options-save
+             :help "Save options set from the menu above"))
+
+(define-key menu-bar-options-menu [custom-separator]
+  '("--"))
+
+;; The "Show/Hide" submenu of menu "Options"
+
+(defvar menu-bar-showhide-menu (make-sparse-keymap "Show/Hide"))
+
+(define-key menu-bar-showhide-menu [column-number-mode]
+  (menu-bar-make-mm-toggle column-number-mode
+                          "Show Column Numbers"
+                          "Show the current column number in the mode line"))
+
+(define-key menu-bar-showhide-menu [line-number-mode]
+  (menu-bar-make-mm-toggle line-number-mode
+                          "Show Line Numbers"
+                          "Show the current line number in the mode line"))
+
+(define-key menu-bar-showhide-menu [linecolumn-separator]
+  '("--"))
+
+(defun showhide-date-time ()
+  "Toggle whether to show date and time in the mode-line."
+  (interactive)
+  (if (display-time-mode)
+      (message "Display-time mode enabled.")
+    (message "Display-time mode disabled."))
+  (customize-mark-as-set 'display-time-mode))
+
+(define-key menu-bar-showhide-menu [showhide-date-time]
+  '(menu-item "Date and Time" showhide-date-time
+             :help "Display date and time in the mode line"
+             :button (:toggle . display-time-mode)))
+
+(define-key menu-bar-showhide-menu [datetime-separator]
+  '("--"))
+
+(define-key menu-bar-showhide-menu [showhide-speedbar]
+  '(menu-item "Speedbar" speedbar-frame-mode
+             :help "Display a Speedbar quick-navigation frame"
+             :button (:toggle
+                      . (and (boundp 'speedbar-frame)
+                             (frame-live-p (symbol-value 'speedbar-frame))
+                             (frame-visible-p
+                              (symbol-value 'speedbar-frame))))))
+
+(defvar menu-bar-showhide-fringe-menu (make-sparse-keymap "Fringe"))
+
+(defun menu-bar-showhide-fringe-menu-customize ()
+  "Show customization buffer for `fringe-mode'."
+  (interactive)
+  (customize-variable 'fringe-mode))
+
+(define-key menu-bar-showhide-fringe-menu [customize]
+  '(menu-item "Customize" menu-bar-showhide-fringe-menu-customize
+             :help "Detailed customization of fringe"
+             :visible (display-graphic-p)))
+
+(defun menu-bar-showhide-fringe-menu-customize-reset ()
+  "Reset the fringe mode: display fringes on both sides of a window."
+  (interactive)
+  (customize-set-variable 'fringe-mode nil))
+
+;; The real definition is in fringe.el.
+;; This is to prevent errors in the :radio conditions below.
+(setq fringe-mode nil)
+
+(define-key menu-bar-showhide-fringe-menu [default]
+  '(menu-item "Default" menu-bar-showhide-fringe-menu-customize-reset
+             :help "Default width fringe on both left and right side"
+             :visible (display-graphic-p)
+             :button (:radio . (eq fringe-mode nil))))
+
+(defun menu-bar-showhide-fringe-menu-customize-left ()
+  "Display fringes only on the left of each window."
+  (interactive)
+  (require 'fringe)
+  (customize-set-variable 'fringe-mode '(nil . 0)))
+
+(define-key menu-bar-showhide-fringe-menu [left]
+  '(menu-item "On the Left" menu-bar-showhide-fringe-menu-customize-left
+             :help "Fringe only on the left side"
+             :visible (display-graphic-p)
+             :button (:radio . (equal fringe-mode '(nil . 0)))))
+
+(defun menu-bar-showhide-fringe-menu-customize-right ()
+  "Display fringes only on the right of each window."
+  (interactive)
+  (require 'fringe)
+  (customize-set-variable 'fringe-mode '(0 . nil)))
+
+(define-key menu-bar-showhide-fringe-menu [right]
+  '(menu-item "On the Right" menu-bar-showhide-fringe-menu-customize-right
+             :help "Fringe only on the right side"
+             :visible (display-graphic-p)
+             :button (:radio . (equal fringe-mode '(0 . nil)))))
+
+(defun menu-bar-showhide-fringe-menu-customize-disable ()
+  "Do not display window fringes."
+  (interactive)
+  (require 'fringe)
+  (customize-set-variable 'fringe-mode 0))
+
+(define-key menu-bar-showhide-fringe-menu [none]
+  '(menu-item "None" menu-bar-showhide-fringe-menu-customize-disable
+             :help "Turn off fringe"
+             :visible (display-graphic-p)
+             :button (:radio . (eq fringe-mode 0))))
+
+(define-key menu-bar-showhide-menu [showhide-fringe]
+  (list 'menu-item "Fringe" menu-bar-showhide-fringe-menu
+       :visible `(display-graphic-p)
+       :help "Select fringe mode"))
+
+(defvar menu-bar-showhide-scroll-bar-menu (make-sparse-keymap "Scroll-bar"))
+
+(define-key menu-bar-showhide-scroll-bar-menu [right]
+  '(menu-item "On the Right"
+             menu-bar-right-scroll-bar
+             :help "Scroll-bar on the right side"
+             :visible (display-graphic-p)
+             :button (:radio . (eq (cdr (assq 'vertical-scroll-bars
+                                              (frame-parameters))) 'right))))
+(defun menu-bar-right-scroll-bar ()
+  "Display scroll bars on the right of each window."
+  (interactive)
+  (customize-set-variable 'scroll-bar-mode 'right))
+
+(define-key menu-bar-showhide-scroll-bar-menu [left]
+  '(menu-item "On the Left"
+             menu-bar-left-scroll-bar
+             :help "Scroll-bar on the left side"
+             :visible (display-graphic-p)
+             :button (:radio . (eq (cdr (assq 'vertical-scroll-bars
+                                              (frame-parameters))) 'left))))
+
+(defun menu-bar-left-scroll-bar ()
+  "Display scroll bars on the left of each window."
+  (interactive)
+  (customize-set-variable 'scroll-bar-mode 'left))
+
+(define-key menu-bar-showhide-scroll-bar-menu [none]
+  '(menu-item "None"
+             menu-bar-no-scroll-bar
+             :help "Turn off scroll-bar"
+             :visible (display-graphic-p)
+             :button (:radio . (eq (cdr (assq 'vertical-scroll-bars
+                                              (frame-parameters))) nil))))
+
+(defun menu-bar-no-scroll-bar ()
+  "Turn off scroll bars."
+  (interactive)
+  (customize-set-variable 'scroll-bar-mode nil))
+
+(define-key menu-bar-showhide-menu [showhide-scroll-bar]
+  (list 'menu-item "Scroll-bar" menu-bar-showhide-scroll-bar-menu
+       :visible `(display-graphic-p)
+       :help "Select scroll-bar mode"))
+
+(define-key menu-bar-showhide-menu [menu-bar-mode]
+  '(menu-item "Menu-bar" menu-bar-mode
+             :help "Toggle menu-bar on/off"
+             :button (:toggle . menu-bar-mode)))
+
+(define-key menu-bar-showhide-menu [showhide-tool-bar]
+  (list 'menu-item "Tool-bar" 'tool-bar-mode
+       :help "Turn tool-bar on/off"
+       :visible `(display-graphic-p)
+       :button `(:toggle . tool-bar-mode)))
+
+(define-key menu-bar-options-menu [showhide]
+  (list 'menu-item "Show/Hide" menu-bar-showhide-menu
+       :help "Toggle on/off various display features"))
+
+(define-key menu-bar-options-menu [showhide-separator]
+  '("--"))
+
+(define-key menu-bar-options-menu [mule]
+  ;; It is better not to use backquote here,
+  ;; because that makes a bootstrapping problem
+  ;; if you need to recompile all the Lisp files using interpreted code.
+  (list 'menu-item "Mule (Multilingual Environment)" mule-menu-keymap
+;; Most of the MULE menu actually does make sense in unibyte mode,
+;; e.g. language selection.
+;;;    ':visible 'default-enable-multibyte-characters
+       ':help "Default language, encodings, input method"))
+;(setq menu-bar-final-items (cons 'mule menu-bar-final-items))
+;(define-key menu-bar-options-menu [preferences]
+;  (list 'menu-item "Preferences" menu-bar-preferences-menu
+;      :help "Toggle important global options"))
+
+(define-key menu-bar-options-menu [mule-separator]
+  '("--"))
 
 (define-key menu-bar-options-menu [debug-on-quit]
   (menu-bar-make-toggle toggle-debug-on-quit debug-on-quit
-                       "Debug on Quit" "Debug on Quit %s"))
+                       "Enter Debugger on Quit/C-g" "Debug on Quit %s"
+                       "Enter Lisp debugger when C-g is pressed"))
 (define-key menu-bar-options-menu [debug-on-error]
   (menu-bar-make-toggle toggle-debug-on-error debug-on-error
-                       "Debug on Error" "Debug on Error %s"))
-(define-key menu-bar-options-menu [options-separator]
+                       "Enter Debugger on Error" "Debug on Error %s"
+                       "Enter Lisp debugger when an error is signaled"))
+(define-key menu-bar-options-menu [debugger-separator]
   '("--"))
+(define-key menu-bar-options-menu [toggle-auto-compression]
+  '(menu-item "Automatic File De/compression"
+             auto-compression-mode
+             :help "Transparently decompress compressed files"
+             :button (:toggle . (rassq 'jka-compr-handler
+                                       file-name-handler-alist))))
+
 (define-key menu-bar-options-menu [save-place]
   (menu-bar-make-toggle toggle-save-place-globally save-place
                        "Save Place in Files between Sessions"
                        "Saving place in files %s"
-                        (require 'saveplace)
-                       (setq-default save-place (not (default-value save-place)))))
+                       "Visit files of previous session when restarting Emacs"))
+
 (define-key menu-bar-options-menu [uniquify]
   (menu-bar-make-toggle toggle-uniquify-buffer-names uniquify-buffer-name-style
                        "Use Directory Names in Buffer Names"
                        "Directory name in buffer names (uniquify) %s"
+                       "Uniquify buffer names by adding parent directory names"
                        (require 'uniquify)
                        (setq uniquify-buffer-name-style
                              (if (not uniquify-buffer-name-style)
                                  'forward))))
+
+(define-key menu-bar-options-menu [edit-options-separator]
+  '("--"))
+(define-key menu-bar-options-menu [cua-mode]
+  '(menu-item "CUA-style cut and paste"
+             menu-bar-toggle-cua-mode
+             :help "Use C-z/C-x/C-c/C-v keys for undo/cut/copy/paste"
+             :button (:toggle . cua-mode)))
+
+(defun menu-bar-toggle-cua-mode ()
+  "Toggle CUA key-binding mode.
+When enabled, using shifted movement keys will activate the region (and
+highlight the region using `transient-mark-mode'), and typed text replaces
+the active selection.  C-z, C-x, C-c, and C-v will undo, cut, copy, and
+paste (in addition to the normal Emacs bindings)."
+  (interactive)
+  (cua-mode nil)
+  (customize-mark-as-set 'cua-mode)
+  (message "CUA-style cut and paste %s"
+          (if cua-mode "enabled" "disabled")))
+
 (define-key menu-bar-options-menu [case-fold-search]
   (menu-bar-make-toggle toggle-case-fold-search case-fold-search
-                       "Case folding in searches"
-                       "Case folding in searches %s"))
-(define-key menu-bar-options-menu [transient-mark-mode]
-  (menu-bar-make-toggle toggle-transient-mark-mode transient-mark-mode
-                       "Transient Mark Mode (highlights region)"
-                       "Transient Mark mode %s"))
-(define-key menu-bar-options-menu [toggle-auto-compression]
-  '(menu-item "Automatic File De/compression"
-             auto-compression-mode .
-             (:button (:toggle . (rassq 'jka-compr-handler
-                                        file-name-handler-alist)))))
+                       "Case-Insensitive Search"
+                       "Case-Insensitive Search %s"
+                       "Ignore letter-case in search"))
+
+(defun menu-bar-text-mode-auto-fill ()
+  (interactive)
+  (toggle-text-mode-auto-fill)
+  ;; This is somewhat questionable, as `text-mode-hook'
+  ;; might have changed outside customize.
+  ;; -- Per Abrahamsen <abraham@dina.kvl.dk> 2002-02-11.
+  (customize-mark-as-set 'text-mode-hook))
+
 (define-key menu-bar-options-menu [auto-fill-mode]
-  '(menu-item "Auto Fill (word wrap) in Text modes"
-              toggle-text-mode-auto-fill .
-              (:button (:toggle . (member 'turn-on-auto-fill text-mode-hook)))))
+  '(menu-item "Word Wrap in Text Modes"
+              menu-bar-text-mode-auto-fill
+             :help "Automatically fill text between left and right margins (Auto Fill)"
+              :button (:toggle . (if (listp text-mode-hook)
+                                    (member 'turn-on-auto-fill text-mode-hook)
+                                  (eq 'turn-on-auto-fill text-mode-hook)))))
+(define-key menu-bar-options-menu [truncate-lines]
+  '(menu-item "Truncate Long Lines in this Buffer"
+             toggle-truncate-lines
+             :help "Truncate long lines on the screen"
+             :button (:toggle . truncate-lines)))
+
+(define-key menu-bar-options-menu [highlight-separator]
+  '("--"))
+(define-key menu-bar-options-menu [highlight-paren-mode]
+  (menu-bar-make-mm-toggle show-paren-mode
+                          "Paren Match Highlighting"
+                          "Highlight matching/mismatched parentheses at cursor (Show Paren mode)"))
+(define-key menu-bar-options-menu [transient-mark-mode]
+  (menu-bar-make-mm-toggle transient-mark-mode
+                          "Active Region Highlighting"
+                          "Make text in active region stand out in color (Transient Mark mode)"
+                          (:enable (not cua-mode))))
 (define-key menu-bar-options-menu [toggle-global-lazy-font-lock-mode]
-  (menu-bar-make-toggle toggle-global-lazy-font-lock-mode global-font-lock-mode
-                       "Global Font Lock (highlights syntax)"
-                       "Global Font Lock mode %s"
-                       ;; Make sure a support mode is used;
-                       ;; otherwise Font Lock will be too slow.
-                       (require 'font-lock)
-                       (if (not global-font-lock-mode)
-                           (or font-lock-support-mode
-                               (setq font-lock-support-mode 'lazy-lock-mode)))
-                       (global-font-lock-mode)))
+  (menu-bar-make-mm-toggle global-font-lock-mode
+                          "Syntax Highlighting"
+                          "Colorize text based on language syntax (Global Font Lock mode)"))
+
 \f
+;; The "Tools" menu items
+
+(defun send-mail-item-name ()
+  (let* ((known-send-mail-commands '((sendmail-user-agent . "sendmail")
+                                    (mh-e-user-agent . "MH")
+                                    (message-user-agent . "Gnus Message")
+                                    (gnus-user-agent . "Gnus")))
+        (name (assq mail-user-agent known-send-mail-commands)))
+    (if name
+       (setq name (cdr name))
+      (setq name (symbol-name mail-user-agent))
+      (if (string-match "\\(.+\\)-user-agent" name)
+         (setq name (match-string 1 name))))
+    name))
+
+(defun read-mail-item-name ()
+  (let* ((known-rmail-commands '((rmail . "RMAIL")
+                                (mh-rmail . "MH")
+                                (gnus . "Gnus")))
+        (known (assq read-mail-command known-rmail-commands)))
+    (if known (cdr known) (symbol-name read-mail-command))))
+
+(defvar menu-bar-games-menu (make-sparse-keymap "Games"))
+
+(define-key menu-bar-tools-menu [games]
+  (list 'menu-item "Games" menu-bar-games-menu))
+
+(define-key menu-bar-tools-menu [separator-games]
+  '("--"))
+
+(define-key menu-bar-games-menu [zone]
+  '(menu-item "Zone Out"  zone
+             :help "Play tricks with Emacs display when Emacs is idle"))
+(define-key menu-bar-games-menu [yow]
+  '(menu-item "Random Quotation"  yow
+             :help "Display a random Zippy quotation"))
+(define-key menu-bar-games-menu [tetris]
+  '(menu-item "Tetris"  tetris))
+(define-key menu-bar-games-menu [solitaire]
+  '(menu-item "Solitaire"  solitaire))
+(define-key menu-bar-games-menu [snake]
+  '(menu-item "Snake"  snake
+             :help "Move snake around avoiding collisions"))
+(define-key menu-bar-games-menu [mult]
+  '(menu-item "Multiplication Puzzle"  mpuz
+             :help "Excercise brain with multiplication"))
+(define-key menu-bar-games-menu [life]
+  '(menu-item "Life"  life
+             :help "Watch how John Conway's cellular automaton evolves"))
+(define-key menu-bar-games-menu [hanoi]
+  '(menu-item "Towers of Hanoi" hanoi
+             :help "Watch Towers-of-Hanoi puzzle solved by Emacs"))
+(define-key menu-bar-games-menu [gomoku]
+  '(menu-item "Gomoku"  gomoku
+             :help "Mark 5 contiguous squares (like tic-tac-toe)"))
+(define-key menu-bar-games-menu [black-box]
+  '(menu-item "Blackbox"  blackbox
+             :help "Find balls in a black box by shooting rays"))
+(define-key menu-bar-games-menu [adventure]
+  '(menu-item "Adventure"  dunnet
+             :help "Dunnet, a text Adventure game for Emacs"))
+(define-key menu-bar-games-menu [5x5]
+  '(menu-item "5x5" 5x5
+             :help "Fill in all the squares on a 5x5 board"))
+
+(define-key menu-bar-tools-menu [simple-calculator]
+  '(menu-item "Simple Calculator" calculator
+             :help "Invoke the Emacs built-in quick calculator"))
+(define-key menu-bar-tools-menu [calc]
+  '(menu-item "Programmable Calculator" calc
+             :help "Invoke the Emacs built-in full scientific calculator"))
+(define-key menu-bar-tools-menu [calendar]
+  '(menu-item "Display Calendar" calendar))
+
+(define-key menu-bar-tools-menu [separator-net]
+  '("--"))
+
+(define-key menu-bar-tools-menu [directory-search]
+  '(menu-item "Directory Search" eudc-tools-menu
+             :help "Query directory servers via LDAP, CCSO PH/QI or BBDB"))
+(define-key menu-bar-tools-menu [compose-mail]
+  (list
+   'menu-item `(format "Send Mail (with %s)" (send-mail-item-name))
+   'compose-mail
+   :visible `(and mail-user-agent (not (eq mail-user-agent 'ignore)))
+   :help "Send a mail message"))
+(define-key menu-bar-tools-menu [rmail]
+  (list
+   'menu-item `(format "Read Mail (with %s)" (read-mail-item-name))
+   'menu-bar-read-mail
+   :visible `(and read-mail-command (not (eq read-mail-command 'ignore)))
+   :help "Read your mail and reply to it"))
+
+(defun menu-bar-read-mail ()
+  "Read mail using `read-mail-command'."
+  (interactive)
+  (call-interactively read-mail-command))
+
+(define-key menu-bar-tools-menu [gnus]
+  '(menu-item "Read Net News (Gnus)" gnus
+             :help "Read network news groups"))
+
+(define-key menu-bar-tools-menu [separator-vc]
+  '("--"))
+
+(defvar vc-menu-map (make-sparse-keymap "Version Control"))
+(define-key menu-bar-tools-menu [pcl-cvs]
+  `(menu-item "PCL-CVS" ,cvs-global-menu
+             :help "Module-level interface to CVS"))
+(define-key menu-bar-tools-menu [vc]
+  (list 'menu-item "Version Control" vc-menu-map
+       :help "Interface to RCS, CVS, SCCS"))
+
+(define-key menu-bar-tools-menu [separator-compare]
+  '("--"))
+
+(define-key menu-bar-tools-menu [ediff-misc]
+  '(menu-item "Ediff Miscellanea" menu-bar-ediff-misc-menu
+             :help "Ediff manual, customization, sessions, etc."))
+(define-key menu-bar-tools-menu [epatch]
+  '(menu-item "Apply Patch" menu-bar-epatch-menu))
+(define-key menu-bar-tools-menu [ediff-merge]
+  '(menu-item "Merge" menu-bar-ediff-merge-menu
+             :help "Merge different revisions of files/directories"))
+(define-key menu-bar-tools-menu [compare]
+  '(menu-item "Compare (Ediff)" menu-bar-ediff-menu
+             :help "Display differences between files/directories"))
+
+
+(define-key menu-bar-tools-menu [separator-spell]
+  '("--"))
+
+(define-key menu-bar-tools-menu [spell]
+  '(menu-item "Spell Checking" ispell-menu-map))
+
+(define-key menu-bar-tools-menu [separator-prog]
+  '("--"))
+
+(define-key menu-bar-tools-menu [gdb]
+  '(menu-item "Debugger (GDB)..." gdb
+             :help "Debug a program from within Emacs with GDB"))
+(define-key menu-bar-tools-menu [shell-on-region]
+  '(menu-item "Shell Command on Region..." shell-command-on-region
+             :enable mark-active
+             :help "Pass marked region to a shell command"))
+(define-key menu-bar-tools-menu [shell]
+  '(menu-item "Shell Command..." shell-command
+             :help "Invoke a shell command and catch its output"))
+(define-key menu-bar-tools-menu [compile]
+  '(menu-item "Compile..." compile
+             :help "Invoke compiler or Make, view compilation errors"))
+(define-key menu-bar-tools-menu [grep]
+  '(menu-item "Search Files (Grep)..." grep
+             :help "Search files for strings or regexps (with Grep)"))
+
+\f
+;; The "Help" menu items
+
 (defvar menu-bar-describe-menu (make-sparse-keymap "Describe"))
 
+(define-key menu-bar-describe-menu [mule-diag]
+  '(menu-item "Show All of Mule Status" mule-diag
+             :visible default-enable-multibyte-characters
+             :help "Display multilingual environment settings"))
+(define-key menu-bar-describe-menu [describe-coding-system-briefly]
+  '(menu-item "Describe Coding System (Briefly)..."
+              describe-current-coding-system-briefly
+              :visible default-enable-multibyte-characters))
+(define-key menu-bar-describe-menu [describe-coding-system]
+  '(menu-item "Describe Coding System..." describe-coding-system
+             :visible default-enable-multibyte-characters))
+(define-key menu-bar-describe-menu [describe-input-method]
+  '(menu-item "Describe Input Method..." describe-input-method
+             :visible default-enable-multibyte-characters
+             :help "Keyboard layout for specific input method"))
+(define-key menu-bar-describe-menu [describe-language-environment]
+  (list 'menu-item "Describe Language Environment"
+       describe-language-environment-map
+       :help "Show multilingual settings for a specific language"))
+
+(define-key menu-bar-describe-menu [separator-desc-mule]
+  '("--"))
+
+(define-key menu-bar-describe-menu [list-keybindings]
+  '(menu-item "List Key Bindings" describe-bindings
+             :help "Display a list of all current keybindings"))
+(define-key menu-bar-describe-menu [describe-current-display-table]
+  '(menu-item "Describe Display Table" describe-current-display-table
+             :help "Describe the current display table"))
+(define-key menu-bar-describe-menu [describe-face]
+  '(menu-item "Describe Face..." describe-face
+              :help "Display the properties of a face"))
 (define-key menu-bar-describe-menu [describe-variable]
-  '("Describe Variable..." . describe-variable))
+  '(menu-item "Describe Variable..." describe-variable
+             :help "Display documentation of variable/option"))
 (define-key menu-bar-describe-menu [describe-function]
-  '("Describe Function..." . describe-function))
+  '(menu-item "Describe Function..." describe-function
+             :help "Display documentation of function/command"))
+(define-key menu-bar-describe-menu [describe-key-1]
+  '(menu-item "Describe Key..." describe-key
+             ;; Users typically don't identify keys and menu items...
+             :help "Display documentation of command bound to a \
+key (or menu-item)"))
 (define-key menu-bar-describe-menu [describe-key]
-  '("Describe Key..." . describe-key))
-(define-key menu-bar-describe-menu [list-keybindings]
-  '("List Key Bindings" . describe-bindings))
-(define-key menu-bar-describe-menu [apropos-variables]
-  '("Apropos Variables..." . apropos-variable))
-(define-key menu-bar-describe-menu [apropos-commands]
-  '("Apropos Commands..." . apropos-command))
+  '(menu-item "What's This? " describe-key
+             ;; Users typically don't identify keys and menu items...
+             :help "Display documentation of command bound to a \
+key (or menu-item)"))
 (define-key menu-bar-describe-menu [describe-mode]
-  '("Describe Buffer Modes" . describe-mode))
+  '(menu-item "Describe Buffer Modes" describe-mode
+             :help "Describe this buffer's major and minor mode"))
 
-(defvar menu-bar-manuals-menu (make-sparse-keymap "Manuals"))
+(defvar menu-bar-apropos-menu (make-sparse-keymap "Apropos"))
+(defun menu-bar-read-lispref ()
+  "Display the Emacs Lisp Reference manual in Info mode."
+  (interactive)
+  (info "elisp"))
+
+(defun menu-bar-read-lispintro ()
+  "Display the Introduction to Emacs Lisp Programming in Info mode."
+  (interactive)
+  (info "eintr"))
+
+(defun search-emacs-glossary ()
+  "Display the Glossary node of the Emacs manual in Info mode."
+  (interactive)
+  (info "(emacs)Glossary"))
+
+(defun emacs-index-search (topic)
+  "Look up TOPIC in the indices of the Emacs User Manual."
+  (interactive "sSubject to look up: ")
+  (info "emacs")
+  (Info-index topic))
+
+(defun elisp-index-search (topic)
+  "Look up TOPIC in the indices of the Emacs Lisp Reference Manual."
+  (interactive "sSubject to look up: ")
+  (info "elisp")
+  (Info-index topic))
+
+(define-key menu-bar-apropos-menu [apropos-documentation]
+  '(menu-item "Search Documentation Strings..." apropos-documentation
+              :help
+             "Find functions and variables whose doc strings match a regexp"))
+(define-key menu-bar-apropos-menu [apropos]
+  '(menu-item "Find Any Object by Name..."  apropos
+              :help "Find symbols of any kind whose names match a regexp"))
+(define-key menu-bar-apropos-menu [apropos-value]
+  '(menu-item "Find Options by Value..." apropos-value
+              :help "Find variables whose values match a regexp"))
+(define-key menu-bar-apropos-menu [apropos-variables]
+  '(menu-item "Find Options by Name..." apropos-variable
+             :help "Find variables whose names match a regexp"))
+(define-key menu-bar-apropos-menu [apropos-commands]
+  '(menu-item "Find Commands by Name..." apropos-command
+             :help "Find commands whose names match a regexp"))
+(define-key menu-bar-apropos-menu [sep1]
+  '("--"))
+(define-key menu-bar-apropos-menu [elisp-index-search]
+  '(menu-item "Look Up Subject in ELisp Manual..." elisp-index-search
+             :help "Find description of a subject in Emacs Lisp manual"))
+(define-key menu-bar-apropos-menu [emacs-index-search]
+  '(menu-item "Look Up Subject in User Manual..." emacs-index-search
+             :help "Find description of a subject in Emacs User manual"))
+(define-key menu-bar-apropos-menu [emacs-glossary]
+  '(menu-item "Emacs Terminology" search-emacs-glossary
+             :help "Display the Glossary section of the Emacs manual"))
+
+(defvar menu-bar-manuals-menu (make-sparse-keymap "More Manuals"))
 
 (define-key menu-bar-manuals-menu [man]
-  '("Read Man Page..." . manual-entry))
+  '(menu-item "Read Man Page..." manual-entry
+             :help "Man-page docs for external commands and libraries"))
 (define-key menu-bar-manuals-menu [sep2]
   '("--"))
-(define-key menu-bar-manuals-menu [key]
-  '("Find Key in Manual" . Info-goto-emacs-key-command-node))
-(define-key menu-bar-manuals-menu [command]
-  '("Find Command in Manual" . Info-goto-emacs-command-node))
+(define-key menu-bar-manuals-menu [order-emacs-manuals]
+  '(menu-item "Ordering Manuals" view-order-manuals
+             :help "How to order manuals from the Free Software Foundation"))
 (define-key menu-bar-manuals-menu [info]
-  '("Browse Manuals with Info" . info))
-(define-key menu-bar-manuals-menu [sep1]
+  '(menu-item "All Other Manuals (Info)" Info-directory
+             :help "Read any of the installed manuals"))
+(define-key menu-bar-manuals-menu [info-elisp]
+  '(menu-item "Emacs Lisp Reference" menu-bar-read-lispref
+             :help "Read the Emacs Lisp Reference manual"))
+(define-key menu-bar-manuals-menu [info-elintro]
+  '(menu-item "Introduction to Emacs Lisp" menu-bar-read-lispintro
+             :help "Read the Introduction to Emacs Lisp Programming"))
+(define-key menu-bar-manuals-menu [sep3]
   '("--"))
-(define-key menu-bar-manuals-menu [emacs-faq]
-  '("Emacs FAQ" . view-emacs-FAQ))
-(define-key menu-bar-manuals-menu [emacs-news]
-  '("Emacs News" . view-emacs-news))
+(define-key menu-bar-manuals-menu [command]
+  '(menu-item "Find Command in Manual" Info-goto-emacs-command-node
+             :help "Display manual section that describes a command"))
+(define-key menu-bar-manuals-menu [key]
+  '(menu-item "Find Key in Manual" Info-goto-emacs-key-command-node
+             :help "Display manual section that describes a key"))
 
+(define-key menu-bar-help-menu [eliza]
+  '(menu-item "Emacs Psychiatrist" doctor
+             :help "Our doctor will help you feel better"))
+(define-key menu-bar-help-menu [sep4]
+  '("--"))
 (define-key menu-bar-help-menu [describe-no-warranty]
-  '("(Non)Warranty" . describe-no-warranty))
+  '(menu-item "(Non)Warranty" describe-no-warranty
+             :help "Explain that Emacs has NO WARRANTY"))
 (define-key menu-bar-help-menu [describe-copying]
-  '("Copying Conditions" . describe-copying))
+  '(menu-item "Copying Conditions" describe-copying
+             :help "Show the Emacs license (GPL)"))
 (define-key menu-bar-help-menu [describe-distribution]
-  '("Getting New Versions" . describe-distribution))
-(define-key menu-bar-help-menu [emacs-version]
-  '("Show Version" . emacs-version))
-(define-key menu-bar-help-menu [report-emacs-bug]
-  '("Send Bug Report..." . report-emacs-bug))
+  '(menu-item "Getting New Versions" describe-distribution
+             :help "How to get latest versions of Emacs"))
+(define-key menu-bar-help-menu [more]
+  '(menu-item "Find Extra Packages"
+             menu-bar-help-extra-packages
+             :help "Where to find some extra packages and possible updates"))
+(defun menu-bar-help-extra-packages ()
+  "Display help about some additional packages available for Emacs."
+  (interactive)
+  (let (enable-local-variables)
+    (view-file (expand-file-name "MORE.STUFF"
+                                data-directory))
+    (goto-address)))
+(define-key menu-bar-help-menu [about]
+  '(menu-item "About Emacs" display-splash-screen
+             :help "Display version number, copyright info, and basic help"))
 (define-key menu-bar-help-menu [sep2]
   '("--"))
 (define-key menu-bar-help-menu [finder-by-keyword]
-  '("Find Emacs Packages..." . finder-by-keyword))
-(define-key menu-bar-help-menu [describe]
-  (cons "Describe" menu-bar-describe-menu))
+  '(menu-item "Find Emacs Packages..." finder-by-keyword
+             :help "Find packages and features by keyword"))
 (define-key menu-bar-help-menu [manuals]
-  (cons "Manuals" menu-bar-manuals-menu))
-(define-key menu-bar-help-menu [emacs-tutorial]
-  '("Emacs Tutorial" . help-with-tutorial))
+  (list 'menu-item "More Manuals" menu-bar-manuals-menu
+       :help "Search and browse on-line manuals"))
+(define-key menu-bar-help-menu [emacs-manual]
+  '(menu-item "Read the Emacs Manual" info-emacs-manual
+             :help "Full documentation of Emacs features"))
+(define-key menu-bar-help-menu [describe]
+  (list 'menu-item "Describe" menu-bar-describe-menu
+       :help "Describe commands, variables, keys"))
+(define-key menu-bar-help-menu [apropos]
+  (list 'menu-item "Search Documentation" menu-bar-apropos-menu
+       :help "Look up terms, find commands, options, etc. (Apropos)"))
 (define-key menu-bar-help-menu [sep1]
   '("--"))
-(define-key menu-bar-help-menu [options]
-  (cons "Options" menu-bar-options-menu))
-(define-key menu-bar-help-menu [customize]
-  (cons "Customize" menu-bar-custom-menu))
+(define-key menu-bar-help-menu [report-emacs-bug]
+  '(menu-item "Send Bug Report..." report-emacs-bug
+             :help "Send e-mail to Emacs maintainers"))
+(define-key menu-bar-help-menu [emacs-problems]
+  '(menu-item "Emacs Known Problems" view-emacs-problems))
+(define-key menu-bar-help-menu [emacs-news]
+  '(menu-item "Emacs News" view-emacs-news
+             :help "New features of this version"))
+(define-key menu-bar-help-menu [emacs-faq]
+  '(menu-item "Emacs FAQ" view-emacs-FAQ))
+
+(defun help-with-tutorial-spec-language ()
+  "Use the Emacs tutorial, specifying which language you want."
+  (interactive)
+  (help-with-tutorial t))
+
+(define-key menu-bar-help-menu [emacs-tutorial-language-specific]
+  '(menu-item "Emacs Tutorial (choose language)..."
+             help-with-tutorial-spec-language
+             :help "Learn how to use Emacs (choose a language)"))
+(define-key menu-bar-help-menu [emacs-tutorial]
+  '(menu-item "Emacs Tutorial" help-with-tutorial
+             :help "Learn how to use Emacs"))
 
 (defun kill-this-buffer ()     ; for the menubar
-  "Kills the current buffer."
+  "Kill the current buffer."
   (interactive)
   (kill-buffer (current-buffer)))
 
@@ -523,34 +1325,10 @@ Do the same for the keys of the same name."
     (and (not (window-minibuffer-p (frame-selected-window menu-updating-frame)))
         (> count 1))))
 
-(put 'kill-this-buffer 'menu-enable '(kill-this-buffer-enabled-p))
-
-(put 'save-buffer 'menu-enable
-     '(and (buffer-modified-p)
-          (not (window-minibuffer-p (frame-selected-window menu-updating-frame)))))
-
-(put 'write-file 'menu-enable
-     '(not (window-minibuffer-p (frame-selected-window menu-updating-frame))))
-
-(put 'find-file 'menu-enable
-     '(not (window-minibuffer-p (frame-selected-window menu-updating-frame))))
-
 (put 'dired 'menu-enable
      '(not (window-minibuffer-p (frame-selected-window menu-updating-frame))))
 
-(put 'insert-file 'menu-enable
-     '(not (window-minibuffer-p (frame-selected-window menu-updating-frame))))
-
-(put 'revert-buffer 'menu-enable
-     '(or revert-buffer-function revert-buffer-insert-file-contents-function
-         (and (buffer-file-name)
-              (or (buffer-modified-p)
-                  (not (verify-visited-file-modtime (current-buffer)))))))
-
 ;; Permit deleting frame if it would leave a visible or iconified frame.
-(put 'delete-frame 'menu-enable
-     '(delete-frame-enabled-p))
-
 (defun delete-frame-enabled-p ()
   "Return non-nil if `delete-frame' should be enabled in the menu bar."
   (let ((frames (frame-list))
@@ -561,13 +1339,6 @@ Do the same for the keys of the same name."
       (setq frames (cdr frames)))
     (> count 1)))
 
-(put 'advertised-undo 'menu-enable
-     '(and (not (eq t buffer-undo-list))
-          (if (eq last-command 'undo)
-              (and (boundp 'pending-undo-list)
-                   pending-undo-list)
-            buffer-undo-list)))
-
 (defcustom yank-menu-length 20
   "*Maximum length to display in the yank-menu."
   :type 'integer
@@ -604,11 +1375,36 @@ Do the same for the keys of the same name."
   (interactive "*")
   (push-mark (point))
   (insert last-command-event))
+
 \f
-;; This definition is just to show what this looks like.
-;; It gets overridden below when menu-bar-update-buffers is called.
-(define-key global-map [menu-bar buffer]
-  (cons "Buffers" (make-sparse-keymap "Buffers")))
+(defcustom buffers-menu-show-directories 'unless-uniquify
+  "If non-nil, show directories in the Buffers menu for buffers that have them.
+The special value `unless-uniquify' means that directories will be shown
+unless `uniquify-buffer-name-style' is non-nil (in which case, buffer
+names should include enough of a buffer's directory to distinguish it
+from other buffers).
+
+Setting this variable directly does not take effect until next time the
+Buffers menu is regenerated."
+  :set (lambda (symbol value)
+        (set symbol value)
+        (menu-bar-update-buffers t))
+  :initialize 'custom-initialize-default
+  :type '(choice (const :tag "Never" nil)
+                (const :tag "Unless uniquify is enabled" unless-uniquify)
+                (const :tag "Always" t))
+  :group 'menu)
+
+(defcustom buffers-menu-show-status t
+  "If non-nil, show modified/read-only status of buffers in the Buffers menu.
+Setting this variable directly does not take effect until next time the
+Buffers menu is regenerated."
+  :set (lambda (symbol value)
+        (set symbol value)
+        (menu-bar-update-buffers t))
+  :initialize 'custom-initialize-default
+  :type 'boolean
+  :group 'menu)
 
 (defvar list-buffers-directory nil)
 
@@ -620,41 +1416,47 @@ Do the same for the keys of the same name."
 
 (defun menu-bar-select-frame ()
   (interactive)
-  (make-frame-visible last-command-event)
-  (raise-frame last-command-event)
-  (select-frame last-command-event))
+  (let (frame)
+    (dolist (f (frame-list))
+      (when (equal last-command-event (frame-parameter f 'name))
+       (setq frame f)))
+  (make-frame-visible frame)
+  (raise-frame frame)
+  (select-frame frame)))
 
 (defun menu-bar-update-buffers-1 (elt)
-  (cons (format
-        (format "%%%ds  %%s%%s  %%s" menu-bar-update-buffers-maxbuf)
-        (cdr elt)
-        (if (buffer-modified-p (car elt))
-            "*" " ")
-        (save-excursion
-          (set-buffer (car elt))
-          (if buffer-read-only "%" " "))
-        (let ((file
-               (or (buffer-file-name (car elt))
-                   (save-excursion
-                     (set-buffer (car elt))
-                     list-buffers-directory)
-                   "")))
-          (setq file (or (file-name-directory file)
-                         ""))
-          (if (> (length file) 20)
-              (setq file (concat "..." (substring file -17))))
-          file))
-       (car elt)))
-
-(defvar menu-bar-buffers-menu-list-buffers-entry nil)
-
-(defun menu-bar-update-buffers ()
+  (let* ((buf (car elt))
+        (file
+         (and (if (eq buffers-menu-show-directories 'unless-uniquify)
+                  (or (not (boundp 'uniquify-buffer-name-style))
+                      (null uniquify-buffer-name-style))
+                buffers-menu-show-directories)
+              (or (buffer-file-name buf)
+                  (buffer-local-value 'list-buffers-directory buf)))))
+    (when file
+      (setq file (file-name-directory file)))
+    (when (and file (> (length file) 20))
+      (setq file (concat "..." (substring file -17))))
+    (cons (if buffers-menu-show-status
+             (let ((mod (if (buffer-modified-p buf) "*" ""))
+                   (ro (if (buffer-local-value 'buffer-read-only buf) "%" "")))
+               (if file
+                   (format "%s  %s%s  --  %s" (cdr elt) mod ro file)
+                 (format "%s  %s%s" (cdr elt) mod ro)))
+           (if file
+               (format "%s  --  %s"  (cdr elt) file)
+             (cdr elt)))
+         buf)))
+
+;; Used to cache the menu entries for commands in the Buffers menu
+(defvar menu-bar-buffers-menu-command-entries nil)
+
+(defun menu-bar-update-buffers (&optional force)
   ;; If user discards the Buffers item, play along.
   (and (lookup-key (current-global-map) [menu-bar buffer])
-       (frame-or-buffer-changed-p)
+       (or force (frame-or-buffer-changed-p))
        (let ((buffers (buffer-list))
             (frames (frame-list))
-            (maxlen 0)
             buffers-menu frames-menu)
         ;; If requested, list only the N most recently selected buffers.
         (if (and (integerp buffers-menu-max-size)
@@ -664,101 +1466,99 @@ Do the same for the keys of the same name."
 
         ;; Make the menu of buffers proper.
         (setq buffers-menu
-              (cons "Select Buffer"
-                    (let* ((buffer-list
-                            (mapcar 'list buffers))
-                           tail
-                           (menu-bar-update-buffers-maxbuf 0)
-                           alist
-                           head)
-                      ;; Put into each element of buffer-list
-                      ;; the name for actual display,
-                      ;; perhaps truncated in the middle.
-                      (setq tail buffer-list)
-                      (while tail
-                        (let ((name (buffer-name (car (car tail)))))
-                          (setcdr (car tail)
-                                  (if (> (length name) 27)
-                                      (concat (substring name 0 12)
-                                              "..."
-                                              (substring name -12))
-                                    name)))
-                        (setq tail (cdr tail)))
-                      ;; Compute the maximum length of any name.
-                      (setq tail buffer-list)
-                      (while tail
-                        (or (eq ?\ (aref (cdr (car tail)) 0))
-                            (setq menu-bar-update-buffers-maxbuf
-                                  (max menu-bar-update-buffers-maxbuf
-                                       (length (cdr (car tail))))))
-                        (setq tail (cdr tail)))
-                      ;; Set ALIST to an alist of the form
-                      ;; ITEM-STRING . BUFFER
-                      (setq tail buffer-list)
-                      (while tail
-                        (let ((elt (car tail)))
-                          (or (eq ?\ (aref (cdr elt) 0))
-                              (setq alist (cons
-                                           (menu-bar-update-buffers-1 elt)
-                                           alist)))
-                          (and alist (> (length (car (car alist))) maxlen)
-                               (setq maxlen (length (car (car alist))))))
-                        (setq tail (cdr tail)))
-                      (setq alist (nreverse alist))
-                      ;; Make the menu item for list-buffers
-                      ;; or reuse the one we already have.
-                      ;; The advantage in reusing one
-                      ;; is that it already has the keyboard equivalent
-                      ;; cached, so we save the time to look that up again.
-                      (or menu-bar-buffers-menu-list-buffers-entry
-                          (setq menu-bar-buffers-menu-list-buffers-entry
-                                (cons
-                                 'list-buffers
-                                 (cons
-                                  ""
-                                  'list-buffers))))
-                      ;; Update the item string for menu's new width.
-                      (setcar (cdr menu-bar-buffers-menu-list-buffers-entry)
-                              (concat (make-string (max (- (/ maxlen 2) 8) 0)
-                                                   ?\ )
-                                      "List All Buffers"))
-                      ;; Now make the actual list of items,
-                      ;; ending with the list-buffers item.
-                      (nconc (mapcar '(lambda (pair)
-                                        ;; This is somewhat risque, to use
-                                        ;; the buffer name itself as the event
-                                        ;; type to define, but it works.
-                                        ;; It would not work to use the buffer
-                                        ;; since a buffer as an event has its
-                                        ;; own meaning.
-                                        (nconc (list (buffer-name (cdr pair))
-                                                     (car pair)
-                                                     (cons nil nil))
-                                               'menu-bar-select-buffer))
-                                     alist)
-                             (list menu-bar-buffers-menu-list-buffers-entry)))))
-
+              (let* ((buffer-list
+                      (mapcar 'list buffers))
+                     (menu-bar-update-buffers-maxbuf 0)
+                     alist)
+                ;; Put into each element of buffer-list
+                ;; the name for actual display,
+                ;; perhaps truncated in the middle.
+                (dolist (buf buffer-list)
+                  (let ((name (buffer-name (car buf))))
+                    (setcdr buf
+                            (if (> (length name) 27)
+                                (concat (substring name 0 12)
+                                        "..."
+                                        (substring name -12))
+                              name))))
+                ;; Compute the maximum length of any name.
+                (dolist (buf buffer-list)
+                  (unless (eq ?\  (aref (cdr buf) 0))
+                    (setq menu-bar-update-buffers-maxbuf
+                          (max menu-bar-update-buffers-maxbuf
+                               (length (cdr buf))))))
+                ;; Set ALIST to an alist of the form
+                ;; ITEM-STRING . BUFFER
+                (dolist (buf buffer-list)
+                  (unless (eq ?\  (aref (cdr buf) 0))
+                    (push (menu-bar-update-buffers-1 buf) alist)))
+                ;; Now make the actual list of items, and add
+                ;; some miscellaneous buffer commands to the end.
+                (mapcar (lambda (pair)
+                          ;; This is somewhat risque, to use
+                          ;; the buffer name itself as the event
+                          ;; type to define, but it works.
+                          ;; It would not work to use the buffer
+                          ;; since a buffer as an event has its
+                          ;; own meaning.
+                          (nconc (list (buffer-name (cdr pair))
+                                       (car pair)
+                                       (cons nil nil))
+                                 'menu-bar-select-buffer))
+                        (nreverse alist))))
 
         ;; Make a Frames menu if we have more than one frame.
-        (if (cdr frames)
-            (let ((name (concat (make-string (max (- (/ maxlen 2) 3) 0)
-                                             ?\ )
-                                "Frames"))
-                  (frames-menu
-                   (cons 'keymap
-                         (cons "Select Frame"
-                               (mapcar '(lambda (frame)
-                                          (nconc (list frame
-                                                       (cdr (assq 'name
-                                                                  (frame-parameters frame)))
-                                                       (cons nil nil))
-                                                 'menu-bar-select-frame))
-                                       frames)))))
-              ;; Put it underneath the Buffers menu.
-              (setq buffers-menu (cons (cons 'frames (cons name frames-menu))
-                                       buffers-menu))))
-        (if buffers-menu
-            (setq buffers-menu (cons 'keymap buffers-menu)))
+        (when (cdr frames)
+          (let ((frames-menu
+                 (cons 'keymap
+                       (cons "Select Frame"
+                             (mapcar
+                              (lambda (frame)
+                                (nconc
+                                 (list (frame-parameter frame 'name)
+                                       (frame-parameter frame 'name)
+                                       (cons nil nil))
+                                 'menu-bar-select-frame))
+                              frames)))))
+            ;; Put it after the normal buffers
+            (setq buffers-menu
+                  (nconc buffers-menu
+                         `((frames-separator "--")
+                           (frames menu-item "Frames" ,frames-menu))))))
+
+        ;; Add in some normal commands at the end of the menu.  We use
+        ;; the copy cached in `menu-bar-buffers-menu-command-entries'
+        ;; if it's been set already.  Note that we can't use constant
+        ;; lists for the menu-entries, because the low-level menu-code
+        ;; modifies them.
+        (unless menu-bar-buffers-menu-command-entries
+          (setq menu-bar-buffers-menu-command-entries
+                (list '(command-separator "--")
+                      (list 'next-buffer
+                            'menu-item
+                            "Next Buffer"
+                            'next-buffer
+                            :help "Switch to the \"next\" buffer in a cyclic order")
+                      (list 'prev-buffer
+                            'menu-item
+                            "Previous Buffer"
+                            'prev-buffer
+                            :help "Switch to the \"previous\" buffer in a cyclic order")
+                      (list 'select-named-buffer
+                            'menu-item
+                            "Select Named Buffer..."
+                            'switch-to-buffer
+                            :help "Prompt for a buffer name, and select that buffer in the current window")
+                      (list 'list-all-buffers
+                            'menu-item
+                            "List All Buffers"
+                            'list-buffers
+                            :help "Pop up a window listing all emacs buffers"
+                            ))))
+        (setq buffers-menu
+              (nconc buffers-menu menu-bar-buffers-menu-command-entries))
+
+        (setq buffers-menu (cons 'keymap (cons "Select Buffer" buffers-menu)))
         (define-key (current-global-map) [menu-bar buffer]
           (cons "Buffers" buffers-menu)))))
 
@@ -767,102 +1567,89 @@ Do the same for the keys of the same name."
 (menu-bar-update-buffers)
 
 ;; this version is too slow
-;;;(defun format-buffers-menu-line (buffer)
-;;;  "Returns a string to represent the given buffer in the Buffer menu.
-;;;nil means the buffer shouldn't be listed.  You can redefine this."
-;;;  (if (string-match "\\` " (buffer-name buffer))
-;;;      nil
-;;;    (save-excursion
-;;;     (set-buffer buffer)
-;;;     (let ((size (buffer-size)))
-;;;       (format "%s%s %-19s %6s %-15s %s"
-;;;           (if (buffer-modified-p) "*" " ")
-;;;           (if buffer-read-only "%" " ")
-;;;           (buffer-name)
-;;;           size
-;;;           mode-name
-;;;           (or (buffer-file-name) ""))))))
+;;(defun format-buffers-menu-line (buffer)
+;;  "Returns a string to represent the given buffer in the Buffer menu.
+;;nil means the buffer shouldn't be listed.  You can redefine this."
+;;  (if (string-match "\\` " (buffer-name buffer))
+;;      nil
+;;    (save-excursion
+;;     (set-buffer buffer)
+;;     (let ((size (buffer-size)))
+;;       (format "%s%s %-19s %6s %-15s %s"
+;;            (if (buffer-modified-p) "*" " ")
+;;            (if buffer-read-only "%" " ")
+;;            (buffer-name)
+;;            size
+;;            mode-name
+;;            (or (buffer-file-name) ""))))))
 \f
 ;;; Set up a menu bar menu for the minibuffer.
 
-(mapcar
- (function
-  (lambda (map)
-    (define-key map [menu-bar minibuf]
-      (cons "Minibuf" (make-sparse-keymap "Minibuf")))))
- (list minibuffer-local-ns-map
-       minibuffer-local-must-match-map
-       minibuffer-local-isearch-map
-       minibuffer-local-map
-       minibuffer-local-completion-map))
-
-(mapcar
- (function
-  (lambda (map)
-    (define-key map [menu-bar minibuf ?\?]
-      '("List Completions" . minibuffer-completion-help))
-    (define-key map [menu-bar minibuf space]
-      '("Complete Word" . minibuffer-complete-word))
-    (define-key map [menu-bar minibuf tab]
-      '("Complete" . minibuffer-complete))
-    ))
- (list minibuffer-local-must-match-map
-       minibuffer-local-completion-map))
-
-(mapcar
- (function
-  (lambda (map)
-    (define-key map [menu-bar minibuf quit]
-      '("Quit" . keyboard-escape-quit))
-    (define-key map [menu-bar minibuf return]
-      '("Enter" . exit-minibuffer))
-    ))
- (list minibuffer-local-ns-map
-       minibuffer-local-must-match-map
-       minibuffer-local-isearch-map
-       minibuffer-local-map
-       minibuffer-local-completion-map))
+(dolist (map (list minibuffer-local-map
+                  ;; This shouldn't be necessary, but there's a funny
+                  ;; bug in keymap.c that I don't understand yet.  -stef
+                  minibuffer-local-completion-map))
+  (define-key map [menu-bar minibuf]
+    (cons "Minibuf" (make-sparse-keymap "Minibuf"))))
+
+(let ((map minibuffer-local-completion-map))
+  (define-key map [menu-bar minibuf ?\?]
+    (list 'menu-item "List Completions" 'minibuffer-completion-help
+         :help "Display all possible completions"))
+  (define-key map [menu-bar minibuf space]
+    (list 'menu-item "Complete Word" 'minibuffer-complete-word
+         :help "Complete at most one word"))
+  (define-key map [menu-bar minibuf tab]
+    (list 'menu-item "Complete" 'minibuffer-complete
+         :help "Complete as far as possible")))
+
+(let ((map minibuffer-local-map))
+  (define-key map [menu-bar minibuf quit]
+    (list 'menu-item "Quit" 'keyboard-escape-quit
+         :help "Abort input and exit minibuffer"))
+  (define-key map [menu-bar minibuf return]
+    (list 'menu-item "Enter" 'exit-minibuffer
+         :help "Terminate input and exit minibuffer")))
 \f
-(defvar menu-bar-mode nil)
-
-(defun menu-bar-mode (flag)
+;;;###autoload
+;; This comment is taken from toolbar/tool-bar.el near
+;; (put 'tool-bar-mode ...)
+;; We want to pretend the menu bar by standard is on, as this will make
+;; customize consider disabling the menu bar a customization, and save
+;; that.  We could do this for real by setting :init-value below, but
+;; that would overwrite disabling the tool bar from X resources.
+(put 'menu-bar-mode 'standard-value '(t))
+
+;;;###autoload
+(define-minor-mode menu-bar-mode
   "Toggle display of a menu bar on each frame.
 This command applies to all frames that exist and frames to be
 created in the future.
 With a numeric argument, if the argument is positive,
 turn on menu bars; otherwise, turn off menu bars."
- (interactive "P")
-
+  :init-value nil
+  :global t
+  :group 'frames
   ;; Make menu-bar-mode and default-frame-alist consistent.
-  (let ((default (assq 'menu-bar-lines default-frame-alist)))
-    (if default
-       (setq menu-bar-mode (not (eq (cdr default) 0)))
-      (setq default-frame-alist
-           (cons (cons 'menu-bar-lines (if menu-bar-mode 1 0))
-                 default-frame-alist))))
-
-  ;; Toggle or set the mode, according to FLAG.
- (setq menu-bar-mode (if (null flag) (not menu-bar-mode)
-                      (> (prefix-numeric-value flag) 0)))
-
- ;; Apply it to default-frame-alist.
- (let ((parameter (assq 'menu-bar-lines default-frame-alist)))
-   (if (consp parameter)
-       (setcdr parameter (if menu-bar-mode 1 0))
-     (setq default-frame-alist
-          (cons (cons 'menu-bar-lines (if menu-bar-mode 1 0))
-                default-frame-alist))))
-
- ;; Apply it to existing frames.
- (let ((frames (frame-list)))
-   (while frames
-     (let ((height (cdr (assq 'height (frame-parameters (car frames))))))
-       (modify-frame-parameters (car frames)
-                               (list (cons 'menu-bar-lines
-                                         (if menu-bar-mode 1 0))))
-       (modify-frame-parameters (car frames)
-                               (list (cons 'height height))))
-     (setq frames (cdr frames)))))
+  (let ((lines (if menu-bar-mode 1 0)))
+    ;; Alter existing frames...
+    (mapc (lambda (frame)
+           (modify-frame-parameters frame
+                                    (list (cons 'menu-bar-lines lines))))
+         (frame-list))
+    ;; ...and future ones.
+    (let ((elt (assq 'menu-bar-lines default-frame-alist)))
+      (if elt
+         (setcdr elt lines)
+       (add-to-list 'default-frame-alist (cons 'menu-bar-lines lines)))))
+
+  ;; Make the message appear when Emacs is idle.  We can not call message
+  ;; directly.  The minor-mode message "Menu-bar mode disabled" comes
+  ;; after this function returns, overwriting any message we do here.
+  (when (and (interactive-p) (not menu-bar-mode))
+    (run-with-idle-timer 0 nil 'message
+                        "Menu-bar mode disabled.  Use M-x menu-bar-mode to make the menu bar appear."))
+  menu-bar-mode)
 
 (provide 'menu-bar)