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

開いたメディアをEscapeキーで閉じれない #11364

Closed
nekobato opened this issue Jul 23, 2023 · 4 comments
Closed

開いたメディアをEscapeキーで閉じれない #11364

nekobato opened this issue Jul 23, 2023 · 4 comments
Labels
⚠️bug? This might be a bug

Comments

@nekobato
Copy link

nekobato commented Jul 23, 2023

💡 Summary

メディア表示に使用しているPhotoSwipeLightboxはデフォルトでEscapeキーによる閉じる動作を持っているが、
メディアを開いた後、Escapeキー入力はMisskey Noteのキーボードショートカット設定に吸われてしまう

グローバルショートカットキー改修に踏み込まずともPhotoSwipeLightboxにfocusTrapを実装すれば正しく動作する模様、そしてfocusTrapの方がa11y上正しくはある

問題なければPR作ります

🥰 Expected Behavior

メディアを開いた後、ユーザーはEscapeキーを押すことでPhotoSwipeLightboxは閉じることができる

🤬 Actual Behavior

  • Chrome (114.0.5735.106): 反応なし
  • Safari (16.5): 2回連続でEscを押すと閉じる
  • Firefox (103.0.1): 反応なし
  • Edge ( 115.0): 反応なし

📝 Steps to Reproduce

  1. タイムラインからメディアを開く
  2. そのままマウス操作せずにEscapeキーを押す

📌 Environment

💻 Frontend

  • Model and OS of the device(s): Macbook Air (M2), macOS Ventura 13.4
  • Browser: Actual Behaviorに記述
  • Misskey: 13.14.1 develop
@kanade
Copy link

kanade commented Apr 22, 2024

このissueに進展はないでしょうか

@samunohito
Copy link
Member

@syuilo @kakkokari-gtyih
👀

@zyoshoka
Copy link
Contributor

#14157 で直った気がします

@kanade
Copy link

kanade commented Jul 10, 2024

developブランチを実行したところ、Escキーで画像を閉じられることが確認できました!
ありがとうございます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

No branches or pull requests

5 participants