From 210222db73d61a27301a951594fefeaf375fafe3 Mon Sep 17 00:00:00 2001 From: Nikita Kitaev Date: Sat, 10 Apr 2010 20:43:11 -0700 Subject: [PATCH] Keyframe dialog: enable apply and delete buttons --- synfig-studio/src/gtkmm/dialog_keyframe.cpp | 44 ++++++++++++++++++++--------- 1 file changed, 31 insertions(+), 13 deletions(-) diff --git a/synfig-studio/src/gtkmm/dialog_keyframe.cpp b/synfig-studio/src/gtkmm/dialog_keyframe.cpp index d69ec50..b746490 100644 --- a/synfig-studio/src/gtkmm/dialog_keyframe.cpp +++ b/synfig-studio/src/gtkmm/dialog_keyframe.cpp @@ -66,11 +66,11 @@ Dialog_Keyframe::Dialog_Keyframe(Gtk::Window& parent, etl::handlesignal_clicked().connect(sigc::mem_fun(*this, &Dialog_Keyframe::on_ok_pressed)); -/* Gtk::Button *apply_button(manage(new class Gtk::Button(Gtk::StockID("gtk-apply")))); + Gtk::Button *apply_button(manage(new class Gtk::Button(Gtk::StockID("gtk-apply")))); apply_button->show(); add_action_widget(*apply_button,1); apply_button->signal_clicked().connect(sigc::mem_fun(*this, &Dialog_Keyframe::on_apply_pressed)); -*/ + Gtk::Button *delete_button(manage(new class Gtk::Button(Gtk::StockID("gtk-delete")))); delete_button->show(); add_action_widget(*delete_button,3); @@ -86,10 +86,13 @@ Dialog_Keyframe::Dialog_Keyframe(Gtk::Window& parent, etl::handlepack_start(*table); +/* // \todo Allow setting descriptions for keyframes + entry_description.set_text(_("Not yet implemented")); - //table->attach(*manage(new Gtk::Label(_("Description"))), 0, 1, 0, 1, Gtk::SHRINK|Gtk::FILL, Gtk::SHRINK|Gtk::FILL, 0, 0); - //table->attach(entry_description, 1, 2, 0, 1, Gtk::EXPAND|Gtk::FILL, Gtk::SHRINK|Gtk::FILL, 0, 0); + table->attach(*manage(new Gtk::Label(_("Description"))), 0, 1, 0, 1, Gtk::SHRINK|Gtk::FILL, Gtk::SHRINK|Gtk::FILL, 0, 0); + table->attach(entry_description, 1, 2, 0, 1, Gtk::EXPAND|Gtk::FILL, Gtk::SHRINK|Gtk::FILL, 0, 0); +*/ table->show_all(); @@ -118,10 +121,15 @@ Dialog_Keyframe::set_keyframe(const synfig::Keyframe& x) void Dialog_Keyframe::on_ok_pressed() { - if(widget_waypoint_model->get_waypoint_model().is_trivial()) - return; + on_apply_pressed(); + hide(); +} - synfigapp::Action::Handle action(synfigapp::Action::create("KeyframeWaypointSet")); + +void +Dialog_Keyframe::on_delete_pressed() +{ + synfigapp::Action::Handle action(synfigapp::Action::create("KeyframeRemove")); assert(action); @@ -130,19 +138,29 @@ Dialog_Keyframe::on_ok_pressed() action->set_param("keyframe",keyframe_); action->set_param("model",widget_waypoint_model->get_waypoint_model()); - if(!canvas_interface->get_instance()->perform_action(action)) + if(canvas_interface->get_instance()->perform_action(action)) { + hide(); } } void -Dialog_Keyframe::on_delete_pressed() +Dialog_Keyframe::on_apply_pressed() { -} + if(widget_waypoint_model->get_waypoint_model().is_trivial()) + return; + synfigapp::Action::Handle action(synfigapp::Action::create("KeyframeWaypointSet")); -void -Dialog_Keyframe::on_apply_pressed() -{ + assert(action); + + action->set_param("canvas",canvas_interface->get_canvas()); + action->set_param("canvas_interface",canvas_interface); + action->set_param("keyframe",keyframe_); + action->set_param("model",widget_waypoint_model->get_waypoint_model()); + + if(!canvas_interface->get_instance()->perform_action(action)) + { + } } -- 2.7.4