Do not allow to configure separator actions via context menu
ClosedPublic

Authored by mwolff on Feb 26 2018, 1:54 PM.

Details

Summary

It's not useful to add shortcuts for separators, nor to add them
to the toolbar. This patch prevents the confusing user interaction.
Note that the menu will instead be hidden, just like when you'd
left-click on a separator. But I guess this strange behavior should
be changed in Qt upstream instead (to keep the menu open).

Diff Detail

Repository
R263 KXmlGui
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mwolff created this revision.Feb 26 2018, 1:54 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 26 2018, 1:54 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
mwolff requested review of this revision.Feb 26 2018, 1:54 PM

+1, note that action can be null

broulik accepted this revision.Feb 26 2018, 2:03 PM
This revision is now accepted and ready to land.Feb 26 2018, 2:03 PM
This revision was automatically updated to reflect the committed changes.