all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#25: frame parameter menu-bar-lines changes height of frame
@ 2008-06-10 19:06 Stefan Monnier
  2008-06-10 19:42 ` Drew Adams
  0 siblings, 1 reply; 8+ messages in thread
From: Stefan Monnier @ 2008-06-10 19:06 UTC (permalink / raw)
  To: 25

severity 25 wishlist
thanks

The menu-bar currently isn't counted as part of the "text area" (whose
size is described by the height and width frame parameters), whereas the
tool-bar is.

I agree that the way the toolbar behaves in this respect is preferable,
but because Emacs still counts sizes in multiple of lines and columns,
it also has drawbacks (typically a few pixels wasted between the
tool-bar and the actual text).

Hopefully, we'll fix this at some point.






^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#25: frame parameter menu-bar-lines changes height of frame
  2008-06-10 19:06 bug#25: frame parameter menu-bar-lines changes height of frame Stefan Monnier
@ 2008-06-10 19:42 ` Drew Adams
  2008-06-10 20:32   ` Jason Rumney
  0 siblings, 1 reply; 8+ messages in thread
From: Drew Adams @ 2008-06-10 19:42 UTC (permalink / raw)
  To: 'Stefan Monnier', 25

Too bad. Apparently people have been agreeing that this should be fixed since it
was first discussed when Gerd was in charge - years.

Previously, the reason given for not fixing it at the time was that the release
was imminent and we didn't want to risk the change.

Now, this has simply been downgraded to "wishlist" - poof!  Dommage.

> From: Stefan Monnier Sent: Tuesday, June 10, 2008 12:07 PM
> severity 25 wishlist
> thanks
> 
> The menu-bar currently isn't counted as part of the "text area" (whose
> size is described by the height and width frame parameters), 
> whereas the tool-bar is.
> 
> I agree that the way the toolbar behaves in this respect is 
> preferable,
> but because Emacs still counts sizes in multiple of lines and columns,
> it also has drawbacks (typically a few pixels wasted between the
> tool-bar and the actual text).
> 
> Hopefully, we'll fix this at some point.







^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: bug#25: frame parameter menu-bar-lines changes height of frame
  2008-06-10 19:42 ` Drew Adams
@ 2008-06-10 20:32   ` Jason Rumney
  2008-06-10 20:52     ` Drew Adams
  0 siblings, 1 reply; 8+ messages in thread
From: Jason Rumney @ 2008-06-10 20:32 UTC (permalink / raw)
  To: Drew Adams; +Cc: Emacs Devel

Drew Adams wrote:
> Now, this has simply been downgraded to "wishlist" - poof!  Dommage.
>   

It is not a downgrade, it is a categorization.




^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: bug#25: frame parameter menu-bar-lines changes height of frame
  2008-06-10 20:32   ` Jason Rumney
@ 2008-06-10 20:52     ` Drew Adams
  2008-06-10 21:20       ` Stefan Monnier
  0 siblings, 1 reply; 8+ messages in thread
From: Drew Adams @ 2008-06-10 20:52 UTC (permalink / raw)
  To: 'Jason Rumney'; +Cc: 'Emacs Devel'

> > Now, this has simply been downgraded to "wishlist" - poof!  Dommage.
> 
> It is not a downgrade, it is a categorization.

Strictly speaking, since this is the first time it has been categorized (the
categories are new), you are correct.

However, relative to what was discussed about this in the past, it is a
downgrade, IMO. It was not previously characterized as "wishlist", but as
something that should be done after the imminent release.





^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: bug#25: frame parameter menu-bar-lines changes height of frame
  2008-06-10 20:52     ` Drew Adams
