diff --git a/src/shape-editor.cpp b/src/shape-editor.cpp
index d73e99e7c8905ca484200ee6e3fead5d6a33e361..bfb0e3c0c61ae905ce78730dc83bdca39e31a205 100644 (file)
--- a/src/shape-editor.cpp
+++ b/src/shape-editor.cpp
#include "desktop-handles.h"
#include "knotholder.h"
#include "live_effects/parameter/pointparam-knotholder.h"
-#include "node-context.h"
+#include "nodepath.h"
#include "xml/node-event-vector.h"
#include "prefs-utils.h"
#include "object-edit.h"
-#include "splivarot.h"
#include "style.h"
+#include "display/curve.h"
+#include <2geom/pathvector.h>
#include "shape-editor.h"
}
ShapeEditor::~ShapeEditor() {
- unset_item();
+ unset_item(SH_KNOTHOLDER);
+ unset_item(SH_NODEPATH);
}
-void ShapeEditor::unset_item() {
-
+void ShapeEditor::unset_item(SubType type, bool keep_knotholder) {
Inkscape::XML::Node *old_repr = NULL;
- if (this->nodepath) {
- old_repr = this->nodepath->repr;
- }
-
- if (!old_repr && this->knotholder) {
- old_repr = this->knotholder->repr;
- }
-
- if (old_repr) { // remove old listener
- sp_repr_remove_listener_by_data(old_repr, this);
- Inkscape::GC::release(old_repr);
- }
+ switch (type) {
+ case SH_NODEPATH:
+ if (this->nodepath) {
+ old_repr = this->nodepath->repr;
+ sp_repr_remove_listener_by_data(old_repr, this);
+ Inkscape::GC::release(old_repr);
- if (this->nodepath) {
- this->grab_node = -1;
- sp_nodepath_destroy(this->nodepath);
- this->nodepath = NULL;
- }
-
- if (this->knotholder) {
- sp_knot_holder_destroy(this->knotholder);
- this->knotholder = NULL;
+ this->grab_node = -1;
+ sp_nodepath_destroy(this->nodepath);
+ this->nodepath = NULL;
+ }
+ break;
+ case SH_KNOTHOLDER:
+ if (this->knotholder) {
+ old_repr = this->knotholder->repr;
+ sp_repr_remove_listener_by_data(old_repr, this);
+ Inkscape::GC::release(old_repr);
+
+ if (!keep_knotholder) {
+ delete this->knotholder;
+ this->knotholder = NULL;
+ }
+ }
+ break;
}
}
return (this->knotholder != NULL);
}
-bool ShapeEditor::has_local_change () {
- if (this->nodepath)
- return (this->nodepath->local_change != 0);
- else if (this->knotholder)
- return (this->knotholder->local_change != 0);
- else
- return false;
+void ShapeEditor::update_knotholder () {
+ if (this->knotholder)
+ this->knotholder->update_knots();
}
-void ShapeEditor::decrement_local_change () {
- if (this->nodepath) {
- if (this->nodepath->local_change > 0)
- this->nodepath->local_change--;
- } else if (this->knotholder) {
- this->knotholder->local_change = FALSE;
+bool ShapeEditor::has_local_change (SubType type) {
+ switch (type) {
+ case SH_NODEPATH:
+ return (this->nodepath && this->nodepath->local_change);
+ case SH_KNOTHOLDER:
+ return (this->knotholder && this->knotholder->local_change != 0);
+ default:
+ g_assert_not_reached();
+ }
+}
+
+void ShapeEditor::decrement_local_change (SubType type) {
+ switch (type) {
+ case SH_NODEPATH:
+ if (this->nodepath && this->nodepath->local_change > 0) {
+ this->nodepath->local_change--;
+ }
+ break;
+ case SH_KNOTHOLDER:
+ if (this->knotholder) {
+ this->knotholder->local_change = FALSE;
+ }
+ break;
+ default:
+ g_assert_not_reached();
}
}
-SPItem *ShapeEditor::get_item () {
- SPItem *item = NULL;
- if (this->has_nodepath()) {
- item = this->nodepath->item;
- } else if (this->has_knotholder()) {
- item = SP_ITEM(this->knotholder->item);
+const SPItem *ShapeEditor::get_item (SubType type) {
+ const SPItem *item = NULL;
+ switch (type) {
+ case SH_NODEPATH:
+ if (this->has_nodepath()) {
+ item = this->nodepath->item;
+ }
+ break;
+ case SH_KNOTHOLDER:
+ if (this->has_knotholder()) {
+ item = this->knotholder->getItem();
+ }
+ break;
}
return item;
}
@@ -146,24 +171,26 @@ static void shapeeditor_event_attr_changed(Inkscape::XML::Node */*repr*/, gchar
gchar const */*old_value*/, gchar const */*new_value*/,
bool /*is_interactive*/, gpointer data)
{
- gboolean changed = FALSE;
+ gboolean changed_np = FALSE;
+ gboolean changed_kh = FALSE;
g_assert(data);
ShapeEditor *sh = ((ShapeEditor *) data);
- if ( sh->has_knotholder() || ( sh->has_nodepath() && sh->nodepath_edits_repr_key(name) ) )
+ if (sh->has_nodepath() && sh->nodepath_edits_repr_key(name))
{
- changed = !sh->has_local_change();
- sh->decrement_local_change();
+ changed_np = !sh->has_local_change(SH_NODEPATH);
+ sh->decrement_local_change(SH_NODEPATH);
+
}
- if (changed) {
+ if (changed_np) {
GList *saved = NULL;
if (sh->has_nodepath()) {
saved = sh->save_nodepath_selection();
}
- sh->reset_item ();
+ sh->reset_item(SH_NODEPATH);
if (sh->has_nodepath() && saved) {
sh->restore_nodepath_selection(saved);
@@ -171,6 +198,18 @@ static void shapeeditor_event_attr_changed(Inkscape::XML::Node */*repr*/, gchar
}
}
+
+ if (sh->has_knotholder())
+ {
+ changed_kh = !sh->has_local_change(SH_KNOTHOLDER);
+ sh->decrement_local_change(SH_KNOTHOLDER);
+ if (changed_kh) {
+ // this can happen if an LPEItem's knotholder handle was dragged, in which case we want
+ // to keep the knotholder; in all other cases (e.g., if the LPE itself changes) we delete it
+ sh->reset_item(SH_KNOTHOLDER, !strcmp(name, "d"));
+ }
+ }
+
sh->update_statusbar(); //TODO: sh->get_container()->update_statusbar();
}
};
-void ShapeEditor::set_item(SPItem *item) {
-
- unset_item();
+void ShapeEditor::set_item(SPItem *item, SubType type, bool keep_knotholder) {
+ // this happens (and should only happen) when for an LPEItem having both knotholder and nodepath the knotholder
+ // is adapted; in this case we don't want to delete the knotholder since this freezes the handles
+ unset_item(type, keep_knotholder);
this->grab_node = -1;
if (item) {
- this->nodepath = sp_nodepath_new(desktop, item, (prefs_get_int_attribute("tools.nodes", "show_handles", 1) != 0));
- if (this->nodepath) {
- this->nodepath->shape_editor = this;
- }
- this->knotholder = sp_item_knot_holder(item, desktop);
-
- if (this->nodepath || this->knotholder) {
- // setting new listener
- Inkscape::XML::Node *repr;
- if (this->knotholder)
- repr = this->knotholder->repr;
- else
- repr = SP_OBJECT_REPR(item);
- if (repr) {
- Inkscape::GC::anchor(repr);
- sp_repr_add_listener(repr, &shapeeditor_repr_events, this);
- }
+ Inkscape::XML::Node *repr;
+ switch(type) {
+ case SH_NODEPATH:
+ if (SP_IS_LPE_ITEM(item)) {
+ this->nodepath = sp_nodepath_new(desktop, item, (prefs_get_int_attribute("tools.nodes", "show_handles", 1) != 0));
+ }
+ if (this->nodepath) {
+ this->nodepath->shape_editor = this;
+
+ // setting new listener
+ repr = SP_OBJECT_REPR(item);
+ Inkscape::GC::anchor(repr);
+ sp_repr_add_listener(repr, &shapeeditor_repr_events, this);
+ }
+ break;
+
+ case SH_KNOTHOLDER:
+ if (!this->knotholder) {
+ // only recreate knotholder if none is present
+ this->knotholder = sp_item_knot_holder(item, desktop);
+ }
+ if (this->knotholder) {
+ this->knotholder->update_knots();
+ // setting new listener
+ repr = this->knotholder->repr;
+ Inkscape::GC::anchor(repr);
+ sp_repr_add_listener(repr, &shapeeditor_repr_events, this);
+ }
+ break;
}
}
}
*/
void ShapeEditor::set_item_lpe_path_parameter(SPItem *item, SPObject *lpeobject, const char * key)
{
- unset_item();
+ unset_item(SH_NODEPATH);
this->grab_node = -1;
@@ -241,9 +293,9 @@ void ShapeEditor::set_item_lpe_path_parameter(SPItem *item, SPObject *lpeobject,
* pass a new knotholder to ShapeEditor to manage (and delete)
*/
void
-ShapeEditor::set_knotholder(SPKnotHolder * knot_holder)
+ShapeEditor::set_knotholder(KnotHolder * knot_holder)
{
- unset_item();
+ unset_item(SH_KNOTHOLDER);
this->grab_node = -1;
/** FIXME: think about this. Is this thing only called when the item needs to be updated?
Why not make a reload function in NodePath and in KnotHolder? */
-void ShapeEditor::reset_item ()
+void ShapeEditor::reset_item (SubType type, bool keep_knotholder)
{
- if ( (this->nodepath) && (IS_LIVEPATHEFFECT(this->nodepath->object)) ) {
- SPItem * item = this->nodepath->item;
- SPObject *obj = this->nodepath->object;
- char * key = g_strdup(this->nodepath->repr_key);
- set_item_lpe_path_parameter(item, obj, key); // the above checks for nodepath, so it is indeed a path that we are editing
- g_free(key);
- } else {
- SPItem * item = get_item();
- set_item(item);
+ switch (type) {
+ case SH_NODEPATH:
+ if ( (this->nodepath) && (IS_LIVEPATHEFFECT(this->nodepath->object)) ) {
+ SPItem * item = this->nodepath->item;
+ SPObject *obj = this->nodepath->object;
+ char * key = g_strdup(this->nodepath->repr_key);
+ set_item_lpe_path_parameter(item, obj, key); // the above checks for nodepath, so it is indeed a path that we are editing
+ g_free(key);
+ } else {
+ SPItem * item = (SPItem *) get_item(SH_NODEPATH);
+ set_item(item, SH_NODEPATH);
+ }
+ break;
+ case SH_KNOTHOLDER:
+ if (this->knotholder) {
+ SPItem * item = (SPItem *) get_item(SH_KNOTHOLDER);
+ set_item(item, SH_KNOTHOLDER, keep_knotholder);
+ }
+ break;
}
}
if (!this->nodepath)
return false; // no stroke in knotholder
- SPItem *item = get_item();
+ const SPItem *item = get_item(SH_NODEPATH);
//Translate click point into proper coord system
this->curvepoint_doc = desktop->w2d(event_p);
this->curvepoint_doc *= sp_item_dt2i_affine(item);
- sp_nodepath_ensure_livarot_path(this->nodepath);
-
- NR::Maybe<Path::cut_position> position = get_nearest_position_on_Path(this->nodepath->livarot_path, this->curvepoint_doc);
- if (!position) {
- return false;
- }
+ SPCurve *curve = this->nodepath->curve; // not sure if np->curve is always up to date...
+ Geom::PathVector const &pathv = curve->get_pathvector();
+ Geom::PathVectorPosition pvpos = Geom::nearestPoint(pathv, this->curvepoint_doc);
- NR::Point nearest = get_point_on_Path(this->nodepath->livarot_path, position->piece, position->t);
+ NR::Point nearest = pathv[pvpos.path_nr].pointAt(pvpos.t);
NR::Point delta = nearest - this->curvepoint_doc;
delta = desktop->d2w(delta);
bool close = (NR::L2 (delta) < stroke_tolerance);
if (remember && close) {
+ // calculate index for nodepath's representation.
+ double int_part;
+ double t = std::modf(pvpos.t, &int_part);
+ unsigned int segment_index = (unsigned int)int_part + 1;
+ for (unsigned int i = 0; i < pvpos.path_nr; ++i) {
+ segment_index += pathv[i].size() + 1;
+ if (pathv[i].closed())
+ segment_index += 1;
+ }
+
this->curvepoint_event[NR::X] = (gint) event_p [NR::X];
this->curvepoint_event[NR::Y] = (gint) event_p [NR::Y];
this->hit = true;
- this->grab_t = position->t;
- this->grab_node = position->piece;
+ this->grab_t = t;
+ this->grab_node = segment_index;
}
return close;
void ShapeEditor::select_segment_near_point(bool toggle) {
if (this->nodepath) {
sp_nodepath_select_segment_near_point(this->nodepath, this->curvepoint_doc, toggle);
- } else if (this->knotholder) {
+ }
+ if (this->knotholder) {
// we do not select segments in knotholder... yet?
}
}
this->curvepoint_event[NR::X] = x;
this->curvepoint_event[NR::Y] = y;
- } else if (this->knotholder) {
+ }
+ if (this->knotholder) {
// we do not drag curve in knotholder
}
sp_nodepath_show_helperpath (this->nodepath, show);
}
-void ShapeEditor::flip (NR::Dim2 axis, NR::Maybe<NR::Point> center) {
+void ShapeEditor::flip (NR::Dim2 axis, boost::optional<NR::Point> center) {
if (this->nodepath)
sp_nodepath_flip (this->nodepath, axis, center);
}