Skip to content
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

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

Merged

Conversation

kakkokari-gtyih
Copy link
Contributor

What

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

Why

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Aug 18, 2024
Copy link

codecov bot commented Aug 18, 2024

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 ⚠️
Additional details and impacted files
@@             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     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Sayamame-beans
Copy link
Member

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

@kakkokari-gtyih
Copy link
Contributor Author

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

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

@syuilo syuilo merged commit b53ee54 into misskey-dev:develop Aug 18, 2024
17 checks passed
@syuilo
Copy link
Member

syuilo commented Aug 18, 2024

🙏🏻

@kakkokari-gtyih kakkokari-gtyih deleted the fix-focustrap-to-consider-zindex branch August 18, 2024 05:28
LemonDouble pushed a commit to LemonDouble/misskey that referenced this pull request Aug 19, 2024
@Sayamame-beans
Copy link
Member

Sayamame-beans commented Aug 19, 2024

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants