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

Map mobile #11

Merged
merged 3 commits into from
Sep 18, 2024
Merged

Map mobile #11

merged 3 commits into from
Sep 18, 2024

Conversation

sgruhier
Copy link
Contributor

On small device, the sidebar is moved to a bottom sheet.
Code required some changes due to state management inside the bottom sheet. It would have been easier and cleaner to use a state library like riverpod, but I didn't want to add dependencies.

@Rahiche
Copy link
Owner

Rahiche commented Sep 18, 2024

LGTM
Thanks a a lot
yeah adding a state for a sample app would be too much

@Rahiche Rahiche merged commit e8be9d3 into Rahiche:main Sep 18, 2024
1 check passed
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.

2 participants