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

アドレスバーを考慮した変更をより良い実装に変更したい #204

Open
yuki-snow1823 opened this issue Feb 21, 2023 · 0 comments

Comments

@yuki-snow1823
Copy link

Ref: #198

こちらのPRで実装していただいたおかげで、FullModalを使用している箇所で画面の縦幅いっぱいに表示されていました。

具体的にはカラーの絞り込みの画面で、モーダルウィンドウの下の商品リストが表示されなくなりました。
https://suzuri.jp/categories/t-shirts

image

一方で、この実装ではモーダルウィンドウのpaddingを引き伸ばしているため実機で動作を確認するとアドレスバーがない時に下が結構開いたり、逆にカラーの一覧が表示しきれずにボタンに重なってしまうなどの問題が挙げられます。

まだ実装の方法は検討できていないのですが、CSS、JSを駆使して下記の要件で修正したいです。

  • FullModalを使用する際に、アドレスバーの有無関係なく縦幅いっぱいにモーダルが表示される
  • カラーの絞り込み画面において、モーダル内の要素(カラーのチェック)がボタンに重ならない

#198 から引用

これは、height:-webkit-fill-available;があるせいで、アドレスが出るとちょうど100vhになるけどスクロールしてアドレスバーが消えると裾足らずでカートボタンが見えてしまうから 🥲
(カートの場合は必ず一番上にスクロールされている状態で開くので、アドレスバーが常に表示されている状態になり問題がないけど、リクエストはある場合とない場合がある...)

なのでsafariの時のみ使用している「height:-webkit-fill-available;」を外して、ボタン下にアドレスバー分のpaddingがつくように新しくclassを作って、リクエストモーダルの時はそのclassを使用するようにするといいかも、と思って現PRで試してみています。アドレスバーがない時に下が結構空いてしまうのが少し嫌だけど...

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

No branches or pull requests

1 participant