Repository navigation
Course page: number the open tasks, and the number is the handle - #180
Merged
Merged
Conversation
The grip hung 18px off a row already pulled into the gutter, so on a phone
it sat off the glass and a course's tasks could not be reordered there at
all. On a laptop it was a faint pair of strokes nobody found.
ReorderList rows can now be numbered: the place in mono in a 22px lane of
its own, swapping to the strokes under a cursor, renumbering live while a
row is carried. The course page uses it, says which order the numbers are
("most pressing first" / "in your order") beside the header, and explains
dragging once with a FirstNote until anything has been placed.
Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01W2qN84qBgHi1Y749Kd9Nq5
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reorder on the course page was hard to find, and on a phone you couldn't do it at all: the grip hung 18px off a row that's already pulled into the gutter, so it ended up off screen.
What changed
ReorderListgets anumberedoption for row lists. Each row shows its place in mono in a 22px lane, swaps to the pencil strokes when you hover the number (only on devices that can hover), and every number updates live while you drag so you see the new order before letting go.mdup the numbers sit in the page margin and the rows stay on the column. Belowmdthey take the lane, and the done rows step in to match.FirstNoteexplains the drag once, and disappears for good after the first placement (read off task positions, not a dismissed flag).readmedesign.mdupdated: Rearranging by hand + The course page.Separate from this diff: the live Supabase project was missing two repo migrations,
add_task_sort_order(the cause of the reorder_tasks error) andday_ending_hour_to_nine. Both are applied now.Checked:
npm run typecheckandnpm run lintpass. I ran it locally on local data at 1400px and 390px, dragged task 5 to the top, and the order held after a reload at both sizes.🤖 Generated with Claude Code
https://claude.ai/code/session_01W2qN84qBgHi1Y749Kd9Nq5
Generated by Claude Code