Developに追従するとノート作成画面でカスタム絵文字を入力できない(385969e適用以降)

[kanarikanaru]

💡 Summary

• fix(frontend): フォーカスの挙動を修正 (fix(frontend): フォーカスの挙動を修正 #14158) (385969e) を適用するとノート作成画面でカスタム絵文字ピッカーを起動した際にフォーカスが当たらない
• 該当のコミットをrevertすることで再現しないことを確認

🥰 Expected Behavior

• ノート作成画面でカスタム絵文字ピッカー起動時にピッカーにフォーカスがいく

🤬 Actual Behavior

• フォーカスが当たらない

📝 Steps to Reproduce

1. 追従してpnpm run
2. ノート作成画面を開いてカスタム絵文字ピッカーを起動
3. wow

💻 Frontend Environment

* Model and OS of the device(s): MacBook Pro (14inch, 2021), macOS 14.5 (23F79)
* Browser: Google Chrome Version 126.0.6478.127 (Official Build) (arm64)
* Server URL: dev container + misskey.backspace.fm(revert済み)
* Misskey: developまで追従(6cd1527)

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service: development environment + systemd
* Misskey: developまで追従(6cd1527)
* Node: 20.x
* PostgreSQL: 15.x
* Redis: 7.x
* OS and Architecture: Ubuntu 22

Do you want to address this bug yourself?

• Yes, I will patch the bug myself and send a pull request

[samunohito]

強火

[samunohito]

@kakkokari-gtyih
心当たりあります？

[samunohito]

ピッカーの親要素にinertが当たってる…？

[samunohito]

①絵文字ピッカー・リアクションピッカーは常にDOM上で待機してる（このへんによるもの）
②新しくダイアログが生成された時に、それ以外の要素に対してinertを当てる

①＋②＝新しくダイアログが生成された時にDOM上で控えてた絵文字ピッカーに対してinertを当てる

[kakkokari-gtyih]

新しくダイアログが生成された時に、それ以外の要素に対してinertを当てる

常時待機要素にinertが当たらないように、本当に表示されているかどうかでfocustrapを判断させていたはず

[samunohito]

けど、domを見る限り当たってしまってるので、その条件に漏れがありそう…？

[kakkokari-gtyih]

参考（focustrapの出し分け制御はここだけのはず）

• https://github.com/misskey-dev/misskey/blob/develop/packages/frontend/src/scripts/focus-trap.ts
• https://github.com/misskey-dev/misskey/blob/develop/packages/frontend/src/components/MkModal.vue

[samunohito]

本当に表示されているかどうか

これどこか教えてほしいです
ソース見てるんだけど、分からなくて…

[kakkokari-gtyih]

Modalの326行目辺り

[samunohito]

まず、ページ表示直後はこいつらがいて（常駐してるピッカーたち）

MkPostForm（黄枠）を表示した段階で、ピッカーたちにinertが設定される

その状態でMkPostFormの絵文字ボタンを押すと、inertが設定されたピッカーが表に出てくる

[kakkokari-gtyih]

あー解除しないといけないのか（できてなかったのか…）

[samunohito]

「自分自身」に対してはModalの制御で良いけど、他の要素に対してまでは及んでない認識

[samunohito]

inertする要素の除外条件を増やした方が丸そうですかね（idふるなり、data-*で独自属性をつけて判定するなり）

[kakkokari-gtyih]

inertが既にかかっていた場合は解除する処理が必要そう

[kakkokari-gtyih]

#14189 を適用すると解消する

[samunohito]

@kanarikanaru
#14189 でもだめそうであれば、お手数ですが再度open（または新規issueの発行）いただけますと幸いです。