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.

Sign up for GitHub

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

Jump to bottom

デッキのカラム幅が変化したとき、全体を再描画するかメディアビューの高さを計算し直す #3633

Closed
tamaina opened this issue Dec 16, 2018 · 10 comments
Closed

デッキのカラム幅が変化したとき、全体を再描画するかメディアビューの高さを計算し直す #3633

tamaina opened this issue Dec 16, 2018 · 10 comments
Labels
packages/frontend Client side specific issue/PR

Comments

@tamaina
Copy link
Contributor

tamaina commented Dec 16, 2018 •
edited by syuilo
Loading

Summary

メディアビューの高さはjavascriptで計算してpx数を決めているため、デッキのカラム幅が狭まったときに以下の画像のように突き出してしまう

image

Environment

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
@syuilo
Copy link
Member

syuilo commented Dec 16, 2018

ちなみにSafariのバグのためにそうしています

@syuilo syuilo added the packages/frontend Client side specific issue/PR label Dec 16, 2018
@acid-chicken
Copy link
Member

acid-chicken commented Dec 16, 2018 •
edited
Loading

Embed Playerに用いられているアスペクト比固定設定を流用するのはどうでしょう。

@tamaina
Copy link
Contributor Author

tamaina commented Dec 16, 2018

@acid-chicken それはSafariではバグらないの?

@acid-chicken
Copy link
Member

ツイキャス（縦横比情報と実際のサイズが違うのが原因：#2610）以外では現状そういったものは確認されていない。

@tamaina
Copy link
Contributor Author

tamaina commented Dec 16, 2018

中身がGridレイアウトだとバグるみたいなのはあるかもしれないし、一度試してみないとわからなさそう

@tamaina tamaina mentioned this issue Feb 18, 2020
Merged
@tamaina
Copy link
Contributor Author

tamaina commented Feb 9, 2023

これってどうなんだろ

@acid-chicken
Copy link
Member

acid-chicken commented Feb 9, 2023 •
edited
Loading

CSS Containment 使ったらもはや起こり得ないのではと思ってるけどどう 別の話と勘違いしてた

@acid-chicken
Copy link
Member

acid-chicken commented Feb 9, 2023 •
edited
Loading

閑話休題、現在は aspect-ratio が使えるから CSS でシンプルに完結できそう

@tamaina
Copy link
Contributor Author

tamaina commented May 19, 2023

解決済み

@tamaina
Copy link
Contributor Author

tamaina commented Feb 4, 2024

新しい計算については #11004 あたりで

@tamaina tamaina closed this as completed Feb 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@syuilo @tamaina @acid-chicken