fix(ui): sort project notes ascending to match branch timeline order#413
fix(ui): sort project notes ascending to match branch timeline order#413
Conversation
Project notes were sorted newest-first (descending) while the branch timeline sorts oldest-first (ascending). Align project notes to use the same chronological order. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bcd343ef53
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| const bIsGenerating = !b.title.trim() && !b.content.trim(); | ||
| if (aIsGenerating !== bIsGenerating) return aIsGenerating ? 1 : -1; | ||
| return b.createdAt - a.createdAt; | ||
| return a.createdAt - b.createdAt; |
There was a problem hiding this comment.
Restore newest-first ordering for project notes
Sorting timelineNotes oldest-first moves the note for the current project session to the bottom of the list. handleSubmitPrompt() explicitly reloads notes so the stub "appears as Generating note…" in ProjectSection.svelte, but this component never scrolls the notes list into view and just renders timelineNotes in order. On projects with enough existing notes to extend below the viewport, the running session and its finished note are no longer visible after submit, so users can miss that their request actually started or completed.
Useful? React with 👍 / 👎.
Summary
createdAtin ascending (oldest-first) order instead of descending, so the timeline matches the chronological branch order.Test plan
🤖 Generated with Claude Code