GNU bug report logs -
#7533
24.0.50; `dired-mark-pop-up': delete frame afterwards if `pop-up-frames'
Previous Next
Reported by: "Drew Adams" <drew.adams <at> oracle.com>
Date: Thu, 2 Dec 2010 18:32:01 UTC
Severity: minor
Found in version 24.0.50
Done: martin rudalics <rudalics <at> gmx.at>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
Your message dated Wed, 03 Oct 2012 11:11:52 +0200
with message-id <506C0158.6040001 <at> gmx.at>
and subject line Re: bug#7533: 24.0.50; `dired-mark-pop-up': delete frame afterwards if `pop-up-frames'
has caused the debbugs.gnu.org bug report #7533,
regarding 24.0.50; `dired-mark-pop-up': delete frame afterwards if `pop-up-frames'
to be marked as done.
(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)
--
7533: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=7533
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
This is another one of those annoyances that come from not testing with
non-nil `pop-up-frames' (just a guess).
emacs -Q
M-x set-variable RET pop-up-frames RET t RET
In Dired, flag several files for deletion, then hit `x'.
Answer `yes' or `no'.
When you hit `x' buffer *Deletions* pops up in its own frame, as it
should. When you type `yes' or `no', the frame should disappear. It
should be deleted. The last part of `dired-mark-pop-up' should be
something like this:
(save-window-excursion
(dired-pop-to-buffer bufname)
(apply function args)
(if (one-window-p) (delete-frame) (delete-window))) ; <===== NEW
If you don't think the `delete-window' is needed, then just use
(when (one-window-p) (delete-frame)).
The important thing, for me, is that the frame that was created just to
show the files that will be deleted (or whatever) goes away. It should
be only a _temporary_ frame because its only raison d'etre is as part of
the deletion etc. _dialog_.
[Martin will explain that a different test from `one-window-p' is more
appropriate. ;-) IIRC, he generally prefers something like this to
(one-window-p win): (eq win (frame-root-window (window-frame win))).]
In GNU Emacs 24.0.50.1 (i386-mingw-nt5.1.2600)
of 2010-11-30 on 3249CTO
Windowing system distributor `Microsoft Corp.', version 5.1.2600
configured using `configure --with-gcc (4.4) --no-opt --cflags
-Ic:/imagesupport/include'
[Message part 3 (message/rfc822, inline)]
>> This should have been fixed in revision 109863 on Emacs trunk.
Bug closed.
martin
This bug report was last modified 12 years and 311 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.