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

configurable POI size #15444

Closed
behrisch opened this issue Sep 6, 2024 · 1 comment
Closed

configurable POI size #15444

behrisch opened this issue Sep 6, 2024 · 1 comment
Assignees
Milestone

Comments

@behrisch
Copy link
Contributor

behrisch commented Sep 6, 2024

Currently the width and height parameters only apply if the POI has an image. I would like to use the width parameter for the diameter of the POI but unfortunately the default for width is 1 while the hard coded diameter is 2.6. @namdre Would reducing the default POI size to 1m be okay?

@behrisch behrisch added this to the 1.21.0 milestone Sep 6, 2024
@behrisch behrisch self-assigned this Sep 6, 2024
@namdre
Copy link
Contributor

namdre commented Sep 6, 2024

I think 1m diameter POIs don't have sufficient visibility.
We could instead change the default for imgWidth since it's quite unlikely that people were using imgFile with the default width/height values.

palvarezlopez pushed a commit that referenced this issue Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants