Use QWidget to see if the window is visible
ClosedPublic

Authored by apol on Feb 14 2018, 8:20 PM.

Details

Summary

Instead of using KWindowInfo::mappingState that won't give reliable
information about the window on wayland.

Test Plan

tested locally

Diff Detail

Repository
R289 KNotifications
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
apol created this revision.Feb 14 2018, 8:20 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 14 2018, 8:20 PM
apol requested review of this revision.Feb 14 2018, 8:20 PM
stikonas accepted this revision.Feb 14 2018, 8:48 PM
stikonas added a subscriber: stikonas.

Works for me.

This revision is now accepted and ready to land.Feb 14 2018, 8:48 PM
apol updated this revision to Diff 27182.Feb 14 2018, 8:48 PM
apol removed a reviewer: stikonas.

Properly detect the window state when not on X11

KWindowInfo doesn't support most features on Wayland (or any other backend)
so we provide an alternative simpler, dumber path.

This revision now requires review to proceed.Feb 14 2018, 8:48 PM
stikonas accepted this revision.Feb 14 2018, 8:52 PM
This revision is now accepted and ready to land.Feb 14 2018, 8:52 PM
This revision was automatically updated to reflect the committed changes.