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

Issue4299 #4434

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Issue4299 #4434

wants to merge 8 commits into from

Conversation

UnABC
Copy link

@UnABC UnABC commented Jan 4, 2025

リアクションをホバーしたときに拡大して表示して欲しい #4433

Copy link

github-actions bot commented Jan 4, 2025

Copy link
Contributor

@Pugma Pugma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

僕も a11y に詳しいわけではないので、どのように対応するのがいいかについてはっきりということはできないんですが、とりあえずここで示した部分について改めて調べて取り組んでみてほしいです…!

@@ -4,6 +4,8 @@
:title="tooltip"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

今の状態だと、↑の画像みたいにマウスホバー時のポップアップが 2 つ同時に出てしまうので title 属性を消していいと思います
ただ、これをしてしまうと a11y (アクセシビリティ) の観点から困ったことになる可能性もあるので、 aria-label などの属性に切り替えるのを検討したほうがいいと思います

Suggested change
:title="tooltip"
<!-- :title="tooltip" ここを消す -->

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

title属性を消去してaria-label属性に切り替えました
Discordを参考にしてaria-labelの内容を定めたのでa11yの観点からも問題はないと思われます

Copy link
Contributor

@Pugma Pugma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

実際に動作確認をしてみたところ、どうやらポップアップの大きさを超える文字列になると表示が壊れてそうです
ここの対応もお願いします…!

@UnABC
Copy link
Author

UnABC commented Jan 6, 2025

対応完了しました
確認のほどよろしくお願いします

@Pugma
Copy link
Contributor

Pugma commented Jan 7, 2025

確認したんですが、今の状態でも 2 列に収まらない場合はそれ以降のスタンプを押した人の名前が表示されなくなってそうです (本番環境で確認しました)
これまでの title 属性による tooltip は全部表示していたみたいだけど、今回みたいに表示が大きくなっているなら「ほか n 人」みたいな感じで圧縮した表示にしちゃってもいいと思います 👍
三角マークを押せば全数見ること自体はできるので

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

リアクションをホバーしたときに拡大して表示して欲しい
2 participants