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

Improve policy list scrolling #346

Merged
merged 4 commits into from
Jun 11, 2024
Merged

Improve policy list scrolling #346

merged 4 commits into from
Jun 11, 2024

Conversation

kariy
Copy link
Member

@kariy kariy commented Jun 7, 2024

Before 🤢👎:

Cartridge.Controller.-.7.June.2024.mp4

After 😎🤯:

Cartridge.Controller.-.8.June.2024.mp4

EDIT: just realized the weird bounce when opening the drawer. fixing it. Fixed

@kariy kariy requested review from broody and JunichiSugiura June 7, 2024 23:04
Copy link

vercel bot commented Jun 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
cartridge-starknet-react-next ✅ Ready (Inspect) Visit Preview Jun 11, 2024 2:00pm
docs ✅ Ready (Inspect) Visit Preview Jun 11, 2024 2:00pm
keychain ✅ Ready (Inspect) Visit Preview Jun 11, 2024 2:00pm
ui ✅ Ready (Inspect) Visit Preview Jun 11, 2024 2:00pm
ui-next ✅ Ready (Inspect) Visit Preview Jun 11, 2024 2:00pm

@tarrencev
Copy link
Contributor

tarrencev commented Jun 8, 2024

Looks slick! Didn't know you were hiding these skills

It would be really sick if when you scrolled up the pane header went all the way to the top of the frame before sticking

Also could be a cool behavior if you could scroll it open from the collapsed position, so you don't need to click the button. On mobile you could just pull up the bottom pane

Copy link

codecov bot commented Jun 8, 2024

The author of this PR, kariy, is not an activated member of this organization on Codecov.
Please activate this user on Codecov to display this PR comment.
Coverage data is still being uploaded to Codecov.io for purposes of overall coverage calculations.
Please don't hesitate to email us at [email protected] with any questions.

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

Successfully merging this pull request may close these issues.

3 participants