Skip to content

Fix StatusNotifierItem checkVisibility on Windows

Freya Lupen requested to merge (removed):freyalupen/notifier-checkvis into kf5

This removes an empty (false) if-def to let Windows use the regular codepath, which appears to work fine.

This is in attempt to fix an issue in Konversation where it was impossible to Restore the window from the system tray icon, because it calls KStatusNotifierItem::activate with an associatedWidget, which calls checkVisibility to activate it, which did nothing on Windows.

To check that this works I tested that calling TrayIcon::restoreWindow in Konversation now restores the window. (Unfortunately the system tray icon doesn't work in my local build, so I modified the show/hide action to hide then reshow, for this.)

Merge request reports