@ 2008-06-10 21:20       ` Stefan Monnier
  0 siblings, 0 replies; 8+ messages in thread
From: Stefan Monnier @ 2008-06-10 21:20 UTC (permalink / raw)
  To: Drew Adams; +Cc: 'Emacs Devel', 'Jason Rumney'

>> > Now, this has simply been downgraded to "wishlist" - poof!  Dommage.
>> It is not a downgrade, it is a categorization.
> Strictly speaking, since this is the first time it has been categorized (the
> categories are new), you are correct.

> However, relative to what was discussed about this in the past, it is a
> downgrade, IMO. It was not previously characterized as "wishlist", but as
> something that should be done after the imminent release.

Yes, it's something that should be done as soon as possible.
But nobody has stepped forward to do it.  And nobody has made a case
that this is an important bug to fix.  So it's a wishlist item, waiting
for a generous soul.


        Stefan





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#25: frame parameter menu-bar-lines changes height of frame
  2007-07-24  4:28 Drew Adams
@ 2014-12-31 18:36 ` martin rudalics
  2014-12-31 18:53   ` Drew Adams
  0 siblings, 1 reply; 8+ messages in thread
From: martin rudalics @ 2014-12-31 18:36 UTC (permalink / raw)
  To: drew.adams, 25-done

 > > emacs -Q
 > >
 > > (make-frame '((width . 70)(height . 50)))
 > > (modify-frame-parameters (selected-frame) '((menu-bar-lines . 0)))
 > > (frame-parameters)
 > > (modify-frame-parameters (selected-frame) '((menu-bar-lines . 1)))
 > > (frame-parameters)
 > >
 > > Adding and removing a menu-bar line changes the visible height of the
 > > frame (incorrect), but it does not change the `height' frame
 > > parameter. The correct behavior is that observed for `tool-bar-lines':
 > > neither the visible frame height nor the `height' parameter is
 > > changed.
 >
 > Use (assq 'height (frame-parameters)) to see that the `height'
 > parameter does not change.

This can now be handled on trunk/master by adding 'menu-bar-lines' to
`frame-inhibit-implied-resize'.  Bug closed.

Thanks, martin





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#25: frame parameter menu-bar-lines changes height of frame
  2014-12-31 18:36 ` bug#25: " martin rudalics
@ 2014-12-31 18:53   ` Drew Adams
  2015-01-01 10:25     ` martin rudalics
  0 siblings, 1 reply; 8+ messages in thread
From: Drew Adams @ 2014-12-31 18:53 UTC (permalink / raw)
  To: martin rudalics, 25-done

> This can now be handled on trunk/master by adding 'menu-bar-lines'
> to `frame-inhibit-implied-resize'.  Bug closed.

I don't have a version with the fix yet.  (I have been unable to use
any Emacs versions since mid-October because they break the use of
frames in various ways, but I will look into those problems and report
them as I narrow things down.)

I gather that `frame-inhibit-implied-resize' is a user option.  It's
great that users can now control the behavior, but I wonder how that
helps code and how it fixes the bug reported here.  The point of
this bug was that `menu-bar-lines' should behave like `tool-bar-lines'
does: adding or removing a menu bar, or increasing/decreasing the
number of its lines, should not change the visible frame height.

If that behavior is only optional, and decided by users, then it
sounds like things are not the same as for `tool-bar-lines' and
that code cannot depend on the behavior being the same.

Anyway, I won't judge the fix until I can check it out.  And my
guess is that it is an improvement.  Thanks for working on this,
in any case.





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#25: frame parameter menu-bar-lines changes height of frame
  2014-12-31 18:53   ` Drew Adams
@ 2015-01-01 10:25     ` martin rudalics
  0 siblings, 0 replies; 8+ messages in thread
From: martin rudalics @ 2015-01-01 10:25 UTC (permalink / raw)
  To: Drew Adams, 25-done

 > I gather that `frame-inhibit-implied-resize' is a user option.  It's
 > great that users can now control the behavior, but I wonder how that
 > helps code and how it fixes the bug reported here.

Please refrain from calling this a bug.  It's a behavior you dislike and
we can try to modify it in your sense.  But let's do that in a minimally
invasive fashion so other users don't get annoyed by our modifications.

 > The point of
 > this bug was that `menu-bar-lines' should behave like `tool-bar-lines'
 > does: adding or removing a menu bar, or increasing/decreasing the
 > number of its lines, should not change the visible frame height.

For graphical displays now the following hold:

If a user adds both `menu-bar-lines' and `tool-bar-lines' to
`frame-inhibit-implied-resize' or the latter is t or the frame is either
maximized or fullscreen, then toggling either `menu-bar-mode' or
`tool-bar-mode' conceptually do not change the visible frame height.

If neither `menu-bar-lines' and `tool-bar-lines' are members of
`frame-inhibit-implied-resize' and the latter is not t and the frame is
neither maximized nor fullscreen, then toggling either `menu-bar-mode'
or `tool-bar-mode' conceptually do change the visible frame height.

And this holds for Gtk just as for Windows.

 > If that behavior is only optional, and decided by users, then it
 > sounds like things are not the same as for `tool-bar-lines' and
 > that code cannot depend on the behavior being the same.

Code can always read the value of `frame-inhibit-implied-resize' as well
as the maximized/fullscreen status of a frame.

martin





^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2015-01-01 10:25 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-06-10 19:06 bug#25: frame parameter menu-bar-lines changes height of frame Stefan Monnier
2008-06-10 19:42 ` Drew Adams
2008-06-10 20:32   ` Jason Rumney
2008-06-10 20:52     ` Drew Adams
2008-06-10 21:20       ` Stefan Monnier
  -- strict thread matches above, loose matches on Subject: below --
2007-07-24  4:28 Drew Adams
2014-12-31 18:36 ` bug#25: " martin rudalics
2014-12-31 18:53   ` Drew Adams
2015-01-01 10:25     ` martin rudalics

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.