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

CWの中に画像を添付した場合、その画像が見られないことがある #12719

Closed
1STEP621 opened this issue Dec 19, 2023 · 1 comment · Fixed by #12721
Closed
Labels
⚠️bug? This might be a bug

Comments

@1STEP621
Copy link
Contributor

💡 Summary

「画像が1枚のみのメディアリストの高さ」を「デフォルト」以外に設定していると、CWの中に添付された画像が見られないことがあります。

🥰 Expected Behavior

画像が見られる

🤬 Actual Behavior

画像が見られない

📝 Steps to Reproduce

  1. 「画像が1枚のみのメディアリストの高さ」を「デフォルト」以外に設定する
  2. CWの中に画像が添付されているノートを閲覧する

💻 Frontend Environment

* Model and OS of the device(s): Ubuntu 22.04.3 LTS
* Browser: Firefox 120.0.1
* Server URL: ローカル環境, misskey.backspace.fm, misskey.systemsで再現
* Misskey: 2023.12.0-beta.5

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:
@1STEP621 1STEP621 added the ⚠️bug? This might be a bug label Dec 19, 2023
@1STEP621
Copy link
Contributor Author

1STEP621 commented Dec 19, 2023

const width = container.value ? await getClientWidthWithCache(root.value, container.value) : root.value.clientWidth;

ここでdisplay: none;のものに対してclientWidthを得ようとして0が返ってきている気がする

aspect-ratioを使った実装にするのが一番楽そうですが、 #11377 でもともとaspect-ratioを使っていたところをheightにしてパフォーマンスを改善しようとした様子があるので、それを戻しちゃうのは微妙かも?と思っています

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
1 participant