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 tweaks #117

Merged
merged 7 commits into from
Nov 18, 2024
Merged

Map tweaks #117

merged 7 commits into from
Nov 18, 2024

Conversation

vmork
Copy link
Contributor

@vmork vmork commented Nov 17, 2024

Some minor fixes/tweaks (i didnt add 4k lines :) its just from the lockfile getting messed up)

  • Changed the function that rescales the logo markers to fit the zoom level - they used to be rescaled continuously but now only grow/shrink on certain zoom levels. For whatever reason this makes zooming a lot smoother , especially on my phone (chrome on android), but maybe someone else could check with a different device to make sure its fine.
  • Decresed the initial zoom level used when switching to a new location in order to get a better overview
  • Removed a layer that only sets the background color on the map, since we get that from the maptiler styles
  • Fixed a bug on large screens (where the drawer component is not used) where the map would not grow to full width and the sidebar header was not displayed.

Copy link

vercel bot commented Nov 17, 2024

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

Name Status Preview Comments Updated (UTC)
armada-nu ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 17, 2024 9:15pm

Copy link
Contributor

@AugustPaulsrud AugustPaulsrud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of good changes, nice work! Being able to see all the companies when switching floors is great

@vmork vmork merged commit f72b18d into main Nov 18, 2024
6 checks passed
@vmork vmork deleted the fix/map branch November 18, 2024 09:11
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