Skip to content

When web-map-links are rendered on map half opacity fill of bbox feature partially covers data #345

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

Closed
Tracked by #535
santilland opened this issue Jun 30, 2023 · 4 comments · May be fixed by #535
Closed
Tracked by #535
Assignees
Labels
Milestone

Comments

@santilland
Copy link

santilland commented Jun 30, 2023

Not a critical issue but would be great if the fill could somehow be disabled.
One approach could be to check if a web-map-links is being rendered and if yes set to opacity to 0 in the bbox feature fill?

@m-mohr m-mohr self-assigned this Jun 30, 2023
@m-mohr m-mohr added this to the 3.1.0 milestone Jun 30, 2023
@m-mohr
Copy link
Collaborator

m-mohr commented Aug 16, 2023

Will be fixed in the OpenLayers variant of stac-layer, so it is probably not worth implmenting it for Leaflet as we'll migrate anyway.

@m-mohr m-mohr modified the milestones: 3.1.0, 3.2.0 Aug 16, 2023
@santilland
Copy link
Author

Sounds good, should me close the issue? Or does it make sense to keep it around as feature for OL migration?

@m-mohr
Copy link
Collaborator

m-mohr commented Aug 17, 2023

Yeah, let's keep it open to ensure once OL migration is done that it really works.

@m-mohr m-mohr modified the milestones: 3.2.0, 3.3.0 Mar 7, 2024
@m-mohr m-mohr modified the milestones: 3.3.0, 3.4.0 Sep 11, 2024
@m-mohr m-mohr added the mapping label Jan 7, 2025
@m-mohr m-mohr mentioned this issue Jan 14, 2025
25 tasks
@m-mohr m-mohr linked a pull request Jan 25, 2025 that will close this issue
25 tasks
@m-mohr
Copy link
Collaborator

m-mohr commented Jan 25, 2025

This is fixed in #531.

@m-mohr m-mohr closed this as completed Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants