Bug 320521 - Get rid of old setupPopupMenu / slotRemovePopupMenu API
Summary: Get rid of old setupPopupMenu / slotRemovePopupMenu API
Status: RESOLVED FIXED
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Andi Fischer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-05-31 08:38 UTC by Andi Fischer
Modified: 2013-11-02 17:23 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.11.0
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Andi Fischer 2013-05-31 08:38:14 UTC 
Most widgets still uses the old setupPopupMenu / slotRemovePopupMenu style for showing context menus.
Instead it should use the Qt API event handler contextMenuEvent() method.


Reproducible: Always

Steps to Reproduce:
1. Open umbrello/widgets/umlwidget.h and look for "setupPopupMenu(".
2. In source tree grep for method "setupPopupMenu(".
3.
Actual Results:  
You will find several locations where this methods have been used.

Expected Results:  
Where a context menu is shown the contextMenuEvent method should be used.

See QGraphicsItem API.
Comment 1 Andi Fischer 2013-06-13 18:26:19 UTC 
Fixed in
94137ee9fe30c2d5a402ab37ea50e61daaa31bb4