commaai / new-connect

a rewrite of comma connect
https://new-connect.connect-d5y.pages.dev/
MIT License
12 stars 29 forks source link

Fixed overflow when scrolling up #18

Closed ugtthis closed 5 months ago

ugtthis commented 5 months ago

Sidebar showing if scrolled up too much

ui-overflow

This shows up for both mobile and desktop.

First one shows the overflow second shows it fixed

https://github.com/commaai/new-connect/assets/142481257/801b3ad6-ea1a-444c-8a8c-52c6bbde3c45

ugtthis commented 5 months ago

Closing an reopening to re-trigger Preview build

github-actions[bot] commented 5 months ago

Welcome to new-connect! Make sure to:

deployed preview: https://18.connect-d5y.pages.dev

incognitojam commented 5 months ago

I couldn't reproduce the overscroll on Windows Chrome or Android... I assume it's on macOS and iOS.

ugtthis commented 5 months ago

Oh I see, for this PR I tested with the preview links on my iPad, iPhone, and Macbook