From b49fc1f2f5d9416f017c97a8216fb48be3680331 Mon Sep 17 00:00:00 2001
From: Kimo Knowles <kimo.knowles@day8.com.au>
Date: Thu, 9 Jan 2025 13:45:49 +0100
Subject: [PATCH] [nested-v-grid] polish theme for cells & headers

---
 src/re_com/nested_v_grid.cljs       |  5 +--
 src/re_com/nested_v_grid/theme.cljs | 65 +++++++++++------------------
 2 files changed, 25 insertions(+), 45 deletions(-)

diff --git a/src/re_com/nested_v_grid.cljs b/src/re_com/nested_v_grid.cljs
index 9f2a81dd..a2f77bfd 100644
--- a/src/re_com/nested_v_grid.cljs
+++ b/src/re_com/nested_v_grid.cljs
@@ -358,10 +358,7 @@
                                         (row-height-resizers {:offset -1})
                                         (column-width-resizers {:style  {:grid-row-end -1}
                                                                 :offset -1}))
-                      :style    {:display               :grid
-                                 :grid-column-start     2
-                                 :grid-row-start        2
-                                 :grid-template-rows    @row-template
+                      :style    {:grid-template-rows    @row-template
                                  :grid-template-columns @column-template}}))
            (u/part column-header-grid
                    (themed ::column-header-grid
diff --git a/src/re_com/nested_v_grid/theme.cljs b/src/re_com/nested_v_grid/theme.cljs
index 290e2947..ab7f4e3a 100644
--- a/src/re_com/nested_v_grid/theme.cljs
+++ b/src/re_com/nested_v_grid/theme.cljs
@@ -1,16 +1,13 @@
 (ns re-com.nested-v-grid.theme
   (:require
    [re-com.theme.default :as default :refer [base main]]
-   [re-com.nested-v-grid :as-alias nvg]
-   [re-com.theme.util :refer [merge-props merge-class]]))
+   [re-com.nested-v-grid :as-alias nvg]))
 
 (def border-light "thin solid #ccc")
 (def border-dark "thin solid #aaa")
 
 (defn style [props & ms] (apply update props :style merge ms))
 
-(defn class [props & ss] (apply update props :class merge-class ss))
-
 (defmethod base ::nvg/wrapper [props _]
   (style props {:height   300
                 :width    500
@@ -20,28 +17,21 @@
 
 (defmethod base ::nvg/cell-grid
   [props _]
-  (class props "rc-nested-v-grid-cell-grid"))
-
-(defmethod main ::nvg/cell-grid
-  [props _]
-  (style props {#_#_:border-top  light-border
-                #_#_:border-left light-border}))
+  (style props {:display           :grid
+                :grid-row-start    2
+                :grid-column-start 2}))
 
 (defmethod base ::nvg/column-header-grid
   [props _]
-  (-> props
-      (class "rc-nested-v-grid-column-header-grid")
-      (style {:display  :grid
-              :position :sticky
-              :top      0})))
+  (style props {:display  :grid
+                :position :sticky
+                :top      0}))
 
 (defmethod base ::nvg/row-header-grid
   [props _]
-  (-> props
-      (class "rc-nested-v-grid-row-header-grid")
-      (style {:display  :grid
-              :position :sticky
-              :left      0})))
+  (style props {:display  :grid
+                :position :sticky
+                :left      0}))
 
 (defmethod base ::nvg/corner-header-grid
   [props _]
@@ -53,14 +43,17 @@
                 :top               0}))
 
 (def header-main
-  (let [{:keys [sm-3 sm-4]}               default/golden-section-50
+  (let [{:keys [sm-3 sm-4]}        default/golden-section-50
         {:keys [light-background]} default/colors]
     {:padding-top      sm-3
      :padding-right    sm-4
      :padding-left     sm-4
      :background-color light-background
      :color            "#666"
-     :font-size        "13px"}))
+     :font-size        "13px"
+     :text-overflow    :ellipsis
+     :overflow         :hidden
+     :white-space      :nowrap}))
 
 (defmethod main ::nvg/corner-header
   [{:keys [edge] :as props} _]
@@ -79,27 +72,17 @@
            :border-bottom border-dark}))
 
 (defmethod main ::nvg/column-header
-  [props {:keys [state]}]
-  (let [{:keys [align-column align-column-header align]} (:header-spec state)]
+  [{:keys [column-path] :as props} _]
+  (let [{:keys [align-column align-column-header align]} (peek column-path)]
     (style props header-main
            {:text-align (or align-column-header align-column align :center)})))
 
-(def row-header-main
-  (let [{:keys [sm-3 sm-6]}               default/golden-section-50
-        {:keys [border light-background]} default/colors]
-    {:padding-top      sm-3
-     :padding-right    sm-3
-     :padding-left     sm-6
-     :background-color light-background
-     :color            "#666"
-     :text-align       "left"
-     :font-size        "13px"
-     :white-space      "nowrap"}))
-
 (defmethod main ::nvg/row-header
   [props _]
-  (style props row-header-main
-         {:border-right border-dark}))
+  (style props header-main
+         {:border-right border-dark
+          :font-size    "13px"
+          :text-align   "left"}))
 
 (def cell-main
   (let [{:keys [sm-3]} default/golden-section-50]
@@ -109,9 +92,9 @@
      :padding-top      sm-3
      :padding-right    sm-3
      :padding-left     sm-3
-     :text-align :right
-     :border-right border-light
-     :border-bottom border-light}))
+     :text-align       :right
+     :border-right     border-light
+     :border-bottom    border-light}))
 
 (defmethod main ::nvg/cell
   [props {{:keys [edge value column-path]} :state}]