From 5cd368792ceb7361fcf81ee98ec8281857139608 Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Wed, 25 Oct 2017 17:58:55 -0300
Subject: [PATCH] Improve UI so undo limit = 0 means "no limit" (#1127)

By default we'll have "no undo limit".
---
 data/pref.xml                    |  2 +-
 data/widgets/options.xml         |  4 ++--
 src/app/commands/cmd_options.cpp | 20 ++++++++++++++++++--
 3 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/data/pref.xml b/data/pref.xml
index c7c7a8d705..6270be672e 100644
--- a/data/pref.xml
+++ b/data/pref.xml
@@ -113,7 +113,7 @@
       <option id="show_menu_bar" type="bool" default="true" />
     </section>
     <section id="undo" text="Undo">
-      <option id="size_limit" type="int" default="64" />
+      <option id="size_limit" type="int" default="0" />
       <option id="goto_modified" type="bool" default="true" />
       <option id="allow_nonlinear_history" type="bool" default="false" />
     </section>
diff --git a/data/widgets/options.xml b/data/widgets/options.xml
index 683472e611..8238c63700 100644
--- a/data/widgets/options.xml
+++ b/data/widgets/options.xml
@@ -249,8 +249,8 @@
         <vbox id="section_undo">
           <separator text="@.section_undo" horizontal="true" />
           <hbox>
-            <label text="@.undo_size_limit" />
-            <entry id="undo_size_limit" maxsize="4" tooltip="@.undo_size_limit_tooltip" />
+            <check id="limit_undo" text="@.undo_size_limit" />
+            <entry id="undo_size_limit" maxsize="6" tooltip="@.undo_size_limit_tooltip" />
             <label text="@.undo_mb" />
           </hbox>
 
diff --git a/src/app/commands/cmd_options.cpp b/src/app/commands/cmd_options.cpp
index be19d43b83..3630f63cdc 100644
--- a/src/app/commands/cmd_options.cpp
+++ b/src/app/commands/cmd_options.cpp
@@ -44,6 +44,8 @@ static const char* kSectionGridId = "section_grid";
 static const char* kSectionThemeId = "section_theme";
 static const char* kSectionExtensionsId = "section_extensions";
 
+static const char* kInfiniteSymbol = "\xE2\x88\x9E"; // Infinite symbol (UTF-8)
+
 using namespace ui;
 
 class OptionsWindow : public app::gen::Options {
@@ -321,7 +323,10 @@ class OptionsWindow : public app::gen::Options {
 #endif
 
     // Undo preferences
-    undoSizeLimit()->setTextf("%d", m_pref.undo.sizeLimit());
+    limitUndo()->Click.connect(base::Bind<void>(&OptionsWindow::onLimitUndoCheck, this));
+    limitUndo()->setSelected(m_pref.undo.sizeLimit() != 0);
+    onLimitUndoCheck();
+
     undoGotoModified()->setSelected(m_pref.undo.gotoModified());
     undoAllowNonlinearHistory()->setSelected(m_pref.undo.allowNonlinearHistory());
 
@@ -418,7 +423,7 @@ class OptionsWindow : public app::gen::Options {
 
     int undo_size_limit_value;
     undo_size_limit_value = undoSizeLimit()->textInt();
-    undo_size_limit_value = MID(1, undo_size_limit_value, 9999);
+    undo_size_limit_value = MID(0, undo_size_limit_value, 999999);
 
     m_pref.undo.sizeLimit(undo_size_limit_value);
     m_pref.undo.gotoModified(undoGotoModified()->isSelected());
@@ -655,6 +660,17 @@ class OptionsWindow : public app::gen::Options {
     app::launcher::open_folder(app::main_config_filename());
   }
 
+  void onLimitUndoCheck() {
+    if (limitUndo()->isSelected()) {
+      undoSizeLimit()->setEnabled(true);
+      undoSizeLimit()->setTextf("%d", m_pref.undo.sizeLimit());
+    }
+    else {
+      undoSizeLimit()->setEnabled(false);
+      undoSizeLimit()->setText(kInfiniteSymbol);
+    }
+  }
+
   void reloadThemes() {
     while (themeList()->firstChild())
       delete themeList()->lastChild();