[test/xdgtest] Create/destroy popup on click
ClosedPublic

Authored by davidedmundson on Oct 18 2018, 10:20 AM.

Details

Summary

This allows a dev to move the window before creating the popup which
is extremely useful in testing constraints. Also makes it very easy to
mod this code into a grabbing popup for other tests.

This patch also improve the painted surfaces to show the anchor rect
around where we place the popup which is easier for visual debugging.

No library code changes

Test Plan

Ran the test
KWin doesn't position the popup according to all constraints
Soon will

Diff Detail

Repository
R127 KWayland
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
Restricted Application added a project: Frameworks. · View Herald TranscriptOct 18 2018, 10:20 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
davidedmundson requested review of this revision.Oct 18 2018, 10:20 AM
davidedmundson edited the summary of this revision. (Show Details)Oct 18 2018, 10:24 AM
zzag accepted this revision.Oct 18 2018, 11:15 AM
zzag added a subscriber: zzag.
zzag added inline comments.
tests/xdgtest.cpp
172

FlipY?

This revision is now accepted and ready to land.Oct 18 2018, 11:15 AM
zzag added inline comments.Oct 18 2018, 11:21 AM
tests/xdgtest.cpp
202

Also, you could use Qt::blue. :-)

This revision was automatically updated to reflect the committed changes.