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

Added an overlay that highlights curves. #171

Merged
merged 1 commit into from
Jan 9, 2024
Merged

Conversation

Sp3EdeR
Copy link
Owner

@Sp3EdeR Sp3EdeR commented Jan 8, 2024

Notes for reviewer:

@Sp3EdeR Sp3EdeR requested review from nbali and Csempi January 8, 2024 20:35
@Sp3EdeR Sp3EdeR enabled auto-merge (rebase) January 8, 2024 20:36
Copy link

github-actions bot commented Jan 8, 2024

PR Preview Action v1.4.6
Preview removed because the pull request was closed.
2024-01-09 06:34 UTC

@Sp3EdeR Sp3EdeR force-pushed the feature/curves-layer branch from cbd0b62 to 0e37bf3 Compare January 8, 2024 20:47
@nbali
Copy link
Collaborator

nbali commented Jan 9, 2024

Something is odd, this is what I see on the original page:
image

... and this is what I see on the preview:
image

Seems different to me. The yellow is missing above Sülysáp, and there is a missing section between Pánd and Káva. Did you filter the data further?

@nbali
Copy link
Collaborator

nbali commented Jan 9, 2024

Nevermind, just checked the original file:
image

... and it's not even there on the 300 version. I assume the webpage merges paths.
image

@Sp3EdeR Sp3EdeR merged commit d54de60 into master Jan 9, 2024
4 checks passed
@Sp3EdeR Sp3EdeR deleted the feature/curves-layer branch January 9, 2024 06:34
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