Differential D15567

Words: Do not popup contextmenu twice
ClosedPublic
Actions

Authored by danders on Sep 17 2018, 7:50 AM.

Details

Reviewers
boemann
leinir
Commits
R8:7bd9b921826e: Words: Do not popup contextmenu twice
Summary

Popup menu was called from mousePressEvent.
No need as it is handled in contextMenuEvent.

Diff Detail

Repository
R8 Calligra
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
danders created this revision.Sep 17 2018, 7:50 AM
Restricted Application added a project: Calligra: 3.0. · View Herald TranscriptSep 17 2018, 7:50 AM
Restricted Application added a subscriber: Calligra-Devel-list. · View Herald Transcript
danders requested review of this revision.Sep 17 2018, 7:50 AM
leinir accepted this revision.Sep 17 2018, 7:54 AM
leinir added a subscriber: leinir.

Yes please, thank you! That has frankly been bugging me for a while... We're sure there's no side effects here? But, yeah, having two menus pop up underneath each other is sort of annoying ;)

This revision is now accepted and ready to land.Sep 17 2018, 7:54 AM
Closed by commit R8:7bd9b921826e: Words: Do not popup contextmenu twice (authored by danders). · Explain WhySep 21 2018, 12:43 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mwords/part/KWCanvas.cpp (4 lines)

Diff 42052

View Options

words/part/KWCanvas.cpp

Loading...