Bug 682 - Context menu event can be ignored on channel labels and flags
Summary: Context menu event can be ignored on channel labels and flags
Status: CONFIRMED
Alias: None
Product: PulseView
Classification: Unclassified
Component: Other (show other bugs)
Version: unreleased development snapshot
Hardware: All All
: Normal normal
Target Milestone: ---
Assignee: Nobody
URL:
Keywords: low_hanging_fruit
Depends on:
Blocks:
 
Reported: 2015-10-30 21:33 CET by jry
Modified: 2017-12-23 01:08 CET (History)
2 users (show)


Attachments
Use event coordinates instead of cached mouse_point_ (878 bytes, patch)
2015-10-30 21:37 CET, jry 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description jry 2015-10-30 21:33:11 CET 
How to reproduce: 
-right-click channel label (or flag label), context menu with Delete command is displayed
-move mouse to another label and right-click again, so context menu is closed
-don't move mouse cursor and right-click again
-context menu is not displayed

Reason: mouse_point_ is used in contextMenuEvent() handlers instead of event->pos(). In described situation mouse_point_ contains (-1, -1) from ViewWidget::leaveEvent(QEvent*).

Patch is attached.
Comment 1 jry 2015-10-30 21:37:20 CET 
Created attachment 174 [details]
Use event coordinates instead of cached mouse_point_