virtual Fill:: and Outline:: methods so that Canvas::Items that cache image renderings of themselves can invalidate those caches when colors etc. change; add Item::{begin,end}_visual_change() so that we can notify the canvas more efficiently when *only* visual properties have changed and not the bounding box (probably needs to be used more widely)

1 files changed, 11 insertions, 1 deletions

diff --git a/libs/canvas/item.cc b/libs/canvas/item.cc
index 7991b3aaaf..da551fade6 100644
--- a/libs/canvas/item.cc
+++ b/libs/canvas/item.cc
@@ -295,7 +295,6 @@ Item::width () const
 	return 0;
 }
 
-/* XXX may be called even if bbox is not changing ... bit grotty */
 void
 Item::begin_change ()
 {
@@ -313,6 +312,17 @@ Item::end_change ()
 }
 
 void
+Item::begin_visual_change ()
+{
+}
+
+void
+Item::end_visual_change ()
+{
+	_canvas->item_visual_property_changed (this);
+}
+
+void
 Item::move (Duple movement)
 {
 	set_position (position() + movement);