From 5363521edabf3647e4a66a36f1f6ad93d6d5747c Mon Sep 17 00:00:00 2001
From: Damien Elmes <git@ichi2.net>
Date: Mon, 27 Oct 2008 19:02:06 +0900
Subject: [PATCH] inverse order of active tag selection

---
 ankiqt/ui/activetags.py | 6 +++---
 ankiqt/ui/facteditor.py | 2 --
 designer/activetags.ui  | 6 +++---
 3 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/ankiqt/ui/activetags.py b/ankiqt/ui/activetags.py
index 0e60f649b..9dd6d227f 100644
--- a/ankiqt/ui/activetags.py
+++ b/ankiqt/ui/activetags.py
@@ -32,9 +32,9 @@ class ActiveTagsChooser(QDialog):
             self.items.append(item)
             idx = self.dialog.list.indexFromItem(item)
             if t in self.suspended:
-                mode = QItemSelectionModel.Deselect
-            else:
                 mode = QItemSelectionModel.Select
+            else:
+                mode = QItemSelectionModel.Deselect
             self.dialog.list.selectionModel().select(idx, mode)
 
     def accept(self):
@@ -42,7 +42,7 @@ class ActiveTagsChooser(QDialog):
         suspended = []
         for item in self.items:
             idx = self.dialog.list.indexFromItem(item)
-            if not self.dialog.list.selectionModel().isSelected(idx):
+            if self.dialog.list.selectionModel().isSelected(idx):
                 suspended.append(self.tags[n])
             n += 1
         self.parent.deck.suspended = joinTags(suspended + ["Suspended"])
diff --git a/ankiqt/ui/facteditor.py b/ankiqt/ui/facteditor.py
index d959db0ff..d7eb7d974 100644
--- a/ankiqt/ui/facteditor.py
+++ b/ankiqt/ui/facteditor.py
@@ -266,8 +266,6 @@ class FactEditor(object):
         if value and not value.strip():
             widget.setText("")
             value = u""
-        if self.fact[field.name] == value:
-            return
         self.fact[field.name] = value
         self.fact.setModified(textChanged=True)
         self.deck.setModified()
diff --git a/designer/activetags.ui b/designer/activetags.ui
index 5391b82ed..6915ec8d1 100644
--- a/designer/activetags.ui
+++ b/designer/activetags.ui
@@ -5,8 +5,8 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>202</width>
-    <height>265</height>
+    <width>248</width>
+    <height>268</height>
    </rect>
   </property>
   <property name="windowTitle" >
@@ -16,7 +16,7 @@
    <item>
     <widget class="QLabel" name="label" >
      <property name="text" >
-      <string>&lt;h1>Select Active Tags&lt;/h1></string>
+      <string>&lt;h1>Select tags to suspend&lt;/h1></string>
      </property>
     </widget>
    </item>