Changeset 71423c

Timestamp:

Oct 31, 2025, 10:21:54 PM (2 months ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Candidate_v1.7.1, stable

Children:

49719b

Parents:

511f02

git-author:

Frederik Heber <frederik.heber@…> (10/31/25 22:10:16)

git-committer:

Frederik Heber <frederik.heber@…> (10/31/25 22:21:54)

Message:

FIX: Context menu destroyed hover functionality.

• The right click activating the context menu was not accepted as an event. This caused the hovering over GLMoleculeObject functionality to break: Whenever the context menu was opened over an atom that we hover over, then the hover would remain active on that atom forever. No other atom can be selected,... This severely limited functionality.

Location:

src/UIElements

Files:

• Qt4/QtMainWindow.cpp (modified) (2 diffs)
• Views/Qt4/Qt3D/GLWorldView.cpp (modified) (1 diff)

src/UIElements/Qt4/QtMainWindow.cpp

@@ -138 +138 @@
 
   // add context menu on right click
-  glWorldView->setContextMenuPolicy(Qt::CustomContextMenu);
+  // NOTE(FH): The following did not emit the customContextMenuRequested on right click, had to trigger manually
+  // glWorldView->setContextMenuPolicy(Qt::CustomContextMenu);
   connect(glWorldView, SIGNAL(customContextMenuRequested(const QPoint&)), this, SLOT(slotContextMenuRequested(const QPoint&)));
 
@@ -211 +212 @@
   }
 }
+
 QtMainWindow::~QtMainWindow()
 {

src/UIElements/Views/Qt4/Qt3D/GLWorldView.cpp

@@ -609 +609 @@
   // check for right mouse button
   if (event->button() == Qt::RightButton) {
+    // NOTE(FH): simply setting setContextMenuPolicy(Qt::CustomContextMenu) did not emit the signal ...
     emit customContextMenuRequested(event->pos());
+    event->accept();
+    return;
   }