After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 324190 - yelp window is maximized but contents of window are not
yelp window is maximized but contents of window are not
Status: RESOLVED FIXED
Product: yelp
Classification: Applications
Component: General
2.13.x
Other Linux
: Normal normal
: ---
Assigned To: Yelp maintainers
Yelp maintainers
: 326803 (view as bug list)
Depends on:
Blocks: 331284
 
 
Reported: 2005-12-15 16:47 UTC by Sebastien Bacher
Modified: 2009-09-22 01:48 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
"Before" (306.23 KB, image/png)
2006-01-02 19:32 UTC, Vincent Trouilliez
  Details
"After" (85.22 KB, image/png)
2006-01-02 19:33 UTC, Vincent Trouilliez
  Details
Proposed fix (2.19 KB, patch)
2006-01-21 16:36 UTC, Don Scorgie
none Details | Review

Description Sebastien Bacher 2005-12-15 16:47:22 UTC
This bug has been opened here: http://bugzilla.ubuntu.com/show_bug.cgi?id=21050

"This is a fresh load of Dapper Flight2 with updates. The first time that the
window is maximized the window expands but contents of window dont expand to
fill window.
workaround: Clicking on a topic link fixes the issue. Window gets saved
correctly on exit and problem never appears again."
Comment 1 Vincent Trouilliez 2005-12-29 23:55:45 UTC
"me too", on Dapper as well. 
Comment 2 Christian Persch 2006-01-02 19:05:05 UTC
Does that mean the whole contents of the window (i.e. including the toolbar and menubar), or just the content area with the displayed help file?
Comment 3 Vincent Trouilliez 2006-01-02 19:32:56 UTC
Created attachment 56678 [details]
"Before"
Comment 4 Vincent Trouilliez 2006-01-02 19:33:47 UTC
Created attachment 56679 [details]
"After"
Comment 5 Vincent Trouilliez 2006-01-02 19:39:01 UTC
> Does that mean the whole contents of the window (i.e. including the toolbar and
> menubar), or just the content area with the displayed help file?

I attached a couple screenshot to show exactly what happens, once you maximize the window for the first time after launching Yelp. It's 100% repeatable: just make sure yelp isn't maximised, close it, open it again, maximize... and witness the bug.
Comment 6 Don Scorgie 2006-01-03 19:43:47 UTC
Hi,

I'm seeing this bug too with gecko 1.8 (firefox 1.5).  With 1.7 (firefox 1.0), the resizing works as expected.

I'm also seeing that it doesn't extend out any of the other widgets i.e. toolbar and side panel stay the same size / position as they were and the window grows a grey border down the right and bottom sides.  Just to confuse matters slightly more ;)
Comment 7 Brent Smith (smitten) 2006-01-08 18:37:48 UTC
So this is a mozilla bug right? chpe?
Comment 8 Don Scorgie 2006-01-08 20:24:39 UTC
I'm guessing this is a mozilla bug.  I can reproduce it in devhelp as well, so its not just Yelp.
Comment 9 Dooglus 2006-01-09 04:18:15 UTC
Notice that the failure to resize the window's contents happens on every resize until you click on a topic link.

I can resize the window multiple times and the contents don't resize until I click on one of the links.
Comment 10 Don Scorgie 2006-01-09 18:53:51 UTC
Ok, a few more notes for any intrepid adventurers attempting to look into this:
1. As Dooglus says, the window can be resized multiple times without the contents following
2. For me at least, clicking anywhere in the content pane will result in the bug fizing itself
3.  When you do the above, with a11y enabled, yelp spits out a couple of warnings:
(yelp:29666): GLib-GObject-WARNING **: gsignal.c:1028: unable to lookup signal "activate" of unloaded type `MaiAtkObject'

(yelp:29666): GLib-GObject-CRITICAL **: g_signal_emit_valist: assertion `signal_id > 0' failed

Breaking on g_log functions produces the backtrace:
Breakpoint 2, IA__g_log (log_domain=0xb71c17bc "GLib-GObject",
    log_level=G_LOG_LEVEL_WARNING,
    format=0xb71c3fd4 "gsignal.c:1028: unable to lookup signal \"%s\" of unloaded type `%s'") at gmessages.c:516
