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

カルーセルの自動スライドでHA-PACSがスキップされる #37

Closed
shuuji3 opened this issue Mar 28, 2024 · 6 comments · Fixed by #42
Closed

カルーセルの自動スライドでHA-PACSがスキップされる #37

shuuji3 opened this issue Mar 28, 2024 · 6 comments · Fixed by #42
Labels
bug Something isn't working

Comments

@shuuji3
Copy link
Member

shuuji3 commented Mar 28, 2024

HA-PACSの1つ手前のHA-PACS/TCAまで来ると、1枚目のPegasusに戻ってしまいます。

@shuuji3 shuuji3 added the bug Something isn't working label Mar 28, 2024
@namachan10777
Copy link
Contributor

これ私の環境だと再現しませんでした。ブラウザ等の情報をいただきたいです

@shuuji3
Copy link
Member Author

shuuji3 commented Mar 29, 2024

改めてテストしてみたところ、Linux (Pop_OS!) のGoogle Chromeでだけ発生する問題だとわかりました (Firefoxや、macOSのGoogle Chrome、Firefoxでは問題なし)。

おそらくこの環境でだけ右側に幅のあるスライダーが常時表示されるので、幅の計算がずれているのだと思います。

Screenshot from 2024-03-29 12-16-30

@shuuji3
Copy link
Member Author

shuuji3 commented Mar 29, 2024

幅の計算で window.innerWidth が使われているのが原因かもしれません。

https://github.com/tsukuba-hpcs/hpcs-web/blob/479ba70672b7af9ec0ee14c7ed0aaea706207dec/src/components/display/Carousel.astro#L135

The read-only Window property innerWidth returns the interior width of the window in pixels (that is, the width of the window's layout viewport). That includes the width of the vertical scroll bar, if one is present.

https://developer.mozilla.org/en-US/docs/Web/API/Window/innerWidth

@namachan10777
Copy link
Contributor

なるほど、ありがとうございます 🙇 下記Issueと合わせて修正します

@shuuji3
Copy link
Member Author

shuuji3 commented Mar 29, 2024

@namachan10777 すみません、今コメントを見ました。

scroll()ではなく、画像の幅が100vwの場合にスクロールバー分(15px)だけ幅が広くなってしまうことが原因だったので、#42のように100%にすると問題が起こらなくなりました。もし別の方法があればcloseしても構いません。

@shuuji3
Copy link
Member Author

shuuji3 commented Mar 29, 2024

ref. New CSS Viewport Units Do Not Solve The Classic Scrollbar Problem — Smashing Magazine - https://www.smashingmagazine.com/2023/12/new-css-viewport-units-not-solve-classic-scrollbar-problem/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants