unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Drew Adams" <drew.adams@oracle.com>
To: "'Chong Yidong'" <cyd@stupidchicken.com>
Cc: 1754@emacsbugs.donarmstrong.com
Subject: bug#1754: 23.0.60; tool-bar is shown with tool-bar-mode off
Date: Wed, 31 Dec 2008 21:17:19 -0800	[thread overview]
Message-ID: <006101c96bd0$387ac7b0$0200a8c0@us.oracle.com> (raw)
In-Reply-To: <877i5f914m.fsf@cyd.mit.edu>

> Arguably, Emacs 23 is now doing the right thing.  Your
> default-frame-alist specifies a tool-bar-lines frame parameter of 1,
> which means the tool-bar should be enabled in new frames.  If 
> you remove that frame parameter from your default-frame-alist, or set it 
> to 0, new frames will not show a tool-bar.
> 
> I am not 100% sure why previous version of Emacs did not show the
> tool-bar, but they probably ought to have.  Possibly it's 
> because we now set up the contents of the tool-bar at startup even if the 
> initial frame cannot or does not display it (a change required by
> the new multi-tty functionality).

Huh? Why is that the right thing? The default-frame-alist is set before turning
off tool-bar-mode. Turning off the mode should turn off the mode everywhere,
until you turn it back on - and it should change the value of
default-frame-alist - which it does, except in the scenario I gave. 

The default-frame-alist doesn't override the current setting of any modes. It
must not. It is simply a default setting. On the contrary - changing a mode such
as tool-bar-mode or menu-bar-mode should (and generally does) change the value
of default-frame-alist.

Tool-bar-mode is global and applies to all frames and future frames. That has
always been the case and should continue to be the case. What would be the point
of having such a mode if the original default frame alist overrode it?

Look at menu-bar-mode. It's exactly the same type of thing. But menu-bar-mode
acts normally, as it always has acted and as tool-bar-mode should also act.

And tool-bar-mode DOES act correctly, except in the scenario I gave, with
pop-up-frames non-nil, pop-up-frame-alist as I gave it, and a standalone
minibuffer frame. If you don't have all three of those, then turning off (or on)
tool-bar-mode, just like turning off (or on) menu-bar-mode, changes the value of
default-frame-alist to reflect the mode change, as it should.







  reply	other threads:[~2009-01-01  5:17 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-01  3:05 bug#1754: 23.0.60; tool-bar is shown with tool-bar-mode off Chong Yidong
2009-01-01  5:17 ` Drew Adams [this message]
2009-01-01  7:03   ` Chong Yidong
2009-01-01 21:28     ` Drew Adams
2009-01-01 21:51       ` Drew Adams
2009-01-02  0:01       ` Jason Rumney
2009-01-02  1:14         ` Drew Adams
2009-01-02  2:27           ` Drew Adams
2009-01-02  2:47             ` Drew Adams
2009-01-02 11:57             ` Jason Rumney
2009-01-02 14:50               ` martin rudalics
2009-01-01  7:36   ` Jason Rumney
2009-01-01 21:28     ` Drew Adams
2009-01-01 23:53       ` Jason Rumney
2009-01-02  1:14         ` Drew Adams
  -- strict thread matches above, loose matches on Subject: below --
2009-01-01 13:47 grischka
2009-01-01 13:51 ` Jason Rumney
2009-01-01 14:30   ` grischka
2008-12-31 22:15 Drew Adams
2009-01-01 15:24 ` martin rudalics
2009-01-01 15:53   ` Jason Rumney
2009-01-01 16:09     ` martin rudalics
2009-01-01 21:28   ` Drew Adams
2009-01-01 21:51     ` Drew Adams
2009-01-02  7:52       ` martin rudalics
2009-01-02  7:52     ` martin rudalics
2009-01-03  3:12       ` Stefan Monnier
2009-01-03 10:13         ` martin rudalics
2009-01-05  4:10           ` Stefan Monnier
2009-01-05  7:57             ` martin rudalics
2009-01-05 22:20               ` Stefan Monnier
2009-01-06 13:37                 ` martin rudalics
2009-01-09 19:37                 ` martin rudalics
2009-01-10  2:29                   ` Stefan Monnier
2009-01-10 10:04                     ` martin rudalics
2009-01-10 22:38                       ` Stefan Monnier
2009-01-11 11:22                         ` martin rudalics
2009-01-11 14:56                           ` Stefan Monnier
2009-01-11 17:14                             ` martin rudalics
2009-01-11 23:12                               ` Stefan Monnier
2009-01-12  9:58                                 ` martin rudalics
2009-01-12 15:23                                   ` Stefan Monnier
2009-01-12 16:00                                     ` martin rudalics
2009-01-12 18:29                                       ` Stefan Monnier
2009-01-12 19:33                                         ` Lennart Borgman
2009-01-12 20:57                                           ` Stefan Monnier
2009-01-12 21:59                                         ` Drew Adams
2009-01-14 14:20                                         ` martin rudalics
2009-01-17 18:33           ` Drew Adams
2009-01-09 19:37 ` martin rudalics
2009-01-09 20:21   ` Chong Yidong

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to='006101c96bd0$387ac7b0$0200a8c0@us.oracle.com' \
    --to=drew.adams@oracle.com \
    --cc=1754@emacsbugs.donarmstrong.com \
    --cc=cyd@stupidchicken.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 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).