From: martin rudalics <rudalics@gmx.at>
To: Drew Adams <drew.adams@oracle.com>
Cc: 8856@debbugs.gnu.org
Subject: bug#8856: 24.0.50; regression: special-display-frame is no longer dedicated
Date: Sat, 25 Jun 2011 19:00:13 +0200 [thread overview]
Message-ID: <4E06141D.80302@gmx.at> (raw)
In-Reply-To: <E6F00900C81642B19985B8E7C1F85D61@us.oracle.com>
>> I downloaded your Icicles tarball and tried with throw-10. Everything
>> seems to work as expected including focus redirection and resurrection
>> of an iconified Completions frame.
>
> There should be no iconfied Completions frame in the test I gave you.
When I type foo and eventually get the sole completion footnote-mode the
Completions frame iconifies automatically. I didn't check why that
happens, shouldn't it? When I then type M-x f TAB again it gets orderly
resurrected.
> The problem arises only when the Completions frame is newly _created_, as I
> tried to explain. If you use `M-x f TAB o' (recipe) after the frame has already
> been created then the input focus is not a problem.
>
> If you have the frame already,
The Completions frame? It pops up automatically when I hit TAB and
redirects focus to the Minibuffer frame.
> then please delete it and try `M-x f TAB o'
> again.
>
>> So if you don't use any additional strings the problem seems
>> fixed. I can't tell why it doesn't work on your machine.
>
> As I said, it's possible that this is Windows-specific. Windows is I think
> (thought) a bit peculiar when it comes to its auto-selecting a new frame - but
> you said that most window mgrs do that now. I still wonder if this isn't
> Windows-specific, since you cannot reproduce it and it is systematic for me when
> I follow the same recipe.
But I test this on Windows XP.
>> Maybe some .el files have to be recompiled in order to work
>> with my later fixes
>
> Do you mean my .el or .el from Emacs sources? In the files I provided for the
> test, please use only .el, no .elc.
From the Emacs sources. Maybe some completions-related stuff.
>> so it's likely best to wait till Sean uploads the new binaries
>> (IIRC this happens every Monday). As soon as you tried with
>> the new binaries tell me what you get.
>
> I'll certainly do that.
OK.
> But what about that `display-buffer' call in
> `icicle-display-candidates-in-Completions'? Can you suggest a change for it?
> Does that code look like it _should_ work after your changes, or should it be
> changed?
It should work vacuously. But I'd like to know why you needed it
earlier. Earlier Emacsen had `display-buffer' do
((let ((frames (or frame
here frame is the value "0" you provide as argument
(and (or use-pop-up-frames
display-buffer-reuse-frames
(not (last-nonminibuffer-frame)))
0)
and the "0" you see in the previous line means exactly the same. Any
visible or iconified frame provided you use either `pop-up-frames' or
`display-buffer-reuse-frames'. And I suppose it doesn't make sense to
use Icicles with these set to nil. Now if you insisted on searching
only visible frames here I would see a problem ...
(last-nonminibuffer-frame))))
(setq window-to-use
(catch 'found
;; Search frames for a window displaying BUFFER. Return
;; the selected window only if we are allowed to do so.
(dolist (window (get-buffer-window-list buffer 'nomini frames))
(when (or can-use-selected-window
(not (eq (selected-window) window)))
(throw 'found window))))))
;; The buffer is already displayed in some window; use that.
(window--display-buffer-1 window-to-use))
I can easily resurrect the old semantics of that argument for the few
values t, 0, visible and a live frame but I would first like to see a
real use case for it.
> And you mentioned the `save-selected-window' around that `display-buffer' call.
> Does that need to be changed somehow?
I'm not quite sure what you expect it to do. IIUC you want the
Completions window selected with focus redirected to the Minibuffer
window. Now which window is the one whose selection you want to save
here? Anyway, it shouldn't harm so leave it alone.
martin
next prev parent reply other threads:[~2011-06-25 17:00 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-13 16:08 bug#8851: 24.0.50; regression: special-display-frame is no longer dedicated Drew Adams
2011-06-13 18:00 ` martin rudalics
2011-06-13 18:41 ` Drew Adams
2011-06-14 9:15 ` martin rudalics
2011-06-14 20:36 ` Drew Adams
[not found] ` <4DFB6BBF.3080504@gmx.at>
2011-06-17 15:51 ` Drew Adams
2011-06-17 16:22 ` bug#8856: " martin rudalics
2011-06-17 17:48 ` Drew Adams
2011-06-19 17:29 ` Drew Adams
2011-06-20 3:04 ` Stefan Monnier
2011-06-17 17:48 ` bug#8856: " Drew Adams
2011-06-19 13:26 ` martin rudalics
2011-06-19 14:31 ` bug#8856: 24.0.50;regression: `special-display-frame' broken Drew Adams
2011-06-19 18:50 ` Chong Yidong
2011-06-19 18:54 ` Drew Adams
[not found] ` <4DFE09A7.10500@gmx.at>
2011-06-19 14:43 ` bug#8856: 24.0.50; regression: special-display-frame is no longer dedicated Drew Adams
2011-06-19 17:26 ` Drew Adams
2011-06-19 18:40 ` martin rudalics
2011-06-19 19:34 ` bug#8856: 24.0.50; regression: `special-display-popup-frame' broken Drew Adams
2011-06-19 19:52 ` Drew Adams
2011-06-20 9:46 ` bug#8856: 24.0.50; regression: special-display-frame is no longer dedicated martin rudalics
2011-06-20 13:01 ` Drew Adams
[not found] ` <4E00C54C.5080108@gmx.at>
2011-06-21 18:10 ` Drew Adams
2011-06-22 0:13 ` Drew Adams
2011-06-22 0:14 ` Drew Adams
2011-06-22 0:15 ` Drew Adams
2011-06-23 16:45 ` Drew Adams
[not found] ` <4E033CBA.1050700@gmx.at>
[not found] ` <DB9EDF1C454F42A0BC437F0E0AEE6CA2@us.oracle.com>
[not found] ` <4E037708.2000205@gmx.at>
2011-06-23 22:06 ` Drew Adams
2011-06-24 8:53 ` martin rudalics
2011-06-24 21:21 ` Drew Adams
2011-06-25 14:15 ` martin rudalics
2011-06-25 14:52 ` Drew Adams
[not found] ` <8A3D5626004B4 945A624B69463A0B849@us.oracle.com>
2011-06-25 15:04 ` Drew Adams
2011-06-25 15:57 ` martin rudalics
2011-06-25 16:15 ` Drew Adams
2011-06-25 17:00 ` martin rudalics [this message]
2011-06-25 17:48 ` Drew Adams
2011-06-26 13:50 ` martin rudalics
2011-06-26 14:56 ` Drew Adams
[not found] ` <0721F495F4A441529FCB91280D284E42@us.oracle.com! >
2011-06-26 15:15 ` Drew Adams
2011-06-26 15:54 ` martin rudalics
2011-06-26 16:06 ` Drew Adams
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=4E06141D.80302@gmx.at \
--to=rudalics@gmx.at \
--cc=8856@debbugs.gnu.org \
--cc=drew.adams@oracle.com \
/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.