Add "Navigation" group to ContextMenuExtension
ClosedPublic
Actions

Authored by antonanikin on Mar 9 2017, 9:03 AM.

Details

Reviewers

Group Reviewers

Commits

R32:081ad4cc0618: Add "Navigation" group to ContextMenuExtension
R33:081ad4cc0618: Add "Navigation" group to ContextMenuExtension

Summary

This patch adds new group to ContextMenuExtension for grouping corresponding actions. Current version places such actions to "Extension" group, which also populated by other common actions which leads to "mix" of actions with different sense.

Old menu:

New menu:

Test Plan

Tested with master branch

Diff Detail

Repository

R33 KDevPlatform

Lint

Automatic diff as part of commit; lint not applicable.

Unit

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

antonanikin created this revision.Mar 9 2017, 9:03 AM

antonanikin edited the summary of this revision. (Show Details)Mar 9 2017, 9:12 AM

kfunk accepted this revision.Mar 12 2017, 10:36 PM

This revision is now accepted and ready to land.Mar 12 2017, 10:36 PM

Closed by commit R33:081ad4cc0618: Add "Navigation" group to ContextMenuExtension (authored by antonanikin). · Explain WhyMar 13 2017, 2:34 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			interfaces/contextmenuextension.h (2 lines)
M			interfaces/contextmenuextension.cpp (5 lines)
M			plugins/classbrowser/classbrowserplugin.cpp (2 lines)
M			plugins/contextbrowser/contextbrowser.cpp (2 lines)
M			plugins/quickopen/quickopenplugin.cpp (4 lines)
M			plugins/switchtobuddy/switchtobuddyplugin.cpp (2 lines)