hi.. this evening i ported KMix to use KNotificationAreaItem. it was very easy and it still works ;)
there are a few caveats: * Activate has not positioning information kmix tries to position the parentWindow (that borderless window with a slider and the Mixer button) to match up with the icon, as if it were a popup associated with it. we don't pass any geometry information back with the Activate() dbus call as we do with ContextMenu(int,int). the questions i have are: a) what to send across to help make this happen: x and y? a rect, emulating geometry()? a Direction? it would be nice is such popups could decide to center on the icon and even more so if they could appear outside of, e.g., panels; e.g. essentially the same thing as Corona::popupPosition does. b) should a setHandleParentAsPopup(bool) method, which would internalize this behaviour and all it's intricacies? this way there could also be two code paths: one for the legacy mode and one for the new mode c) is there someway to avoid having to support this at all? ;) kmix is using it rather .. uhm .. creatively. but it seems like a valid use case in that situation. * activateRequested() signal - unused? the activateRequested() signal doesn't seem to be used; is there a purpose for it? * context menu handling when using KSystemTrayIcon there are some additional items added (activate, quit). thinking about how to handle this, we could either do exactly what KSystemtrayIcon does when in "new and improved" mode and add these items after aboutToShow is called the first time or we could improve on this somewhat ;) the thing with KSystemTrayIcon is that if you clear the menu after it's shown once, you won't get those actions in the menu anymore. nobody does this it seems, however., so that's all good so far. so maybe this is Good Enough(tm) however, we could do thing a few different ways: - provide a KActionCollection to register actions with, and then when the menu shows clear it and populate it with the context of the action collection plus our own - provide an addAction / removeAction pair that would do more or less the same thing internally * Tooltip -> ToolTip for consistency with the rest of the Qt/KDE api (already changed and commmitted to svn) -- Aaron J. Seigo humru othro a kohnu se GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 KDE core developer sponsored by Qt Software
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
