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 557153 - crash: deleting the last visible sheet
crash: deleting the last visible sheet
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: GUI
unspecified
Other All
: Normal critical
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2008-10-20 21:25 UTC by Andreas J. Guelzow
Modified: 2008-11-20 14:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Andreas J. Guelzow 2008-10-20 21:25:09 UTC 
start gnumeric
3 sheets
make two invisible using the manage sheet dialog
close the dialog
right click on the last sheet tab and select remove
use a menu item... 

crash
Comment 1 Andreas J. Guelzow 2008-10-21 06:03:32 UTC 
Well, we are counting both visible and non-visible sheets:

static void
wbcg_menu_state_sheet_count (WBCGtk *wbcg)
{
	int const sheet_count = g_list_length (wbcg->notebook->children);
	/* Should we enable commands requiring multiple sheets */
	gboolean const multi_sheet = (sheet_count > 1);

	/* Scrollable if there are more than 3 tabs */
	gtk_notebook_set_scrollable (wbcg->notebook, sheet_count > 3);

	wbc_gtk_set_action_sensitivity (wbcg, "SheetRemove", multi_sheet);
}

We should only be counting visible ones.
Comment 2 Morten Welinder 2008-11-20 14:44:40 UTC 
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.