Bubble: handle link/selection cases of context menu on c++ side

psi-im · Jul 11, 2024 · 449720c · 449720c
1 parent 3429937
commit 449720c
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/src/psirosterwidget.cpp b/src/psirosterwidget.cpp
@@ -151,6 +151,7 @@ PsiRosterWidget::PsiRosterWidget(QWidget *parent) :
     connect(filterPageView_, SIGNAL(quitFilteringMode()), SLOT(quitFilteringMode()));
     filterPageView_->installEventFilter(this);
     filterPageLayout->addWidget(filterPageView_);
+    connect(filterPageView_, &ContactListView::contactSelected, this, &PsiRosterWidget::contactPick);
 }
 
 PsiRosterWidget::~PsiRosterWidget() { }
@@ -317,6 +318,8 @@ void PsiRosterWidget::setPickContactMode(bool value)
     filterPageView_->setActivateAction(value ? ContactListView::SignalSelected : ContactListView::Activate);
 }
 
+QList<PsiContact *> PsiRosterWidget::selectedContacts() const { return {}; }
+
 void PsiRosterWidget::clearFilterEdit()
 {
     if (filterEdit_->text().isEmpty() && !pickContactMode_)

diff --git a/src/psirosterwidget.h b/src/psirosterwidget.h
@@ -41,7 +41,8 @@ class PsiRosterWidget : public QWidget {
 
     void setContactList(PsiContactList *contactList);
 
-    void setPickContactMode(bool);
+    void                setPickContactMode(bool);
+    QList<PsiContact *> selectedContacts() const;
 signals:
     void contactPick(PsiContact *);
 

diff --git a/themes/chatview/psi/bubble/index.html b/themes/chatview/psi/bubble/index.html
@@ -134,6 +134,9 @@
 
 function setupContextMenu() {
     chatMenu.addItemProvider((event) => {
+        if (event.target.href || getSelection().toString()) {
+            return []; // C++ will handle this
+        }
         const isNick = event.target.classList.contains("nick");
         let msgNode = undefined;
         let msgGroup = isNick? event.target.parentElement.parentElement : event.target;