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

Fixes #31 and #37 where icons were not properly aligned in images #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lesserj
Copy link

@lesserj lesserj commented Mar 19, 2020

The 2x version of the button sprite had different proportions from the 1x version of the sprite, so the background-size property was incorrect and icons were not correctly positioned on some screens. Updating to the latest Leaflet.Draw didn't fix the issue and in fact made it worse, so I just modified the sprites to match so the only difference is the 2x scale.

…n images

The 2x version of the button sprite had different proportions from the 1x version of the sprite, so the background-size property was incorrect and icons were not correctly positioned on some screens.  Updating to the latest Leaflet.Draw didn't fix the issue and in fact made it worse, so I just modified the sprites to match so the only difference is the 2x scale.
@turbotimon
Copy link

would love to see this merged soon, so its fixed!

@MightyPork
Copy link

The CSS fix alone fixes it perfectly for me (tried it with dev tools)

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