drop use of bounding box to determine whether an item covers a point; add Item::covers(Duple const&)

Default implementation for Item still uses bounding box, but specializations for Arc (Circle), Polygon, Line and PolyLine have been added

1 files changed, 3 insertions, 7 deletions

diff --git a/libs/canvas/lookup_table.cc b/libs/canvas/lookup_table.cc
index 38b8be4afb..0cae7c9c4f 100644
--- a/libs/canvas/lookup_table.cc
+++ b/libs/canvas/lookup_table.cc
@@ -62,13 +62,9 @@ DumbLookupTable::items_at_point (Duple point) const
 		if (!(*i)->visible()) {
 			continue;
 		}
-
-		boost::optional<Rect> item_bbox = (*i)->bounding_box ();
-
-		if (item_bbox) {
-			if ((*i)->item_to_canvas (item_bbox.get ()).contains (point)) {
-				vitems.push_back (*i);
-			}
+		
+		if ((*i)->covers (point)) {
+			vitems.push_back (*i);
 		}
 	}