Repository navigation
feat(web): replace account up/down buttons with drag-and-drop sorting - #61
Merged
Merged
Conversation
8000
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… sort Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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 c
3080
an 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.
问题
总览和手机首页的账户排序只能逐个点上移、下移,账户一多就要点很多次,交互很差。
结果
@dnd-kit/core、@dnd-kit/sortable、@dnd-kit/utilities,开启减少动态效果时关闭过渡动画。docs/reporting.md的排序说明已同步。验证
bun run typecheck通过。bun test tests588 个全部通过,新增reorderAccount单元测试,覆盖首尾、相邻、跨归档账户和无效 ID。git diff --check通过。bun run dev:demo的真实浏览器中验证了以下场景,截图留在本地被忽略的test-results中,未附在本说明里:🤖 Generated with Claude Code