Add sidebar editing mode with card toggles and tag reordering

- Allow toggling visibility of Insights and Places cards in sidebar
- Enable drag-and-drop reordering of user-created tags
- Add sidebar edit mode with controls for card visibility and tag order
- Update SettingsManager to persist sidebar card and tag order settings
- Improve tag row UI for editing and drag handle display
- Fix bug with All Entries row selection logic

Author: lainsce

data/style.css

@@ -70,6 +70,10 @@ window {
     color: var(--text-primary);
 }
 
+.selection-mode * {
+    border-radius: 999px;
+}
+
 /* --- Sidebar --- */
 .sidebar {
     background: var(--sidebar-bg);

src/settings/SettingsManager.vala

@@ -19,6 +19,11 @@
  *   "streak": {
  *     "enabled": true,
  *     "time": "20:00"
+ *   },
+ *   "sidebar": {
+ *      "insights_card_enabled": true,
+ *      "places_card_enabled": true,
+ *      "tag_order": []
  *   }
  * }
  */
@@ -42,6 +47,10 @@ namespace Notejot {
         private bool streak_enabled = false;
         private string streak_time = "20:00"; // 24h HH:MM
 
+        private bool insights_card_enabled = true;
+        private bool places_card_enabled = true;
+        private string[] tag_order = {};
+
         // Singleton
         private static SettingsManager? instance = null;
 
@@ -91,6 +100,18 @@ namespace Notejot {
             return this.streak_time;
         }
 
+        public bool get_insights_card_enabled () {
+            return this.insights_card_enabled;
+        }
+
+        public bool get_places_card_enabled () {
+            return this.places_card_enabled;
+        }
+
+        public string[] get_tag_order () {
+            return this.tag_order;
+        }
+
         // ---- Public setters (auto-save) ------------------------------------
 
         public void set_scheduling_enabled (bool enabled) {
@@ -128,6 +149,23 @@ namespace Notejot {
             save ();
         }
 
+        public void set_insights_card_enabled (bool enabled) {
+            if (this.insights_card_enabled == enabled)return;
+            this.insights_card_enabled = enabled;
+            save ();
+        }
+
+        public void set_places_card_enabled (bool enabled) {
+            if (this.places_card_enabled == enabled)return;
+            this.places_card_enabled = enabled;
+            save ();
+        }
+
+        public void set_tag_order (string[] order) {
+            this.tag_order = order;
+            save ();
+        }
+
         // Convenience batch updates (auto-save)
         public void update_scheduling (bool enabled, string time_hhmm, int[] days_of_week) {
             this.scheduling_enabled = enabled;
@@ -145,58 +183,73 @@ namespace Notejot {
         // ---- Persistence ----------------------------------------------------
 
         private void load () {
-            try {
-                if (!GLib.FileUtils.test (this.settings_path, GLib.FileTest.EXISTS)) {
-                    return; // keep defaults
-                }
+            if (!GLib.FileUtils.test (this.settings_path, GLib.FileTest.EXISTS)) {
+                return; // keep defaults
+            }
 
-                string contents;
-                GLib.FileUtils.get_contents (this.settings_path, out contents);
+            string contents;
+            GLib.FileUtils.get_contents (this.settings_path, out contents);
 
-                var parser = new Json.Parser ();
-                parser.load_from_data (contents);
-                var root_node = parser.get_root ();
-                if (root_node.get_node_type () != Json.NodeType.OBJECT)return;
+            var parser = new Json.Parser ();
+            parser.load_from_data (contents);
+            var root_node = parser.get_root ();
+            if (root_node.get_node_type () != Json.NodeType.OBJECT)return;
 
-                var root = root_node.get_object ();
+            var root = root_node.get_object ();
 
-                // Scheduling
-                if (root.has_member ("scheduling")) {
-                    var sched = root.get_object_member ("scheduling");
+            // Scheduling
+            if (root.has_member ("scheduling")) {
+                var sched = root.get_object_member ("scheduling");
 
-                    if (sched.has_member ("enabled")) {
-                        this.scheduling_enabled = sched.get_boolean_member ("enabled");
-                    }
-                    if (sched.has_member ("time")) {
-                        var t = sched.get_string_member ("time");
-                        this.scheduling_time = normalize_time_or_fallback (t, this.scheduling_time);
-                    }
-                    if (sched.has_member ("days_of_week")) {
-                        var arr = sched.get_array_member ("days_of_week");
-                        int[] tmp = {};
-                        foreach (var el in arr.get_elements ()) {
-                            int v = (int) el.get_int ();
-                            tmp += v;
-                        }
-                        this.scheduling_days = sanitize_days (tmp);
+                if (sched.has_member ("enabled")) {
+                    this.scheduling_enabled = sched.get_boolean_member ("enabled");
+                }
+                if (sched.has_member ("time")) {
+                    var t = sched.get_string_member ("time");
+                    this.scheduling_time = normalize_time_or_fallback (t, this.scheduling_time);
+                }
+                if (sched.has_member ("days_of_week")) {
+                    var arr = sched.get_array_member ("days_of_week");
+                    int[] tmp = {};
+                    foreach (var el in arr.get_elements ()) {
+                        int v = (int) el.get_int ();
+                        tmp += v;
                     }
+                    this.scheduling_days = sanitize_days (tmp);
                 }
+            }
 
-                // Streak
-                if (root.has_member ("streak")) {
-                    var st = root.get_object_member ("streak");
+            // Streak
+            if (root.has_member ("streak")) {
+                var st = root.get_object_member ("streak");
 
-                    if (st.has_member ("enabled")) {
-                        this.streak_enabled = st.get_boolean_member ("enabled");
-                    }
-                    if (st.has_member ("time")) {
-                        var t = st.get_string_member ("time");
-                        this.streak_time = normalize_time_or_fallback (t, this.streak_time);
+                if (st.has_member ("enabled")) {
+                    this.streak_enabled = st.get_boolean_member ("enabled");
+                }
+                if (st.has_member ("time")) {
+                    var t = st.get_string_member ("time");
+                    this.streak_time = normalize_time_or_fallback (t, this.streak_time);
+                }
+            }
+
+            // Sidebar
+            if (root.has_member ("sidebar")) {
+                var sidebar = root.get_object_member ("sidebar");
+
+                if (sidebar.has_member ("insights_card_enabled")) {
+                    this.insights_card_enabled = sidebar.get_boolean_member ("insights_card_enabled");
+                }
+                if (sidebar.has_member ("places_card_enabled")) {
+                    this.places_card_enabled = sidebar.get_boolean_member ("places_card_enabled");
+                }
+                if (sidebar.has_member ("tag_order")) {
+                    var arr = sidebar.get_array_member ("tag_order");
+                    string[] tmp = {};
+                    foreach (var el in arr.get_elements ()) {
+                        tmp += el.get_string ();
                     }
+                    this.tag_order = tmp;
                 }
-            } catch (Error e) {
-                warning ("Failed to load settings: %s", e.message);
-                // Keep defaults on failure
             }
         }
 
@@ -223,6 +276,18 @@ namespace Notejot {
             streak_obj.set_string_member ("time", this.streak_time);
             root_obj.set_object_member ("streak", streak_obj);
 
+            // Sidebar object
+            var sidebar_obj = new Json.Object ();
+            sidebar_obj.set_boolean_member ("insights_card_enabled", this.insights_card_enabled);
+            sidebar_obj.set_boolean_member ("places_card_enabled", this.places_card_enabled);
+
+            var tag_order_array = new Json.Array ();
+            foreach (var uuid in this.tag_order) {
+                tag_order_array.add_string_element (uuid);
+            }
+            sidebar_obj.set_array_member ("tag_order", tag_order_array);
+            root_obj.set_object_member ("sidebar", sidebar_obj);
+
             // Write pretty JSON
             var generator = new Json.Generator ();
             generator.set_pretty (true);

src/views/EntriesView.vala

@@ -76,17 +76,17 @@ namespace Notejot {
                 entries_to_show = this.data_manager.get_entries (true);
             } else if (this.current_tag_uuid == "pinned") {
                 entries_to_show = this.data_manager.get_pinned_entries ();
-            } else if (this.current_tag_uuid == null) {
+            } else if (this.current_tag_uuid == "") {
                 entries_to_show = this.data_manager.get_entries (false);
             } else {
                 entries_to_show = this.data_manager.get_entries_for_tag (this.current_tag_uuid);
             }
 
-            var q = "";
+            string? q = null;
             if (this.search_entry != null) {
                 q = this.search_entry.get_text ().strip ();
             }
-            if (q != "") {
+            if (q != null) {
                 try {
                     var pattern = GLib.Regex.escape_string (q, -1);
                     var regex = new GLib.Regex (pattern, GLib.RegexCompileFlags.CASELESS | GLib.RegexCompileFlags.OPTIMIZE, 0);