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

センシティブ設定した文章に画像が含まれると表示されないことがある #11768

Closed
Tsut-ps opened this issue Aug 28, 2023 · 2 comments
Labels
⚠️bug? This might be a bug

Comments

@Tsut-ps
Copy link

Tsut-ps commented Aug 28, 2023

💡 Summary

「画像が1枚のみのメディアリストの高さ」をデフォルト以外に設定した上で、文章中にセンシティブ(閲覧注意/CW)を付けたとき、そのノートに含まれるメディア(画像)が見れないことがある。

※画像をセンシティブに設定するのではなく、文章をセンシティブに設定する。

関連ノート → https://misskey.io/notes/9iyf7nmc7m
当該ノート例 → https://misskey.io/notes/9ixbl2qzvk

🥰 Expected Behavior

「もっと見る」を押したときに、問題なく画像が表示される

image

🤬 Actual Behavior

「もっと見る」を押したときに、画像が表示されない

image

📝 Steps to Reproduce

再現方法1(稀に起こる)

  1. UIをデフォルトモードにする
  2. クライアント設定にて「画像が1枚のみのメディアリストの高さ」をデフォルト以外(16:9など)に設定する
  3. 文章をセンシティブに設定し、画像を合わせて投稿する
  4. そのノートの「もっと見る」を押す

再現方法2(確定で起こる)

  1. UIをデッキモードにする
  2. クライアント設定にて「画像が1枚のみのメディアリストの高さ」をデフォルト以外(16:9など)に設定する
  3. 文章をセンシティブに設定し、画像を合わせて投稿する
  4. そのノートを「新しいタブで開く」(新規タブ)にて確認する

補足

  • 「画像が1枚のみのメディアリストの高さ」をデフォルトに戻すと表示される
  • CSSの .xutAY.xsfFg.xkfNk クラスが付けられた div 要素の element.style の height の値が0になっている
    • この height 値を変更すると見られる(画像はおそらく読み込めているようです)
    • もしかすると:適切な高さが取得されていない可能性……?

image

📌 Environment

💻 Frontend

  • Desktop
    • Model and OS of the device(s): Windows 11 Home 22H2
    • Browser: Chrome 116.0.5845.111 / Firefox 116.0.3
  • Mobile
    • Model and OS of the device(s): Xperia 1 III (SOG03), Android 13
    • Browser: Chrome 116.0.5845.114
  • Server URL: misskey.io
  • Misskey: 13.14.2-io.5

🛰 Backend (for server admin)

※新規でローカル環境を作り確認したところ、同様に再現したので念のため記載。

  • Installation Method or Hosting Service: Docker Compose
  • Misskey: 13.14.2
  • Node: 20.3.1
  • PostgreSQL: 15.4
  • Redis: 7.2.0
  • OS and Architecture: Ubuntu 20.04.6 LTS
@Tsut-ps Tsut-ps added the ⚠️bug? This might be a bug label Aug 28, 2023
@Tsut-ps
Copy link
Author

Tsut-ps commented Aug 31, 2023

一部訂正(済)

「再現方法2」→(4.)
誤:「ページで表示」(新規タブ)
正:「新しいタブで開く」(新規タブ)

一部補足(済)

  • CSSの .xutAY.xsfFg.xkfNk クラスが付けられた div 要素の element.style の height の値が0になっている

@Tsut-ps
Copy link
Author

Tsut-ps commented Dec 28, 2023

この問題は 2023.12.0 で解決しました

Misskey.io(2023.12.2-io)にて、問題が再現しなくなったことを確認しました 🎉
そのため、Issueをクローズします。

2023.12.0

Fix: 「画像が1枚のみのメディアリストの高さ」を「デフォルト」以外に設定していると、CWの中などに添付された画像が見られないバグを修正

関連Isuue → #12719  関連PR → #12721

@Tsut-ps Tsut-ps closed this as completed Dec 28, 2023
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

1 participant