From 17e845b6b5cf5cd2b386f63a2c1cbe5776d66dff Mon Sep 17 00:00:00 2001 From: Kimo Knowles Date: Wed, 8 May 2024 23:41:39 +0200 Subject: [PATCH] [simple-v-table] Fix null-ref with sortable headers in top-left --- CHANGELOG.md | 5 +++++ src/re_com/simple_v_table.cljs | 5 +++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 446c9fbe..29932a71 100644 --- 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 index 72e17aa2..b798ade1 100644 --- 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)