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

diff --git a/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp b/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
index ea1f83b..d095331 100644
--- a/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
+++ b/synfig-studio/trunk/src/gtkmm/canvasproperties.cpp
@@ -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;
 }