Code review comment for lp:~quequotion/slingshot/fix-1635776

Revision history for this message
Santiago (santileortiz) wrote :

The merge was reverted because it breaks scrolling with the mouse wheel for people even without IMs, I think events are being "eaten" by the search_entry widget because it has the focus. Previously even though this happened too, event_box was getting all of them too.

Anyway this is just speculation, I haven't tested all of this, but maybe adding EventBox back will fix it? (I have no idea what will happen to fcitx here).

« Back to merge proposal