Pass the clutter event to activate()

Keruspe / GPaste

Clipboard management system

BSD 2-Clause "Simplified" License

790 stars 57 forks source link

Pass the clutter event to activate() #300

Closed afn closed 4 years ago

afn commented 4 years ago

The activate() method on ui.PopupMenu takes the triggering event as an argument; in one place where it was called, that argument was not being passed.

Fixes #299

Keruspe commented 4 years ago

Huh, I wonder when it started to fail. It definitely used to work. Will dig in gnome-shell history to find out

Keruspe commented 4 years ago

Anyways, nice catch, thanks ! Will do bug fix releases tomorrow

Keruspe commented 4 years ago

Wow, I didn't remember this feature was that ancient, this particular line was last changed: " Keruspe committed on 26 Oct 2014"

Keruspe commented 4 years ago

And it seems this method actually always took the event as a param

afn commented 4 years ago

Yeah, I don’t get how this ever worked :) Thanks for the quick turnaround!