Differential D5057

Add KAcceleratorManager
ClosedPublic
Actions

Authored by broulik on Mar 15 2017, 3:15 PM.

Details

Reviewers

mart

Group Reviewers

Plasma

Commits

R242:ff78b9e53113: Add KAcceleratorManager

Summary

This way containment context menus and PlasmaComponents Menu get accelerators assigned to by default.

CHANGELOG: Most menus have accelerators (Alt+letter shortcuts) now

BUG: 361915

Test Plan

There's probably some other places we need to adjust, like System Tray but this should cover most cases, like desktop context menu (cf the linked bug report) and menus created from QML

Diff Detail

Repository

R242 Plasma Framework (Library)

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

broulik created this revision.Mar 15 2017, 3:15 PM

Restricted Application added projects: Plasma, Frameworks. · View Herald TranscriptMar 15 2017, 3:15 PM

Restricted Application added subscribers: Frameworks, plasma-devel. · View Herald Transcript

long due papercut :)

This revision is now accepted and ready to land.Mar 15 2017, 4:17 PM

Closed by commit R242:ff78b9e53113: Add KAcceleratorManager (authored by broulik). · Explain WhyMar 16 2017, 10:53 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			src/declarativeimports/plasmacomponents/CMakeLists.txt (1 line)
M			src/declarativeimports/plasmacomponents/qmenu.cpp (3 lines)
M			src/scriptengines/qml/plasmoid/containmentinterface.cpp (3 lines)