diff options
author | Paul Davis <paul@linuxaudiosystems.com> | 2013-10-28 12:27:03 -0400 |
---|---|---|
committer | Paul Davis <paul@linuxaudiosystems.com> | 2013-10-28 12:27:03 -0400 |
commit | 1eb680f937efc9833c52dad52de170e30df06d60 (patch) | |
tree | 5c6634a3c261222caf6c707487fd830b20aa99ca /libs/canvas/lookup_table.cc | |
parent | f25ff95390f6e7d576783c55c2e7098db322578f (diff) |
add coordinate space comment
Diffstat (limited to 'libs/canvas/lookup_table.cc')
-rw-r--r-- | libs/canvas/lookup_table.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/libs/canvas/lookup_table.cc b/libs/canvas/lookup_table.cc index 9697ba8103..776d873de2 100644 --- a/libs/canvas/lookup_table.cc +++ b/libs/canvas/lookup_table.cc @@ -49,18 +49,19 @@ DumbLookupTable::get (Rect const &) return vitems; } -/* XXX: what coordinate system is the point in? parent of our group I think */ vector<Item *> DumbLookupTable::items_at_point (Duple point) const { + /* Point is in canvas coordinate system */ + list<Item *> items = _group.items (); vector<Item *> vitems; for (list<Item *>::const_iterator i = items.begin(); i != items.end(); ++i) { boost::optional<Rect> item_bbox = (*i)->bounding_box (); if (item_bbox) { - Rect parent_bbox = (*i)->item_to_parent (item_bbox.get ()); - if (parent_bbox.contains (point)) { + Rect canvas_bbox = (*i)->item_to_canvas (item_bbox.get ()); + if (canvas_bbox.contains (point)) { vitems.push_back (*i); } } |