diff options
| author | Claude <noreply@anthropic.com> | 2026-02-28 22:53:24 +0000 |
|---|---|---|
| committer | Claude <noreply@anthropic.com> | 2026-02-28 22:53:24 +0000 |
| commit | df5e7d93963f0256285b13ddf750761930797e78 (patch) | |
| tree | 020cbd30711b1ee7eea3d077389ec381eded4a4e /frontend-vanilla/src/style.css | |
| parent | 15aa629972c4bd9fd3ab5ddb442783483f57a61f (diff) | |
| download | neko-df5e7d93963f0256285b13ddf750761930797e78.tar.gz neko-df5e7d93963f0256285b13ddf750761930797e78.tar.bz2 neko-df5e7d93963f0256285b13ddf750761930797e78.zip | |
Fix star/read buttons: _id string vs number type mismatchclaude/agents-file-mobile-stars-2nj6y
The Go API returns _id as a JSON string (due to the `json:",string"` tag
on Item.Id), but the frontend compared it with === against numbers from
parseInt(). String "5" === number 5 is always false in JS, so toggleStar,
mark-as-read, and keyboard shortcuts silently did nothing.
Fix:
- Coerce _id to Number() when items are loaded from the API
- Use Number() coercion in all store.items.find() comparisons as defense
- Revert the CSS touch-target changes (the issue was never about size)
- Add a regression test with string _id to prevent reintroduction
https://claude.ai/code/session_012CNdRhGhU3hxjrwvZt2BHZ
Diffstat (limited to 'frontend-vanilla/src/style.css')
| -rw-r--r-- | frontend-vanilla/src/style.css | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/frontend-vanilla/src/style.css b/frontend-vanilla/src/style.css index 757d213..47b2de2 100644 --- a/frontend-vanilla/src/style.css +++ b/frontend-vanilla/src/style.css @@ -490,15 +490,6 @@ select:focus { vertical-align: middle; transition: color 0.2s; line-height: 1; - flex-shrink: 0; - min-width: 44px; - min-height: 44px; - display: flex; - align-items: center; - justify-content: center; - height: auto; - text-transform: none; - font-weight: normal; } .star-btn.is-starred { |
