update-notifier-kde

Merge lp:~debfx/update-notifier-kde/codecs-context-menu into lp:update-notifier-kde

  1. codecs-context-menu
  2. Merge into trunk
Proposed by Felix Geyer
Status: Merged
Merged at revision: not available
Proposed branch: lp:~debfx/update-notifier-kde/codecs-context-menu
Merge into: lp:update-notifier-kde
Diff against target: 15 lines
1 file modified
update-notifier-kde (+3/-1)
To merge this branch: bzr merge lp:~debfx/update-notifier-kde/codecs-context-menu
Reviewer Review Type Date Requested Status
Jonathan Riddell Pending
Review via email: mp+13341@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Felix Geyer (debfx) wrote :

Currently when you right-click on the codecs tray icon, the dialog to actually install the software is shown instead of the context menu.
This makes it impossible to permanently hide the codecs tray icon.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'update-notifier-kde'
2--- update-notifier-kde 2009-10-03 21:52:48 +0000
3+++ update-notifier-kde 2009-10-14 13:25:17 +0000
4@@ -434,8 +434,10 @@
5 config.sync()
6 self.codecsTray.hide()
7
8- def codecsActivated(self):
9+ def codecsActivated(self, activationReason):
10 """ user wants to install, show dialogue and install """
11+ if activationReason != QSystemTrayIcon.Trigger:
12+ return
13 dialogue = KDialog()
14 dialogue.setButtons(KDialog.ButtonCode(KDialog.Ok | KDialog.Cancel))
15 dialogue.setButtonText(KDialog.Ok, i18n("Install Selected"))

Subscribers

People subscribed via source and target branches

to all changes: