knopersikcuo/Planner
Watch
1
0
Fork
You've already forked Planner
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix notepad reader and tree sizing #31

Merged
kleb merged 1 commit from fix/notepad-layout-issues into main 2026-07-23 08:39:17 +02:00
Conversation 1 Commits 1 Files changed 5 +28 -8
kleb commented 2026-07-23 08:37:09 +02:00
Collaborator
Copy link

Closes #12.
Closes #29.

Summary

  • let the note reader use its intended responsive desktop width instead of the shared dialog cap
  • make nested note-tree indentation grow by one fixed step per level
  • add source and browser regression coverage for both layout invariants

Verification

  • npm run check (357 tests, lint/typecheck, production build)
  • node scripts/e2e/run.mjs --no-screenshots --only reader currently stops before Notepad at the pre-existing project-flow getByLabel('Title') timeout
Closes #12. Closes #29. ## Summary - let the note reader use its intended responsive desktop width instead of the shared dialog cap - make nested note-tree indentation grow by one fixed step per level - add source and browser regression coverage for both layout invariants ## Verification - `npm run check` (357 tests, lint/typecheck, production build) - `node scripts/e2e/run.mjs --no-screenshots --only reader` currently stops before Notepad at the pre-existing project-flow `getByLabel('Title')` timeout
kAgent commented 2026-07-23 08:37:11 +02:00
Collaborator
Copy link

kReview review

Verdict: no findings

No findings to address in the reviewed diff.

No meaningful correctness, security, or regression risks are provable from the supplied diff.

Reviewed by kReview at cc8f1a01e8. This comment is conservative and based only on the PR diff and metadata.

Est. cost ~$0.12 (25.0k in / 232 out / gpt-5.6-sol).

<!-- codex-forgejo-review --> <!-- codex-forgejo-review-head:cc8f1a01e8909a5248881d857b7fd7d10a69720b --> ## kReview review **Verdict:** no findings No findings to address in the reviewed diff. No meaningful correctness, security, or regression risks are provable from the supplied diff. _Reviewed by kReview at `cc8f1a01e8`. This comment is conservative and based only on the PR diff and metadata._ _Est. cost ~$0.12 (25.0k in / 232 out / gpt-5.6-sol)._ <!-- codex-forgejo-review-state:eyJoZWFkU2hhIjoiY2M4ZjFhMDFlODkwOWE1MjQ4ODgxZDg1N2I3ZmQ3ZDEwYTY5NzIwYiIsInN1bW1hcnkiOiJObyBtZWFuaW5nZnVsIGNvcnJlY3RuZXNzLCBzZWN1cml0eSwgb3IgcmVncmVzc2lvbiByaXNrcyBhcmUgcHJvdmFibGUgZnJvbSB0aGUgc3VwcGxpZWQgZGlmZi4iLCJmaW5kaW5ncyI6W10sImN1bXVsYXRpdmVDb3N0Ijp7InVzZCI6MC4xMTUzNTU5OTk5OTk5OTk5OSwiaW5wdXRUb2tlbnMiOjI1MDIwLCJvdXRwdXRUb2tlbnMiOjIzMiwibW9kZWwiOiJncHQtNS42LXNvbCJ9fQ== -->
kleb merged commit 34a2053cd3 into main 2026-07-23 08:39:17 +02:00
kleb deleted branch fix/notepad-layout-issues 2026-07-23 08:39:18 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
knopersikcuo/Planner!31
No description provided.