[simple-v-table] Fix null-ref with sortable headers in top-left

day8 · May 8, 2024 · 17e845b · 17e845b
1 parent 1677ea6
commit 17e845b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,11 @@
 > Committed but unreleased changes are put here, at the top. Older releases are detailed chronologically below. 
 
 
+## 2.21.3 (2024-05-09)
+
+#### Fixed
+
+- Crash when declaring :sort-by for simple-v-table headers in top-left section
 
 ## 2.21.2 (2024-05-09)
 

diff --git a/src/re_com/simple_v_table.cljs b/src/re_com/simple_v_table.cljs
@@ -338,8 +338,9 @@
                       ;:max-width               (px (or max-width (+ fixed-content-width content-width v-table/scrollbar-tot-thick))) ; :max-width handled by enclosing parent above
                         ;; ===== Corners (section 1, 3)
                       :top-left-renderer       (fn [i row] [column-header-renderer {:columns        fixed-cols
-                                                                                    :parts          parts
-                                                                                    :sort-by-column sort-by-column}]) ;; Used when there are fixed columns
+                                                                                   :hover?         header-hover?
+                                                                                   :parts          parts
+                                                                                   :sort-by-column sort-by-column}]) ;; Used when there are fixed columns
                       :top-right-renderer      (when show-export-button?
                                                  #(let [rows           (deref-or-value model)
                                                         columns        (deref-or-value columns)