X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=synfig-studio%2Ftrunk%2Fsrc%2Fgtkmm%2Fcanvasproperties.cpp;h=d09533173cb91da5b085413c2c3c29ea8ab5e0d7;hb=1f453e8ab546a357781372fe98aa1d004387d4b1;hp=5bd3f2e701ae8ef64ad4b006ae4cf162699d98b8;hpb=334e15ce6c4d9b1f30a168a55e7ef4d31320d568;p=synfig.git

diff --git a/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp b/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
index 5bd3f2e..d095331 100644
--- a/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
+++ b/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
@@ -6,7 +6,7 @@
 **
 **	\legal
 **	Copyright (c) 2002-2005 Robert B. Quattlebaum Jr., Adrian Bentley
-**	Copyright (c) 2007 Chris Moore
+**	Copyright (c) 2007, 2008 Chris Moore
 **
 **	This package is free software; you can redistribute it and/or
 **	modify it under the terms of the GNU General Public License as
@@ -112,6 +112,7 @@ CanvasProperties::CanvasProperties(Gtk::Window& parent,etl::handle<synfigapp::Ca
 	dialogBox->pack_start(widget_rend_desc, false, false, 0);
 
 	canvas_interface_->signal_rend_desc_changed().connect(sigc::mem_fun(*this,&studio::CanvasProperties::refresh));
+	canvas_interface_->signal_id_changed().connect(sigc::mem_fun(*this,&studio::CanvasProperties::refresh));
 
 	Gtk::Button *ok_button(manage(new class Gtk::Button(Gtk::StockID("gtk-ok"))));
 	ok_button->show();
@@ -222,14 +223,19 @@ CanvasProperties::on_rend_desc_changed()
 void
 CanvasProperties::on_apply_pressed()
 {
-	if(entry_id.get_text()!=canvas_interface_->get_canvas()->get_id() && !entry_id.get_text().empty())
-		canvas_interface_->set_id(entry_id.get_text());
-	if(entry_name.get_text()!=canvas_interface_->get_canvas()->get_name())
-		canvas_interface_->set_name(entry_name.get_text());
-	if(entry_description.get_text()!=canvas_interface_->get_canvas()->get_description())
-		canvas_interface_->set_description(entry_description.get_text());
-	if(dirty_rend_desc)			// do this last, because it causes the dialog to be refreshed with currently set values
-		canvas_interface_->set_rend_desc(widget_rend_desc.get_rend_desc());
+	synfigapp::Action::PassiveGrouper group(canvas_interface_->get_instance().get(),_("Edit Canvas Properties"));
+
+	// fetch these three values first, because each set_() method refreshes the dialog with currently set values
+	String id = entry_id.get_text();
+	String name = entry_name.get_text();
+	String description = entry_description.get_text();
+
+	// do this first, because the other three cause the dialog to be refreshed with currently set values
+	if (dirty_rend_desc) canvas_interface_->set_rend_desc(widget_rend_desc.get_rend_desc());
+
+	if (id != canvas_interface_->get_canvas()->get_id() && !id.empty())		canvas_interface_->set_id(id);
+	if (name != canvas_interface_->get_canvas()->get_name())				canvas_interface_->set_name(name);
+	if (description != canvas_interface_->get_canvas()->get_description())	canvas_interface_->set_description(description);
 
 	dirty_rend_desc=false;
 }