Open kakkokari-gtyih opened 1 month ago
Attention: Patch coverage is 10.96606%
with 341 lines
in your changes missing coverage. Please review.
Project coverage is 19.37%. Comparing base (
ed89b4b
) to head (dfdf714
). Report is 2 commits behind head on develop.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
親子関係があるドラッグ(ページエディタとかコンディショナルロールエディタとか)では若干動きがぎこちなくなったけど概ねちゃんとうごく
MkWidgetの挙動が奇妙なので一旦draft
有識者募集中
MkWidgetの挙動が奇妙なので一旦draft
これはなおった
RoleCondEditorのand/orロジックの入れ子になっている場合その親子間でドラッグできるっていう機能が依然として不安定なので一旦無効化した(and/orロジック内でドラッグするのに関してはまともに動くので)
オッ
Pageのブロックの入れ子のドラッグとかは問題ないのかしら
Pageのブロックの入れ子のドラッグとかは問題ないのかしら
あーまだ試してないけどそれもだめな可能性がある
コード見た感じvue上で動くのではなく直接DOMをいじって実現するタイプっぽいからvueのvaporモードとかその他の機能と相性が悪い可能性があるのが懸念点だわね
コード見た感じvue上で動くのではなく直接DOMをいじって実現するタイプっぽいからvueのvaporモードとかその他の機能と相性が悪い可能性があるのが懸念点だわね
これ、ドラッグ系のイベントのハンドリングだけして要素の入れ替えはrefを更新することによりvueがやるというしくみだったはず(なので他の機能を通して要素に変更が入るとうまくいかず工夫が必要なケースがあったりした)
オッ
目が節穴だったかも
https://drag-and-drop.formkit.com/
Dragging items within your list will automatically change your list’s state to reflect the new order (which in turn allows the framework to re-render to the correct order)
What
vuedraggable@next
を@formkit/drag-and-drop
に置き換えたWhy
https://github.com/misskey-dev/misskey/issues/7681 の実現に必要
Additional info (optional)
次のリリースで
onSort
イベントハンドラが実装され、値の変更時にwatchを呼ぶ必要がなくなる( https://github.com/formkit/drag-and-drop/commit/a09cfcfdec827e5cbfbbbc6e27ad5240b6e4aee1 )→v0.2.0で来たChecklist