From ec566c4a3bfe5786e0769fab80ad0b707c15db65 Mon Sep 17 00:00:00 2001
From: shenleban tongying <shenlebantongying@gmail.com>
Date: Sat, 1 Jul 2023 23:41:05 -0400
Subject: [PATCH]  misc fts fixes

 * config fts.enablePosition is not updated
 * update preferences layout
---
 src/ui/preferences.cc |  3 ++-
 src/ui/preferences.ui | 22 +++++++++++-----------
 2 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/src/ui/preferences.cc b/src/ui/preferences.cc
index 92d1ffc1..da5e328b 100644
--- a/src/ui/preferences.cc
+++ b/src/ui/preferences.cc
@@ -465,8 +465,9 @@ Config::Preferences Preferences::getPreferences()
 
   p.addonStyle = ui.addonStyles->getCurrentStyle();
 
-  p.fts.enabled = ui.ftsGroupBox->isChecked();
+  p.fts.enabled           = ui.ftsGroupBox->isChecked();
   p.fts.maxDictionarySize = ui.maxDictionarySize->value();
+  p.fts.enablePosition    = ui.enablePosition->isChecked();
 
   buildDisabledTypes( p.fts.disabledTypes, ui.allowAard->isChecked(), "AARD" );
   buildDisabledTypes( p.fts.disabledTypes, ui.allowBGL->isChecked(), "BGL" );
diff --git a/src/ui/preferences.ui b/src/ui/preferences.ui
index 883a5cc1..91ba2587 100644
--- a/src/ui/preferences.ui
+++ b/src/ui/preferences.ui
@@ -1296,16 +1296,6 @@ download page.</string>
             </property>
            </widget>
           </item>
-          <item row="7" column="0">
-           <widget class="QCheckBox" name="enablePosition">
-            <property name="toolTip">
-             <string>Positional information is required to use Xapian's phrase searching and NEAR operator, but the database size will be much bigger. Applies only to new incoming dictionaries.</string>
-            </property>
-            <property name="text">
-             <string>Enable index with positional information</string>
-            </property>
-           </widget>
-          </item>
           <item row="0" column="0">
            <widget class="QCheckBox" name="allowAard">
             <property name="text">
@@ -1362,7 +1352,7 @@ download page.</string>
             </property>
            </widget>
           </item>
-          <item row="6" column="0">
+          <item row="6" column="0" colspan="2">
            <layout class="QHBoxLayout" name="horizontalLayout_14">
             <item>
              <widget class="QLabel" name="label_14">
@@ -1403,6 +1393,16 @@ download page.</string>
             </item>
            </layout>
           </item>
+          <item row="7" column="0" colspan="2">
+           <widget class="QCheckBox" name="enablePosition">
+            <property name="toolTip">
+             <string>Positional information is required to use Xapian's phrase searching and NEAR operator, but the database size will be much bigger. Applies only to new incoming dictionaries.</string>
+            </property>
+            <property name="text">
+             <string>Enable index with positional information</string>
+            </property>
+           </widget>
+          </item>
          </layout>
         </widget>
        </item>