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

Symbology fixes and finer customizability #34

Merged
merged 4 commits into from
Jun 11, 2024
Merged

Symbology fixes and finer customizability #34

merged 4 commits into from
Jun 11, 2024

Conversation

gberaudo
Copy link
Contributor

@gberaudo gberaudo commented Jun 11, 2024

This PR:

  • update some dependencies;
  • fix black border appearing when using circle style;
  • pass the featureObject parameter to allow finer customizability (breaking change).

@gberaudo gberaudo requested a review from sbrunner June 11, 2024 09:40
gberaudo added 2 commits June 11, 2024 14:50
The fontFamily was incorrectly set to the full canvas font.
Ex: 10px sans-serif
See https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/font
This avoid getting a black border.
Without the feature object customizers have very little information to apply style changes.
Now they will be able to look at the feature properties.

This is also true for the protected methods of the Vector Encoder.
@gberaudo gberaudo marked this pull request as ready for review June 11, 2024 14:33
@gberaudo gberaudo changed the title Symbology fixes Symbology fixes and finer customizability Jun 11, 2024
@gberaudo gberaudo merged commit 918de2b into main Jun 11, 2024
1 check passed
@gberaudo gberaudo deleted the symbology_fixes branch June 11, 2024 14:43
Copy link
Member

@ger-benjamin ger-benjamin left a comment

Choose a reason for hiding this comment

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

Looks good, thanks

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