From 63273692c4e638b46a9e84026f62721784ec2cb4 Mon Sep 17 00:00:00 2001 From: "Johan B. C. Engelen" Date: Tue, 15 Jan 2008 14:29:14 +0000 Subject: prepare LPE parameter widgets to be owned by multiple dialogs, as it should be. For each parameter function that has not been re-implemented, a warning is issued. Sorry for the huge number of warnings showing when working with LPE. It is important to fix this asap. (bzr r4495) --- src/live_effects/parameter/point.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'src/live_effects/parameter/point.cpp') diff --git a/src/live_effects/parameter/point.cpp b/src/live_effects/parameter/point.cpp index eea337c85..5284bc797 100644 --- a/src/live_effects/parameter/point.cpp +++ b/src/live_effects/parameter/point.cpp @@ -80,8 +80,10 @@ PointParam::param_writeSVGValue() const } Gtk::Widget * -PointParam::param_getWidget() +PointParam::param_newWidget(Gtk::Tooltips * tooltips) { + // WIDGET TODO: This implementation is incorrect, it should create a *new* widget for the caller, not just return an already created widget + g_warning("PointParam::param_newWidget still needs recoding to work with multiple document views"); if (!_widget) { pointwdg = new Inkscape::UI::Widget::RegisteredPoint(); pointwdg->init(param_label, param_tooltip, param_key, *param_wr, param_effect->getRepr(), param_effect->getSPDoc()); @@ -104,8 +106,7 @@ PointParam::param_getWidget() static_cast(_widget)->pack_start(*(pointwdg->getPoint()), true, true); static_cast(_widget)->show_all_children(); - _tooltips = new Gtk::Tooltips(); - _tooltips->set_tip(*pButton, _("Edit on-canvas")); + tooltips->set_tip(*pButton, _("Edit on-canvas")); } return dynamic_cast (_widget); } -- cgit v1.2.3