Repository navigation
Conversation
…rawer Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Haaaiawd
pushed a commit
that referenced
this pull request
Oct 7, 2026
Owner
Author
|
已 squash 合并入 master(commit baa37fe)。 |
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.
Drag did not move — root cause
HTML5 native DnD in WebView2 + a mode gate. Three suspect paths, two disproven with evidence:
data-wheel-idIS rendered (prompts.js:47) → NaNdragIdhypothesis disprovenstate.wheelSortdefaults to'auto'(store.js:67) →dragstartrunse.preventDefault()+ toast → the drag is blocked before it ever starts. This is the blocker.draggableset inmousedownand cleared inmouseupleaves a ~0-width window and is unreliable on WebView2.Replaced with pointer-event based reordering: hold row → lift (shadow/opacity) → dashed accent insertion slot tracks the pointer → release commits via
set_pin_order.Hide internal fields (user request)
Removed from the drawer UI: 应用范围 (
#fApps) and 轮盘位置 (#fOrder) — internal mechanics should not be user-editable. JS read/write and dirty-watchers cleaned; i18n keys removed both languages.Backend untouched:
app_scopes_json/inject_ordercolumns stay. On save, existing values are passed through so a save cannot silently wipe the orderset_pin_orderwrites. New prompts sendnull(verifiedOption<String>binds NULL; readers useunwrap_or_default(), sonull ≡ "[]").Verified
mouse.down/move/up, asserts DOM reorder +set_pin_ordercalled + auto→manual flip + drawer fields gone + keyboard reorder. Screenshots captured mid-drag (lifted card + insertion slot) and post-drop.commitPinOrderflippedwheelSortbefore the IPC resolved → mode said manual while order never persisted. Flip moved after IPC success.