Tyler/StepForge
1
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Issue #9: Reorder text, code, and table blocks in the editor #16

Closed
Tyler wants to merge 0 commits from tyler/fix/issue-9-block-reorder AGit into main
pull from: tyler/fix/issue-9-block-reorder
merge into: :main
Conversation 0 Commits - Files Changed -
Tyler commented 2026-06-12 17:24:20 +00:00
Owner
Copy Link
Copy Source

Improvement Area

  • Documentation
  • Tests
  • Git / workflow
  • Other

Issue

Summary

Preserve the live step state while reordering text, code, and table blocks so the move up/down buttons keep the block contents intact and the block cards still move immediately in the editor.

Changes

  • Kept saved step updates merged into the existing in-memory step object instead of replacing block array references.
  • Reflowed the block cards in place after reorder so the editor updates without a full rerender.
  • Preserved undo and crop/save paths so they keep using the same live block objects.

Testing

  • bash tests/run_test.sh
  • Manual verification completed
  • Tests not run

Testing Notes

  • node --check app/renderer/editor.js
  • npm test
  • bash tests/run_test.sh

Screenshots or Recordings

N/A

Deployment / Rollout Notes

  • No special rollout steps
  • Documentation only
  • Other

Notes

None

Checklist

  • Scope is limited to the selected area above.
  • Related docs or comments were updated if needed.
  • I ran bash tests/run_test.sh.
  • The PR references a issue number.
  • Any follow-up work is tracked in TODO.md or an issue.
## Improvement Area - [ ] Documentation - [ ] Tests - [ ] Git / workflow - [x] Other ## Issue - Closes #9 ## Summary Preserve the live step state while reordering text, code, and table blocks so the move up/down buttons keep the block contents intact and the block cards still move immediately in the editor. ## Changes - Kept saved step updates merged into the existing in-memory step object instead of replacing block array references. - Reflowed the block cards in place after reorder so the editor updates without a full rerender. - Preserved undo and crop/save paths so they keep using the same live block objects. ## Testing - [x] `bash tests/run_test.sh` - [x] Manual verification completed - [ ] Tests not run ### Testing Notes - `node --check app/renderer/editor.js` - `npm test` - `bash tests/run_test.sh` ## Screenshots or Recordings N/A ## Deployment / Rollout Notes - [x] No special rollout steps - [ ] Documentation only - [ ] Other ### Notes None ## Checklist - [x] Scope is limited to the selected area above. - [x] Related docs or comments were updated if needed. - [x] I ran `bash tests/run_test.sh`. - [x] The PR references a issue number. - [ ] Any follow-up work is tracked in TODO.md or an issue.
Tyler added 1 commit 2026-06-12 17:24:20 +00:00
Fix guide editor issues 4-10
Template tests / tests (pull_request) Has been cancelled
Details
Template tests / tests (push) Has been cancelled
Details
f88ff0259e
Tyler closed this pull request 2026-06-13 21:04:58 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Tyler (Tyler Westbrook)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Tyler/StepForge#16