Add support for 'activate' signal callbacks to libappindicator
Bug #1428473 reported by
microcai
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
libappindicator |
Won't Fix
|
Undecided
|
Unassigned |
Bug Description
Title says it all!
Yes, it seems to be fixed, but, no, no. the fix adds 'activate' signal for menu.
no no no, but double click the tray icon will infact sends 'activate'. and yet your code only parses "Scroll" and 'SecondaryActivate' signals in `bus_method_call`
Related branches
lp:~microcai/libappindicator/libappindicator
Rejected
for merging
into
lp:libappindicator/15.04
- Ted Gould (community): Disapprove
-
Diff: 92 lines (+21/-22)2 files modifiedsrc/app-indicator.c (+20/-21)
src/app-indicator.h (+1/-1)
To post a comment you must log in.
Sorry, we're not interested in activate support because it creates a user experience that isn't menu based. We want the appindicators to be a single menu and work that way. The activate signal is for things that don't have a menu there, but we'll always have a menu.