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 98878 - Closing a tab that hasn't finished loading leaves the spinner spinning
Closing a tab that hasn't finished loading leaves the spinner spinning
Status: RESOLVED FIXED
Product: galeon
Classification: Deprecated
Component: User interface
1.3.0
Other other
: Normal minor
: ---
Assigned To: Crispin Flowerday (not receiving bugmail)
Yanko Kaneti
Depends on: 58674
Blocks:
 
 
Reported: 2002-11-18 14:15 UTC by Tommi Komulainen
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Tommi Komulainen 2002-11-18 14:15:19 UTC
Open http://tko.ods.org/sleep.php in a new tab, close the tab during the
ten seconds it takes to load it, see the spinner keep on spinning.
Comment 1 Christophe Fergeau 2002-11-21 15:54:30 UTC
There should probably be something like
src/galeon-window.c:update_spinner_control in
utils/gul-notebook:galeon_tab_close_button_clicked_cb, but that's a
little tricky since GulNotebook shouldn't know anything about
GaleonWindow and stuff like that :(
Comment 2 Tommi Komulainen 2002-11-21 16:59:10 UTC
I tried catching the GtkContainer::remove signal from GulNotebook in
galeon-window.c, and update the spinner control from there, but it
didn't seem to have any effect.  Maybe I was missing something.
Comment 3 Tommi Komulainen 2003-01-20 13:44:22 UTC
bah.. Silly me assumed gtk would be behaving as advertised.  Can I
resolve this bug as NOTMYFAULT? :-P
Comment 4 Crispin Flowerday (not receiving bugmail) 2003-08-18 14:13:03 UTC
I hope no one minds if I take this one. One of my changes added a
signal that is called when a tab is removed. I'll hook onto it and
update the spinner control.
Comment 5 Crispin Flowerday (not receiving bugmail) 2003-08-19 07:58:02 UTC
Fixed in CVS