516       va_start (args, format);
(gdb) where
  • #0 IA__g_log
    at gmessages.c line 516
  • #1 IA__g_signal_lookup
    at gsignal.c line 1027
  • #2 nsDocAccessibleWrap::FireToolkitEvent
    at nsDocAccessibleWrap.cpp line 419
  • #3 nsWindow::DispatchActivateEvent
    at nsWindow.cpp line 4303
  • #4 nsWindow::OnButtonPressEvent
    at nsWindow.cpp line 1533
  • #5 button_press_event_cb
    at nsWindow.cpp line 3706
  • #6 _gtk_marshal_BOOLEAN__BOXED
    at gtkmarshalers.c line 83
  • #7 IA__g_closure_invoke
    at gclosure.c line 490
  • #8 signal_emit_unlocked_R
    at gsignal.c line 2449
  • #9 IA__g_signal_emit_valist
    at gsignal.c line 2218
  • #10 IA__g_signal_emit
    at gsignal.c line 2252
  • #11 gtk_widget_event_internal
    at gtkwidget.c line 3735
  • #12 IA__gtk_propagate_event
    at gtkmain.c line 2175
  • #13 IA__gtk_main_do_event
    at gtkmain.c line 1412
  • #14 gdk_event_dispatch
    at gdkevents-x11.c line 2291
  • #15 IA__g_main_context_dispatch
    at gmain.c line 1913
  • #16 g_main_context_iterate
    at gmain.c line 2544
  • #17 IA__g_main_loop_run
    at gmain.c line 2748
  • #18 bonobo_main
    at bonobo-main.c line 312
  • #19 main
    at yelp-main.c line 387

Not entirely certain that the warnings are related to the bug itself, but I don't recall seeing them prior to the bug appearing and they only appear once, when the bug manifests itself.  After the warnings appear and the window fixes itself, the warnings no longer appear when clicking in the window...

Enjoy
Comment 11 Christian Persch 2006-01-11 23:42:37 UTC
The warnings are entirely unrelated to this bug (they should however be filed in a separate yelp bug :).

I suspect this is https://bugzilla.mozilla.org/show_bug.cgi?id=312998 .
Comment 12 Don Scorgie 2006-01-21 16:36:57 UTC
Created attachment 57801 [details] [review]
Proposed fix

Hi,

I managed to stumble across a possible solution to this, but I'm not entirely certain how it works ;)

The problem seems to be related to the uimanager stuff.  The patch removes the call to gtk_ui_manager_ensure_update and moves some other stuff out of window_populate.  This seems to make the problem go away for me.  Note, that this is the minimum moving of stuff that makes the problem go away.

This doesn't seem to introduce any new problems for me, but can someone test this, see if it causes any problems for anyone else?

Cheers
Comment 13 Brent Smith (smitten) 2006-01-21 16:51:48 UTC
doesn't seem to cause any regressions against ff 1.0.7 (gecko 1.7?) at first glance
Comment 14 Don Scorgie 2006-01-22 17:10:24 UTC
*** Bug 326803 has been marked as a duplicate of this bug. ***
Comment 15 Christian Persch 2006-01-23 12:53:34 UTC
Don: can you file the traces from comment 10 in a separate bug? They're unrelated to this.

About the patch: I'm suspecting that it somehow causes the initial focus (i.e. focus when the window is shown) to be on the YelpHtml widget. You could try doing that directly, and see if it has the same effect?
Comment 16 Don Scorgie 2006-01-23 18:47:33 UTC
Hi,

I've just committed a (1 line) fix for this.  The fix is to set the initial focus to the html widget.

Christian: I didn't file a seperate bug as it is related to this.  I can't reproduce the problem I described when this bug is fixed.

Closing

2006-01-23  Don Scorgie  <dscorgie@cvs.gnome.org>

	* src/yelp-window.c: 
	Set initial focus to html widget, fixes #324190