X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=synfig-studio%2Ftrunk%2Fsrc%2Fgtkmm%2Fcanvasproperties.cpp;h=d09533173cb91da5b085413c2c3c29ea8ab5e0d7;hb=9459638ad6797b8139f1e9f0715c96076dbf0890;hp=73c9425bd3c09400520c372a77db25152d5b88fe;hpb=c2813b4d63804cac9b4067146a9e6eca717cb3e4;p=synfig.git

diff --git a/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp b/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
index 73c9425..d095331 100644
--- a/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
+++ b/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
@@ -6,6 +6,7 @@
 **
 **	\legal
 **	Copyright (c) 2002-2005 Robert B. Quattlebaum Jr., Adrian Bentley
+**	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
@@ -111,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();
@@ -221,14 +223,19 @@ CanvasProperties::on_rend_desc_changed()
 void
 CanvasProperties::on_apply_pressed()
 {
-	if(entry_id.get_text()!=canvas_interface_->get_canvas()->get_id())
-		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;
 }