From 46c51107018125d1818221e72b131920ac48dc7f Mon Sep 17 00:00:00 2001
From: Gabriel Horner <gabriel@logseq.com>
Date: Tue, 30 Jul 2024 17:20:12 -0400
Subject: [PATCH] fix: document and validate a db only config.edn option

Also fix another outdated comment
---
 deps/common/resources/templates/config.edn          | 5 +++++
 deps/db/src/logseq/db/sqlite/build.cljs             | 2 +-
 src/main/frontend/schema/handler/common_config.cljc | 1 +
 src/main/frontend/state.cljs                        | 3 ++-
 4 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/deps/common/resources/templates/config.edn b/deps/common/resources/templates/config.edn
index 3995fe4ac..93a3a16ab 100644
--- a/deps/common/resources/templates/config.edn
+++ b/deps/common/resources/templates/config.edn
@@ -55,6 +55,11 @@
  ;; Default value: true
  :ui/auto-expand-block-refs? true
 
+ ;; Hide empty block properties
+ ;; This is _only_ for DB graphs.
+ ;; Default value: false
+ ;; :ui/hide-empty-properties? false
+
  ;; Disable accent marks when searching.
  ;; After changing this setting, rebuild the search index by pressing (^C ^S).
  ;; Default value: true
diff --git a/deps/db/src/logseq/db/sqlite/build.cljs b/deps/db/src/logseq/db/sqlite/build.cljs
index 7763b1400..8858af2da 100644
--- a/deps/db/src/logseq/db/sqlite/build.cljs
+++ b/deps/db/src/logseq/db/sqlite/build.cljs
@@ -555,7 +555,7 @@
    * :pages-and-blocks - This is a vector of maps containing a :page key and optionally a :blocks
      key when defining a page's blocks. More about each key:
      * :page - This is a datascript attribute map for pages with
-       :block/title required e.g. `{:block/original/name \"foo\"}`. Additional keys available:
+       :block/title required e.g. `{:block/title \"foo\"}`. Additional keys available:
        * :build/journal - Define a journal pages as an integer e.g. 20240101 is Jan 1, 2024. :block/title
          is not required if using this since it generates one
        * :build/properties - Defines properties on a page
diff --git a/src/main/frontend/schema/handler/common_config.cljc b/src/main/frontend/schema/handler/common_config.cljc
index 38ad703d2..c5c91f587 100644
--- a/src/main/frontend/schema/handler/common_config.cljc
+++ b/src/main/frontend/schema/handler/common_config.cljc
@@ -42,6 +42,7 @@
     [:ui/show-command-doc? :boolean]
     [:ui/show-empty-bullets? :boolean]
     [:ui/show-full-blocks? :boolean]
+    [:ui/hide-empty-properties? :boolean]
     [:ui/auto-expand-block-refs? :boolean]
     [:query/views [:map-of
                    :keyword
diff --git a/src/main/frontend/state.cljs b/src/main/frontend/state.cljs
index 8dff7557b..45c68d49a 100644
--- a/src/main/frontend/state.cljs
+++ b/src/main/frontend/state.cljs
@@ -424,7 +424,8 @@
                       [(< ?d ?next)]]
              :inputs [:today :7d-after]
              :group-by-page? false
-             :collapsed? false}]}}))
+             :collapsed? false}]}
+          :ui/hide-empty-properties? false}))
 
 ;; State that most user config is dependent on
 (declare get-current-repo sub set-state!)