From 8984531a8c5dfeb7ef553f90f9f1da9f19ba71d1 Mon Sep 17 00:00:00 2001
From: Matthias Koefferlein <matthias@klayout.de>
Date: Sun, 17 Mar 2024 18:36:49 +0100
Subject: [PATCH] Small enhancements of netlist browser: multiple items can be
 selected in log view (for manyfold geometries), highlights are cleared on tab
 change

---
 src/layui/layui/NetlistBrowserPage.ui    | 5 ++++-
 src/layui/layui/layNetlistBrowserPage.cc | 9 +++++++++
 src/layui/layui/layNetlistBrowserPage.h  | 1 +
 3 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/src/layui/layui/NetlistBrowserPage.ui b/src/layui/layui/NetlistBrowserPage.ui
index 88687f10e..57a2e16e0 100644
--- a/src/layui/layui/NetlistBrowserPage.ui
+++ b/src/layui/layui/NetlistBrowserPage.ui
@@ -134,7 +134,7 @@
             <string>...</string>
            </property>
            <property name="icon">
-            <iconset>
+            <iconset resource="../../icons/icons.qrc">
              <normaloff>:/run_16px.png</normaloff>:/run_16px.png</iconset>
            </property>
            <property name="shortcut">
@@ -576,6 +576,9 @@
             <property name="contextMenuPolicy">
              <enum>Qt::ActionsContextMenu</enum>
             </property>
+            <property name="selectionMode">
+             <enum>QAbstractItemView::ExtendedSelection</enum>
+            </property>
             <property name="headerHidden">
              <bool>true</bool>
             </property>
diff --git a/src/layui/layui/layNetlistBrowserPage.cc b/src/layui/layui/layNetlistBrowserPage.cc
index 66cbdec49..9a6d5dc7b 100644
--- a/src/layui/layui/layNetlistBrowserPage.cc
+++ b/src/layui/layui/layNetlistBrowserPage.cc
@@ -235,6 +235,8 @@ NetlistBrowserPage::NetlistBrowserPage (QWidget * /*parent*/)
   connect (actionExportAll, SIGNAL (triggered ()), this, SLOT (export_all ()));
   connect (actionExportSelected, SIGNAL (triggered ()), this, SLOT (export_selected ()));
 
+  connect (mode_tab, SIGNAL (currentChanged (int)), this, SLOT (mode_tab_changed (int)));
+
   forward->setEnabled (false);
   backward->setEnabled (false);
 }
@@ -330,6 +332,13 @@ NetlistBrowserPage::eventFilter (QObject *watched, QEvent *event)
   }
 }
 
+void
+NetlistBrowserPage::mode_tab_changed (int)
+{
+  clear_highlights ();
+  dm_update_highlights ();
+}
+
 void
 NetlistBrowserPage::layer_list_changed (int)
 {
diff --git a/src/layui/layui/layNetlistBrowserPage.h b/src/layui/layui/layNetlistBrowserPage.h
index 5b7a057e4..b051e30f2 100644
--- a/src/layui/layui/layNetlistBrowserPage.h
+++ b/src/layui/layui/layNetlistBrowserPage.h
@@ -214,6 +214,7 @@ private slots:
   void log_selection_changed ();
   void browse_color_for_net ();
   void select_color_for_net ();
+  void mode_tab_changed (int);
 
 protected:
   bool eventFilter (QObject *watched, QEvent *event);