* bug#9699: frame-auto-hide-function
@ 2011-10-07 22:10 Glenn Morris
2011-10-07 22:13 ` Drew Adams
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Glenn Morris @ 2011-10-07 22:10 UTC (permalink / raw)
To: 9699
Package: emacs
Version: 24.0.90
frame-auto-hide-function has no NEWS entry, and is missing a :version tag.
It needs a NEWS entry (which should contain some mention of "iconify")
because it changes existing behaviour.
Example: I like my diff buffers in a separate frame, so I use:
emacs -Q --eval "(setq special-display-regexps '(\".*diff\\*\"))"
C-x C-f /path/to/modified/file/under/vcs
C-x v =
-> diff buffer pops up in new frame
Now using "C-x k" to kill the diff buffer iconifies the associated
frame. It used to just delete it (which I think was better, but YMMV).
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#9699: frame-auto-hide-function
2011-10-07 22:10 bug#9699: frame-auto-hide-function Glenn Morris
@ 2011-10-07 22:13 ` Drew Adams
2011-10-08 13:23 ` martin rudalics
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Drew Adams @ 2011-10-07 22:13 UTC (permalink / raw)
To: 'Glenn Morris', 9699
> Example: I like my diff buffers in a separate frame, so I use:
> emacs -Q --eval "(setq special-display-regexps '(\".*diff\\*\"))"
> C-x C-f /path/to/modified/file/under/vcs
> C-x v = -> diff buffer pops up in new frame
>
> Now using "C-x k" to kill the diff buffer iconifies the associated
> frame. It used to just delete it (which I think was better, but YMMV).
Ouch! Someone (TM) is (still) on the wrong track...
I agree: delete, don't iconify.
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#9699: frame-auto-hide-function
2011-10-07 22:10 bug#9699: frame-auto-hide-function Glenn Morris
2011-10-07 22:13 ` Drew Adams
@ 2011-10-08 13:23 ` martin rudalics
2011-10-08 17:59 ` Glenn Morris
2011-10-08 14:08 ` martin rudalics
2011-10-09 12:35 ` martin rudalics
3 siblings, 1 reply; 6+ messages in thread
From: martin rudalics @ 2011-10-08 13:23 UTC (permalink / raw)
To: Glenn Morris; +Cc: 9699
> frame-auto-hide-function has no NEWS entry, and is missing a :version tag.
>
> It needs a NEWS entry (which should contain some mention of "iconify")
> because it changes existing behaviour.
Will be done as soon as I have fixed some bugs ...
> Example: I like my diff buffers in a separate frame, so I use:
>
> emacs -Q --eval "(setq special-display-regexps '(\".*diff\\*\"))"
>
> C-x C-f /path/to/modified/file/under/vcs
> C-x v =
> -> diff buffer pops up in new frame
>
> Now using "C-x k" to kill the diff buffer iconifies the associated
> frame. It used to just delete it (which I think was better, but YMMV).
... like this - the argument KILL of `window--delete' is unused.
martin, who feels sorry for the inconvenience
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#9699: frame-auto-hide-function
2011-10-07 22:10 bug#9699: frame-auto-hide-function Glenn Morris
2011-10-07 22:13 ` Drew Adams
2011-10-08 13:23 ` martin rudalics
@ 2011-10-08 14:08 ` martin rudalics
2011-10-09 12:35 ` martin rudalics
3 siblings, 0 replies; 6+ messages in thread
From: martin rudalics @ 2011-10-08 14:08 UTC (permalink / raw)
To: Glenn Morris; +Cc: 9699
> Example: I like my diff buffers in a separate frame, so I use:
>
> emacs -Q --eval "(setq special-display-regexps '(\".*diff\\*\"))"
>
> C-x C-f /path/to/modified/file/under/vcs
> C-x v =
> -> diff buffer pops up in new frame
>
> Now using "C-x k" to kill the diff buffer iconifies the associated
> frame. It used to just delete it (which I think was better, but YMMV).
Should be fixed now.
martin
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#9699: frame-auto-hide-function
2011-10-08 13:23 ` martin rudalics
@ 2011-10-08 17:59 ` Glenn Morris
0 siblings, 0 replies; 6+ messages in thread
From: Glenn Morris @ 2011-10-08 17:59 UTC (permalink / raw)
To: martin rudalics; +Cc: 9699
martin rudalics wrote:
>> Now using "C-x k" to kill the diff buffer iconifies the associated
>> frame. It used to just delete it (which I think was better, but YMMV).
>
> ... like this - the argument KILL of `window--delete' is unused.
>
> martin, who feels sorry for the inconvenience
No problem! Onwards and upwards...
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#9699: frame-auto-hide-function
2011-10-07 22:10 bug#9699: frame-auto-hide-function Glenn Morris
` (2 preceding siblings ...)
2011-10-08 14:08 ` martin rudalics
@ 2011-10-09 12:35 ` martin rudalics
3 siblings, 0 replies; 6+ messages in thread
From: martin rudalics @ 2011-10-09 12:35 UTC (permalink / raw)
To: 9699-done
> frame-auto-hide-function has no NEWS entry, and is missing a :version tag.
>
> It needs a NEWS entry (which should contain some mention of "iconify")
> because it changes existing behaviour.
>
> Example: I like my diff buffers in a separate frame, so I use:
>
> emacs -Q --eval "(setq special-display-regexps '(\".*diff\\*\"))"
>
> C-x C-f /path/to/modified/file/under/vcs
> C-x v =
> -> diff buffer pops up in new frame
>
> Now using "C-x k" to kill the diff buffer iconifies the associated
> frame. It used to just delete it (which I think was better, but YMMV).
Should be all fixed now. Closing.
Thanks, martin
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-10-09 12:35 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-07 22:10 bug#9699: frame-auto-hide-function Glenn Morris
2011-10-07 22:13 ` Drew Adams
2011-10-08 13:23 ` martin rudalics
2011-10-08 17:59 ` Glenn Morris
2011-10-08 14:08 ` martin rudalics
2011-10-09 12:35 ` martin rudalics
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).