Merge pull request #1809 from astrofrog/unglue-multiple

Make it possible to unglue multiple links in one go

CHANGES.md

@@ -24,6 +24,8 @@ v0.14.0 (unreleased)
 
   - ``glue.viewers.common.qt.DataViewerWithState`` is now deprecated.
 
+* Make it possible to unglue multiple links in one go. [#1809]
+
 * Make it so that adding a subset to a viewer no longer adds the
   associated data, since in some cases the viewer can handle the
   subset size, but not the full data. [#1807]

glue/dialogs/link_editor/qt/link_editor.py

@@ -127,14 +127,11 @@ def links(self):
         return self._links
 
     def _remove_link(self):
-
-        current = self._ui.current_links.currentItem()
-        if current is None:
+        if self._ui.current_links.currentItem() is None:
             return
-        link = current.data(0, Qt.UserRole)
-
-        self._links.remove(link)
-
+        for item in self._ui.current_links.selectedItems():
+            link = item.data(0, Qt.UserRole)
+            self._links.remove(link)
         self._ui.graph_widget.set_links(self._links)
         self._update_links_list()
 

glue/dialogs/link_editor/qt/link_editor.ui

@@ -137,6 +137,9 @@
          <height>0</height>
         </size>
        </property>
+       <property name="selectionMode">
+        <enum>QAbstractItemView::MultiSelection</enum>
+       </property>
        <column>
         <property name="text">
          <string>Function</string>