Skip to content

Claude/zealous goldberg f31c40#1

Merged
ijbo merged 2 commits into
mainfrom
claude/zealous-goldberg-f31c40
May 14, 2026
Merged

Claude/zealous goldberg f31c40#1
ijbo merged 2 commits into
mainfrom
claude/zealous-goldberg-f31c40

Conversation

@ijbo
Copy link
Copy Markdown
Contributor

@ijbo ijbo commented May 14, 2026

No description provided.

ijbo added 2 commits May 14, 2026 19:01
- New Tools template category (bi-tools icon) with Calculator as first template
- iOS-style two-line display: dim expression above, large result below
- Both lines contenteditable — click expression to edit and re-evaluate live
- Full BODMAS support: ( ) ^ ÷ × − +, with x^y mapped to JS ** for correct precedence
- Strict allowlist safeEval ([-+*/().\d^]+) — no eval, no identifiers, alert(1) → Error
- History side panel: editable expr/result cells, ↩ to send back to display, Clear button
- Paste support for full expressions (12*7+3, (2+3)^2*4, 1,234.5)
- Operator-aware backspace, keyboard support, unicode operator normalization
- Renderer round-trip fix: entity strings built via String.fromCharCode to survive textContent extraction
- 10 new Playwright cases (calculator-tool.spec.js) + Tools-pill assertion in template-loading.spec.js
CI hook (changelog-location) fails when a CHANGELOG-*.md file is at the
repo root. Four were identical duplicates of files already in changelogs/,
so the root copies were removed. One (pretext-performance) only existed
at root, so it was moved.

- rm CHANGELOG-annotate-pretext-reflow.md (duplicate of changelogs/ copy)
- rm CHANGELOG-chart-bugfixes.md (duplicate)
- rm CHANGELOG-podcast-system.md (duplicate, flagged by CI on the calculator PR)
- rm CHANGELOG-share-link-loader.md (duplicate)
- mv CHANGELOG-pretext-performance.md → changelogs/
@ijbo ijbo merged commit 0dbad99 into main May 14, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant