On Fri, 30 Nov 2018 15:38:11 +0000 Yordan Karadzhov <ykaradzhov@xxxxxxxxxx> wrote: > The "Y" coordinate of the Quick Context Menu has to be corrected in > the case when the menu gets opened from the Graph widget. This is > needed because the Graph widget is nested inside a scroll area and > we have to take into account the offset of the vertical scrollbar. > > Signed-off-by: Yordan Karadzhov <ykaradzhov@xxxxxxxxxx> Thanks! I applied both patches. -- Steve > --- > kernel-shark-qt/src/KsTraceGraph.cpp | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/kernel-shark-qt/src/KsTraceGraph.cpp b/kernel-shark-qt/src/KsTraceGraph.cpp > index 0b5a8b1..858930c 100644 > --- a/kernel-shark-qt/src/KsTraceGraph.cpp > +++ b/kernel-shark-qt/src/KsTraceGraph.cpp > @@ -782,6 +782,12 @@ void KsTraceGraph::_onCustomContextMenu(const QPoint &point) > connect(menu, &KsQuickMarkerMenu::deselect, > this, &KsTraceGraph::deselect); > > - menu->exec(mapToGlobal(point)); > + QPoint global = mapToGlobal(point); > + /* > + * The global coordinates have to be corrected for the offset > + * of the vertical scrollbar. > + */ > + global.ry() -= _scrollArea.verticalScrollBar()->value(); > + menu->exec(global); > } > }
|