Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@ Conventional Commits by release-please.

## Unreleased

- The dashboard's Add words sheet no longer shows a stray "null" beside "Learning" when you
read one language.

## [1.0.1](https://github.com/ScriptKittyOS/kotiko/compare/v1.0.0...v1.0.1) (2026-10-08)

The first version in the stores. It fixes the package so the Chrome Web Store accepts it;
Expand Down
3 changes: 2 additions & 1 deletion extension/bulk/sheet.js
Original file line number Diff line number Diff line change
Expand Up @@ -351,7 +351,8 @@
: null;
controls.replaceChildren(
el("span", { class: "bulk-control" }, el("span", { class: "field-label" }, t("bulk_learning")), learn),
meanings ? el("label", { class: "bulk-control" }, el("span", { class: "field-label" }, t("bulk_meanings_in")), meanings) : null,
// Only with more than one language read: replaceChildren would write a null as "null".
...(meanings ? [el("label", { class: "bulk-control" }, el("span", { class: "field-label" }, t("bulk_meanings_in")), meanings)] : []),
);
}

Expand Down
1 change: 1 addition & 0 deletions test/dom/dashboard.test.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -1329,6 +1329,7 @@ describe("bulk add (slice 13)", () => {
test("200 lines of 'native = meaning': all ready, saved in one batch with no lookup, then a summary with Undo", async () => {
const d = await openDashboard({ hash: "#add", sync: EN });
await learn(d, "Spanish");
assert.equal(d.text(".bulk-controls"), "LearningSpanish", "one language read: no meanings picker, and no stray text");
const lines = Array.from({ length: 200 }, (_, i) => `palabra${i} = word${i}`).join("\n");
await paste(d, lines);
assert.equal(d.text(".bulk-counts").startsWith("200 words"), true);
Expand Down