misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
9.95k stars 1.35k forks source link

fix(frontend): focustrapがzindexを考慮するように #14431

Closed kakkokari-gtyih closed 1 month ago

kakkokari-gtyih commented 1 month ago

What

単に呼ばれた順にfocustrapをはめるのではなく、もしそれより上に表示されている要素があれば一番手前のものにfocustrapをかけて、その一番手前のものに対するfocustrapが解除されたらその次に手前にいたものにfocustrapを切り替える…というふうにした

Why

Additional info (optional)

Checklist

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 3.33333% with 58 lines in your changes missing coverage. Please review.

Project coverage is 20.20%. Comparing base (3cd5f86) to head (ae92c41). Report is 3 commits behind head on develop.

Files Patch % Lines
packages/frontend/src/scripts/focus-trap.ts 3.33% 58 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #14431 +/- ## ============================================ - Coverage 39.87% 20.20% -19.67% ============================================ Files 1547 725 -822 Lines 190882 100518 -90364 Branches 3506 1050 -2456 ============================================ - Hits 76116 20313 -55803 + Misses 114204 79663 -34541 + Partials 562 542 -20 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Sayamame-beans commented 1 month ago

これって、接続切断時に画面右下に表示されるダイアログが(一番上にあるように見えるのに)絵文字ピッカーなどの操作判定に負ける(そっちが押される)問題も解決しそうですか? issue書きそびれてたやつなのですが…

kakkokari-gtyih commented 1 month ago

これって、接続切断時に画面右下に表示されるダイアログが(一番上にあるように見えるのに)絵文字ピッカーなどの操作判定に負ける(そっちが押される)問題も解決しそうですか? issue書きそびれてたやつなのですが…

その可能性はある(そっちは動作未検証)

syuilo commented 1 month ago

🙏🏻

Sayamame-beans commented 1 month ago

これって、接続切断時に画面右下に表示されるダイアログが(一番上にあるように見えるのに)絵文字ピッカーなどの操作判定に負ける(そっちが押される)問題も解決しそうですか? issue書きそびれてたやつなのですが…

その可能性はある(そっちは動作未検証)

直ってなさそうでした(謎) そういえばスマホで起きますがPCでは検証できてない気がします。ドロワー表示で起きる可能性…? 追記: 投稿フォームを範囲外タップで閉じる操作が右下のダイアログより優先されていたので、ドロワーだからというわけでもなさそうです