! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
0.6.91 ignores width and height, defaults to 80x23
Status:
RESOLVED: FIXED
Product:
Xfce4-terminal
Component:
General

Comments

Description Kevin Fenzi 2016-08-31 23:30:08 CEST
With 0.6.91 here it seems to ignore my width x height geometry. 

Doing a: 

xfce4-terminal --geometry=100x50

I see the terminal appear with what looks like the correct size, then it immedately resizes to 80x23. 

I have in prefs default geometry set to 100x50. 

Also, on login to my session, all my saved terminals resize to 80x23 as well. 

If I give it something gigantic like 500x500 it comes up fullscreen.
Comment 1 Igor editbugs 2016-09-01 15:42:40 CEST
Please describe your system.

I'm not seeing anything like that on Arch with Cinnamon, Fedora 24 with Gnome, Debian 8 with Xfce, all running X. The geometry is set very precisely.

I do see something similar on Fedora 24 with wayland - the initial geometry is incorrect, as well as the terminal window keeps increasing on opening and closing tabs. It seems that window resize gtk functions are functioning differently under wayland.
Comment 2 Kevin Fenzi 2016-09-01 22:22:11 CEST
Odd. My setup is Fedora rawhide with Xfce/X11 in this case. 

I'll try some things to isolate this more soon (newly created user to rule out a weird setting and downgrading back to 0.6.90 to confirm it works there).
Comment 3 Kevin Fenzi 2016-09-02 22:41:57 CEST
I still see this on a newly created user. 

I definitely see this go away when I downgrade to 0.6.90. ;( 

I can't think of anything else odd in my setup off hand. 
I guess I will ask the folks in the recent Fedora bugs to try my scratch build and see if they see it.
Comment 4 poma 2016-09-03 02:47:15 CEST
Reverted commit
https://git.xfce.org/apps/xfce4-terminal/commit/?id=39f966d
tentatively resolves the "geometry" issue with GTK+ 3.21.5,
on Rawhide / Fedora 26,
it also produces a known regression - "inconstant" rows number,
already well described in several bug reports, and resolved in a few commits.

---
 terminal/terminal-app.c    | 4 ++++
 terminal/terminal-screen.c | 3 +++
 2 files changed, 7 insertions(+)

diff --git a/terminal/terminal-app.c b/terminal/terminal-app.c
index 85b8af3..42fc860 100644
--- a/terminal/terminal-app.c
+++ b/terminal/terminal-app.c
@@ -813,8 +813,12 @@ terminal_app_open_window (TerminalApp        *app,
             {
               active_terminal = terminal_window_get_active (TERMINAL_WINDOW (window));
               if (G_LIKELY (active_terminal != NULL))
+#if GTK_CHECK_VERSION (3,21,5)
+                terminal_screen_set_size (active_terminal, width, height);
+#else
                 terminal_screen_force_resize_window (active_terminal, GTK_WINDOW (window),
                                                      width, height);
+#endif
             }
           if ((mask & XValue) && (mask & YValue))
             {
diff --git a/terminal/terminal-screen.c b/terminal/terminal-screen.c
index 23f4d92..8e7f56b 100644
--- a/terminal/terminal-screen.c
+++ b/terminal/terminal-screen.c
@@ -1820,6 +1820,9 @@ terminal_screen_force_resize_window (TerminalScreen *screen,
   glong          char_width;
   glong          char_height;
 
+#if GTK_CHECK_VERSION (3,21,5)
+  return;
+#endif
 
   terminal_return_if_fail (TERMINAL_IS_SCREEN (screen));
   terminal_return_if_fail (VTE_IS_TERMINAL (screen->terminal));
-- 
2.9.3
Comment 5 Kevin Fenzi 2016-09-03 18:22:17 CEST
Yep. That does indeed fix things here.
Comment 6 Igor editbugs 2016-09-05 07:06:39 CEST
Gtk is killing me...
Looks like I have to update Fedora in my VM in order to 3.21.5 and try to deal with it.
Thanks for the info, guys.
Comment 7 Igor editbugs 2016-09-05 18:24:36 CEST
Kevin, poma,

Could you please check if the following patch fixes it for you?
https://github.com/f2404/xfce4-terminal3/commit/e96ebb7927fd93b19ca744c011abbafe2fad62f1
Comment 8 Kevin Fenzi 2016-09-05 19:24:43 CEST
It does indeed appear to here.
Comment 9 poma 2016-09-06 00:05:10 CEST
GTK+ 3.21.5
&
GTK+ 3.20.9

All right!
Comment 11 Igor editbugs 2016-09-06 18:27:47 CEST
I have just released 0.6.92 containing the fix.
Comment 12 Kevin Fenzi 2016-09-06 19:17:30 CEST
Thanks much.
Comment 13 poma 2016-09-06 21:51:16 CEST
doc: refers to the proper gtk_minor_version 3.21.5 xfbz#12810

---
 NEWS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/NEWS b/NEWS
index c4b86f2..bb2c28c 100644
--- a/NEWS
+++ b/NEWS
@@ -1,7 +1,7 @@
 0.6.92
 ======
 - Fix geometry setting (default ot provided via the command line option)
-  for GTK 3.20.5 (bug #12810)
+  for GTK 3.21.5 (bug #12810)
 - Remove "Update utmp/wtmp records when command is launched" setting since
   the feature has been removed from vte (bug #12817)
 - Replace deprecated GtkTable with GtkGrid in Preferences
--
Comment 14 poma 2016-09-06 21:58:52 CEST
doc: typo fix

---
 NEWS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/NEWS b/NEWS
index bb2c28c..63ea301 100644
--- a/NEWS
+++ b/NEWS
@@ -1,6 +1,6 @@
 0.6.92
 ======
-- Fix geometry setting (default ot provided via the command line option)
+- Fix geometry setting (default or provided via the command line option)
   for GTK 3.21.5 (bug #12810)
 - Remove "Update utmp/wtmp records when command is launched" setting since
   the feature has been removed from vte (bug #12817)
--
Comment 15 poma 2016-09-06 22:00:04 CEST
Time for beauty sleep ;)
Comment 16 Igor editbugs 2016-09-07 09:14:31 CEST
Thanks poma!

Bug #12810

Reported by:
Kevin Fenzi
Reported on: 2016-08-31
Last modified on: 2016-09-07

People

CC List:
2 users

Version

Version:
0.6.91

Attachments

Additional information