From: Vivek Dasmohapatra <vivek@etla.org>
To: martin rudalics <rudalics@gmx.at>
Cc: 22000@debbugs.gnu.org, David Engster <deng@randomsample.de>
Subject: bug#22000: Patch addressing the menu-bar frame-resize interaction
Date: Tue, 17 Jul 2018 20:02:12 +0100 (BST) [thread overview]
Message-ID: <alpine.DEB.2.02.1807172000440.921@platypus.pepperfish.net> (raw)
In-Reply-To: <alpine.DEB.2.02.1807171139470.921@platypus.pepperfish.net>
[-- Attachment #1: Type: TEXT/PLAIN, Size: 255 bytes --]
>> The height of the initial window goes down from 35 lines (630 pixels)
>> to 33 lines (595 pixels).
This patch (added on top of the previous one) should fix the frame
size calculation.
Still looking into whether there's a purely truncating solution.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: TEXT/x-diff; name=0002-GTK3-correct-frame-height-calculation-with-scrollabl.patch, Size: 1483 bytes --]
From eaae86389d2862dc10804f45161f07cb475b49a0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Vivek=20Das=C2=A0Mohapatra?= <vivek@collabora.com>
Date: Tue, 17 Jul 2018 19:53:42 +0100
Subject: [PATCH 2/2] GTK3 - correct frame height calculation with scrollable
menu bars
The frame height calculation needs to be done based on the new
scrollable window that wraps the menu bars to be accurate.
---
src/gtkutil.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/gtkutil.c b/src/gtkutil.c
index d16274f6ab..dc78976d22 100644
--- a/src/gtkutil.c
+++ b/src/gtkutil.c
@@ -3445,7 +3445,11 @@ menubar_map_cb (GtkWidget *w, gpointer user_data)
{
GtkRequisition req;
struct frame *f = user_data;
- gtk_widget_get_preferred_size (w, NULL, &req);
+ struct x_output *x = f->output_data.x;
+
+ /* Use the menubar viewport for size if there is one: */
+ gtk_widget_get_preferred_size (x->menubar_viewport ?: w, NULL, &req);
+
if (FRAME_MENUBAR_HEIGHT (f) != req.height)
{
FRAME_MENUBAR_HEIGHT (f) = req.height;
@@ -3498,7 +3502,7 @@ xg_update_frame_menubar (struct frame *f)
g_signal_connect (x->menubar_widget, "map", G_CALLBACK (menubar_map_cb), f);
gtk_widget_show_all (x->menubar_viewport);
- gtk_widget_get_preferred_size (x->menubar_widget, NULL, &req);
+ gtk_widget_get_preferred_size (x->menubar_viewport, NULL, &req);
if (FRAME_MENUBAR_HEIGHT (f) != req.height)
{
--
2.11.0
next prev parent reply other threads:[~2018-07-17 19:02 UTC|newest]
Thread overview: 97+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-23 20:55 bug#22000: 25.0.50; Running dired changes frame width, gtk_distribute_natural_allocation throws assertion David Engster
2015-11-24 8:28 ` martin rudalics
2015-11-24 16:48 ` David Engster
2015-11-24 19:26 ` martin rudalics
2015-11-25 16:15 ` David Engster
2015-11-25 17:48 ` martin rudalics
2015-11-25 19:00 ` David Engster
2015-11-26 8:22 ` martin rudalics
2018-07-15 18:09 ` bug#22000: Patch addressing the menu-bar frame-resize interaction Vivek Dasmohapatra
2018-07-16 7:28 ` martin rudalics
2018-07-16 9:46 ` Vivek Dasmohapatra
2018-07-16 19:58 ` Vivek Dasmohapatra
2018-07-17 7:48 ` martin rudalics
2018-07-17 13:45 ` Vivek Dasmohapatra
2018-07-17 19:02 ` Vivek Dasmohapatra [this message]
2018-07-18 7:01 ` martin rudalics
2018-07-18 7:07 ` martin rudalics
2018-07-18 10:39 ` Vivek Dasmohapatra
2018-07-19 8:19 ` martin rudalics
2018-07-19 12:04 ` Vivek Dasmohapatra
2018-07-20 8:14 ` martin rudalics
2018-07-20 9:21 ` Vivek Dasmohapatra
2018-07-20 12:34 ` martin rudalics
2018-07-20 17:44 ` Vivek Dasmohapatra
2018-07-21 7:43 ` martin rudalics
2018-07-21 13:24 ` Vivek Dasmohapatra
2018-07-22 7:24 ` martin rudalics
2018-07-22 12:29 ` Vivek Dasmohapatra
2018-07-23 6:50 ` martin rudalics
2018-10-11 13:05 ` Vivek Dasmohapatra
2018-10-11 18:17 ` martin rudalics
2018-10-11 18:27 ` martin rudalics
2018-10-11 18:48 ` Vivek Dasmohapatra
2018-10-11 20:51 ` Vivek Dasmohapatra
2018-10-12 8:44 ` martin rudalics
2018-10-12 12:47 ` Vivek Dasmohapatra
2018-10-12 18:12 ` martin rudalics
2018-10-12 18:25 ` Vivek Dasmohapatra
2018-10-13 8:20 ` martin rudalics
2018-10-13 10:03 ` Vivek Dasmohapatra
2018-10-15 13:57 ` Vivek Dasmohapatra
2018-10-15 18:23 ` martin rudalics
2018-10-16 1:19 ` Vivek Dasmohapatra
2018-10-16 8:47 ` martin rudalics
2018-10-16 18:58 ` Vivek Dasmohapatra
2018-10-17 7:29 ` martin rudalics
2018-10-18 1:02 ` Vivek Dasmohapatra
2018-10-18 8:06 ` martin rudalics
2018-10-18 12:23 ` Vivek Dasmohapatra
2018-10-18 12:48 ` Robert Pluim
2018-10-18 13:24 ` Vivek Dasmohapatra
2018-10-18 13:46 ` Robert Pluim
2018-10-18 13:56 ` Eli Zaretskii
2018-10-18 17:08 ` Vivek Dasmohapatra
2018-10-18 18:16 ` Eli Zaretskii
2018-10-18 18:34 ` Vivek Dasmohapatra
2018-10-18 13:51 ` Stephen Berman
2018-10-18 14:31 ` Robert Pluim
2018-10-18 13:51 ` Eli Zaretskii
2018-10-18 17:26 ` Vivek Dasmohapatra
2018-10-18 18:20 ` Eli Zaretskii
2018-10-18 18:32 ` Vivek Dasmohapatra
2018-10-18 19:55 ` Drew Adams
2018-10-19 6:23 ` Eli Zaretskii
2018-10-18 13:34 ` Eli Zaretskii
2018-10-18 14:22 ` Robert Pluim
2018-10-18 16:41 ` martin rudalics
2018-10-19 7:41 ` Robert Pluim
2018-10-19 8:34 ` martin rudalics
2018-10-19 9:14 ` Robert Pluim
2018-10-19 13:44 ` Robert Pluim
2018-10-19 17:57 ` martin rudalics
2018-10-23 10:07 ` Robert Pluim
2018-10-23 13:45 ` martin rudalics
2018-10-23 14:02 ` Robert Pluim
2018-10-23 18:18 ` martin rudalics
2018-10-23 19:19 ` Robert Pluim
2018-10-24 9:44 ` martin rudalics
2018-10-24 11:52 ` Robert Pluim
2018-10-24 12:18 ` Vivek Dasmohapatra
2018-10-19 14:17 ` Stephen Berman
2018-10-19 14:44 ` Vivek Dasmohapatra
2018-10-19 16:21 ` Stephen Berman
2018-10-21 15:44 ` Vivek Dasmohapatra
2018-10-21 15:50 ` Vivek Dasmohapatra
2018-10-22 17:11 ` Vivek Dasmohapatra
2018-10-19 17:57 ` martin rudalics
2018-10-18 16:40 ` martin rudalics
2018-10-18 17:07 ` Eli Zaretskii
2018-10-18 17:13 ` Vivek Dasmohapatra
2018-10-19 7:26 ` Robert Pluim
2018-10-19 8:23 ` Eli Zaretskii
2018-10-19 9:25 ` Robert Pluim
2018-10-12 18:34 ` Vivek Dasmohapatra
2018-10-12 18:16 ` Vivek Dasmohapatra
[not found] <<87k2p8h1vn.fsf@isaac.fritz.box>
[not found] ` <<5B52E425.8010608@gmx.at>
[not found] ` <<alpine.DEB.2.02.1807211421040.921@platypus.pepperfish.net>
[not found] ` <<5B543148.1010004@gmx.at>
[not found] ` <<alpine.DEB.2.02.1807221324380.921@platypus.pepperfish.net>
[not found] ` <<5B557ACA.4020106@gmx.at>
[not found] ` <<alpine.DEB.2.02.1810111400480.5980@platypus.pepperfish.net>
[not found] ` <<5BBF93CF.4060301@gmx.at>
[not found] ` <<alpine.DEB.2.02.1810112148100.5980@platypus.pepperfish.net>
[not found] ` <<5BC05EEB.9010609@gmx.at>
[not found] ` <<alpine.DEB.2.02.1810121316230.5980@platypus.pepperfish.net>
[not found] ` <<5BC0E405.90805@gmx.at>
[not found] ` <<alpine.DEB.2.02.1810121917570.5980@platypus.pepperfish.net>
[not found] ` <<5BC1AAE2.7070808@gmx.at>
[not found] ` <<alpine.DEB.2.02.1810151455060.19047@platypus.pepperfish.net>
[not found] ` <<5BC4DB0E.3050501@gmx.at>
[not found] ` <<alpine.DEB.2.02.1810161954120.19047@platypus.pepperfish.net>
[not found] ` <<5BC6E4F2.2030607@gmx.at>
[not found] ` <<alpine.DEB.2.02.1810180200180.19047@platypus.pepperfish.net>
[not found] ` <<5BC83F03.4050006@gmx.at>
[not found] ` <<alpine.DEB.2.02.1810181321230.19047@platypus.pepperfish.net>
[not found] ` <<83o9brqs6e.fsf@gnu.org>
[not found] ` <<alpine.DEB.2.02.1810181813500.19047@platypus.pepperfish.net>
[not found] ` <<83bm7rqfpo.fsf@gnu.org>
[not found] ` <<766dc2b9-7931-48b2-b2f2-6b57d7ca85dd@default>
[not found] ` <<835zxyqwtg.fsf@gnu.org>
2018-10-20 23:58 ` Drew Adams
2018-10-21 2:39 ` Eli Zaretskii
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=alpine.DEB.2.02.1807172000440.921@platypus.pepperfish.net \
--to=vivek@etla.org \
--cc=22000@debbugs.gnu.org \
--cc=deng@randomsample.de \
--cc=rudalics@gmx.at \
/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.