all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Nick Roberts <nickrob@snap.net.nz>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: Dan Nicolaescu <dann@ics.uci.edu>, emacs-devel@gnu.org
Subject: Re: mode-line under vc
Date: Tue, 17 Jul 2007 17:06:44 +1200	[thread overview]
Message-ID: <18076.20068.12125.346765@kahikatea.snap.net.nz> (raw)
In-Reply-To: <jwvmyxvd4b9.fsf-monnier+emacs@gnu.org>

 > > In any case this binding (VC Menu) doesn't work if the window isn't already
 > > selected. You need to do something like mouse-major-mode-menu:
 > 
 > > (defun mouse-major-mode-menu (event &optional prefix)
 > >   "Pop up a mode-specific menu of mouse commands.
 > > Default to the Edit menu if the major mode doesn't define a menu."
 > >   ;; Switch to the window clicked on, because otherwise
 > >      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 > >   ;; the mode's commands may not make sense.
 > 
 > Are you sure it's needed?  It'd sound like a bug in keyboard.c (or somesuch)
 > if it is.

If you enable debugger on error, put a file under version control in a sole
buffer, do C-x C-b, click mouse-1 in the Buffer List buffer to select it, then
click mouse-1 the on the versrion control string in the mode-line and select
"Show Other Version" you get:

Debugger entered--Lisp error: (error "Buffer *Buffer List* is not associated with a file")
  signal(error ("Buffer *Buffer List* is not associated with a file"))
  error("Buffer %s is not associated with a file" "*Buffer List*")

So it doesn't work, whereas doing something similar on mode-name does work.

Therefore a change is needed, although it might be different to the one I
suggested.

-- 
Nick                                           http://www.inet.net.nz/~nickrob

  reply	other threads:[~2007-07-17  5:06 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-16 23:49 mode-line under vc Nick Roberts
2007-07-17  2:28 ` Dan Nicolaescu
2007-07-17  4:11   ` Nick Roberts
2007-07-17  4:49     ` Stefan Monnier
2007-07-17  5:06       ` Nick Roberts [this message]
2007-07-17  6:09     ` Dan Nicolaescu
2007-07-17  8:11 ` David Kastrup

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=18076.20068.12125.346765@kahikatea.snap.net.nz \
    --to=nickrob@snap.net.nz \
    --cc=dann@ics.uci.edu \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.