17 Mar
2010
17 Mar
'10
11:46 p.m.
On Wed, Mar 17, 2010 at 7:07 PM, Diederik van Lierop <mail@...1689...> wrote:
I did ;-) and found out that after the snap indicator was deleted, the current_item was no longer valid (null). Events get lost in such a case because we don't know where to send them to. Although canvas->need_repick was flagged, a repick hadn't occured yet. I fixed this (rev. 9158) by explicitely calling pick_current_item() in emit_event() in such a case, which fixed scrolling when hovering above a snap indicator. I'm still not sure though if this is a proper fix, but I haven't heard of any regressions or side effects yet.
Ah, sorry for belated advice :) I think your fix is OK, but I will have it in mind if I notice any regressions.
--
bulia byak
Inkscape. Draw Freely.
http://www.inkscape.org