-
-
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
feat: 新カスタム絵文字管理画面(β)の追加 #13473
base: develop
Are you sure you want to change the base?
feat: 新カスタム絵文字管理画面(β)の追加 #13473
Conversation
# Conflicts: # packages/misskey-js/src/api.types.ts # packages/misskey-js/src/autogen/apiClientJSDoc.ts # packages/misskey-js/src/autogen/endpoint.ts # packages/misskey-js/src/autogen/entities.ts # packages/misskey-js/src/autogen/models.ts # packages/misskey-js/src/autogen/types.ts
# Conflicts: # packages/frontend/package.json # packages/misskey-js/src/autogen/apiClientJSDoc.ts # packages/misskey-js/src/autogen/endpoint.ts # packages/misskey-js/src/autogen/entities.ts # packages/misskey-js/src/autogen/models.ts # packages/misskey-js/src/autogen/types.ts # pnpm-lock.yaml
# Conflicts: # packages/misskey-js/src/autogen/apiClientJSDoc.ts # packages/misskey-js/src/autogen/endpoint.ts # packages/misskey-js/src/autogen/entities.ts # packages/misskey-js/src/autogen/models.ts # packages/misskey-js/src/autogen/types.ts
取り込む |
# Conflicts: # CHANGELOG.md
ready for review |
}, 100); | ||
}, { passive: true, once: true }); | ||
}, { passive: true }); | ||
nextTick(() => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nextTickが必要な理由を書いておくと良さそう
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
content.value.children[0]がレンダリング完了前にアクセスするとエラーになるからだと思う
v-ifの都合でまだVDOMに存在しないとかかな
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
6baf88e
(rerunすれば通る系のfailだけど、ちょいちょい起こるので気になる) |
もう少し全体的なUXをブラッシュアップする必要があるわね |
引き継ぎます |
とりあえずマージした後調整するかしら |
9.0に入れるの?(新機能モリモリになるから次でもいい気がしてたけど) |
次でいいka |
Thank you for sending us a great Pull Request! 👍 example: pnpm run build-misskey-js-with-types |
# Conflicts: # packages/frontend/src/components/MkFolder.vue
What
カスタム絵文字の登録画面をリニューアルし、複数の絵文字に対して同時に登録・更新作業を行えるようにしました。
Why
resolve #10996
Additional info (optional)
既存のカスタム絵文字管理画面はそのままのこし、新画面はβ版の位置づけで追加しています(安定したら完全に置き換える考え)。
Checklist