Skip to content

Commit f83fdee

Browse files
committed
fix NaN displayed for empty cells in number columns
1 parent e96048b commit f83fdee

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

sqlpage/sqlpage.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,9 +80,10 @@ function apply_number_formatting(table_el) {
8080
const column_type = col_types[idx];
8181
const is_raw_number = col_rawnums[idx];
8282
const cell_el = cells[idx];
83+
const text = cell_el.textContent;
8384

84-
if (column_type === "number" && !is_raw_number) {
85-
const num = Number.parseFloat(cell_el.textContent);
85+
if (column_type === "number" && !is_raw_number && text) {
86+
const num = Number.parseFloat(text);
8687
const is_money = col_money[idx];
8788
cell_el.textContent = num.toLocaleString(number_format_locale, {
8889
maximumFractionDigits: number_format_digits,

sqlpage/sqlpage.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
{
2-
"database_url": "sqlite://./sqlpage/sqlpage.db?mode=rwc"
2+
"database_url": "sqlite:///tmp/bug.db?mode=rwc"
33
}

0 commit comments

Comments
 (0)