Merge lp:~donadigo/wingpanel-indicator-notifications/notification-open-app into lp:~wingpanel-devs/wingpanel-indicator-notifications/wingpanel-indicator-notifications
Proposed by
Adam Bieńkowski
Status: | Superseded |
---|---|
Proposed branch: | lp:~donadigo/wingpanel-indicator-notifications/notification-open-app |
Merge into: | lp:~wingpanel-devs/wingpanel-indicator-notifications/wingpanel-indicator-notifications |
Diff against target: |
308 lines (+97/-61) 6 files modified
src/CMakeLists.txt (+1/-1) src/Services/Notification.vala (+7/-4) src/Services/NotificationMonitor.vala (+29/-9) src/Widgets/AppEntry.vala (+9/-0) src/Widgets/NotificationEntry.vala (+12/-0) src/Widgets/NotificationsList.vala (+39/-47) |
To merge this branch: | bzr merge lp:~donadigo/wingpanel-indicator-notifications/notification-open-app |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
WingPanel Devs | Pending | ||
Review via email: mp+297927@code.launchpad.net |
Commit message
* Fix bug #1594227: "Clicking a notification should open/focus the app".
* General code improvements.
Description of the change
Fixes bug #1594227: "Clicking a notification should open/focus the app".
This branch enables functionality to open / focus the app by clicking on the notification, if the notification has a default action, it will be launched instead of the app.
It also contains a large code improvements, simplifies the method names and makes it more easier to read.
To post a comment you must log in.
Unmerged revisions
- 97. By Adam Bieńkowski
-
General code improvements; if notification clicked and does not have a default action, focus the app instead