-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(frontend): vuedraggableを@formkit/drag-and-drop
に変更
#14486
base: develop
Are you sure you want to change the base?
refactor(frontend): vuedraggableを@formkit/drag-and-drop
に変更
#14486
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14486 +/- ##
============================================
- Coverage 39.57% 19.37% -20.20%
============================================
Files 1553 721 -832
Lines 195043 102444 -92599
Branches 3561 992 -2569
============================================
- Hits 77183 19851 -57332
+ Misses 117257 82042 -35215
+ Partials 603 551 -52 ☔ View full report in Codecov by Sentry. |
親子関係があるドラッグ(ページエディタとかコンディショナルロールエディタとか)では若干動きがぎこちなくなったけど概ねちゃんとうごく |
MkWidgetの挙動が奇妙なので一旦draft |
有識者募集中 |
これはなおった RoleCondEditorのand/orロジックの入れ子になっている場合その親子間でドラッグできるっていう機能が依然として不安定なので一旦無効化した(and/orロジック内でドラッグするのに関してはまともに動くので) |
オッ |
Pageのブロックの入れ子のドラッグとかは問題ないのかしら |
あーまだ試してないけどそれもだめな可能性がある |
コード見た感じvue上で動くのではなく直接DOMをいじって実現するタイプっぽいからvueのvaporモードとかその他の機能と相性が悪い可能性があるのが懸念点だわね |
これ、ドラッグ系のイベントのハンドリングだけして要素の入れ替えはrefを更新することによりvueがやるというしくみだったはず(なので他の機能を通して要素に変更が入るとうまくいかず工夫が必要なケースがあったりした) |
オッ |
目が節穴だったかも |
https://drag-and-drop.formkit.com/
|
What
vuedraggable@next
を@formkit/drag-and-drop
に置き換えたWhy
#7681 の実現に必要
Additional info (optional)
次のリリースで
onSort
イベントハンドラが実装され、値の変更時にwatchを呼ぶ必要がなくなる( formkit/drag-and-drop@a09cfcf )→v0.2.0で来たChecklist