Port to new AppMenu iface
ClosedPublic

Authored by davidedmundson on Dec 8 2017, 6:30 PM.

Details

Summary

Commit 2

Basic AppMenu Test

It tests the basic working functionality.

Don't merge test yet - depends a bit on Marco's
unrelated changes to appmenu.

Test Plan

Ran kwin with menus and patched QPT
Ran test

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Dec 8 2017, 6:30 PM
Restricted Application added a project: KWin. · View Herald TranscriptDec 8 2017, 6:30 PM
Restricted Application added subscribers: KWin, kwin. · View Herald Transcript
davidedmundson requested review of this revision.Dec 8 2017, 6:30 PM
graesslin accepted this revision.Dec 8 2017, 8:01 PM
graesslin added a subscriber: graesslin.

Looks good to me.

wayland_server.cpp
268

Unrelated empty line.

This revision is now accepted and ready to land.Dec 8 2017, 8:01 PM
This revision was automatically updated to reflect the committed changes.