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.

Sign up for GitHub

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

Jump to bottom

Frontend improvements for the map #210

Merged
merged 5 commits into from
Sep 24, 2024
Merged

Frontend improvements for the map #210

merged 5 commits into from
Sep 24, 2024

Conversation

sSwiergosz
Copy link
Collaborator

This PR adds a couple of improvements to the map component:

  • memorize location filtering
  • move detecting screen size to the useEffect hook with an addition of debounce method
  • add the second boolean variable to indicate that the map tile is loading
  • remove the redundant console.logs
  • get rid of MaterialUI Autocomplete warnings

@sSwiergosz sSwiergosz added the enhancement New feature or request label Sep 24, 2024
@sSwiergosz sSwiergosz requested a review from kubawajs September 24, 2024 20:12
@vercel Vercel
Copy link

vercel bot commented Sep 24, 2024

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

Name Status Preview Comments Updated (UTC)
nursery-nav ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 24, 2024 8:12pm
nursery-nav-api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 24, 2024 8:12pm

@kubawajs kubawajs merged commit b7c6ffe into main Sep 24, 2024
4 checks passed
@kubawajs kubawajs deleted the map-loading branch September 24, 2024 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kubawajs kubawajs kubawajs approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sSwiergosz @kubawajs