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.

Sign up for GitHub

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

Jump to bottom

refine edge labels #683

Merged

visr merged 1 commit into main from label-edges

Oct 25, 2023

Member

visr commented Oct 19, 2023

Fixes #175
Most of the work for this was already done in #658.

This changes a few things:

Stop using brackets around the ID to reduce visual clutter: now we use "Lake IJssel" #10 in printed errors and Lake IJssel #10 as a QGIS label
Add the edge ID to the label, this is the part that fixed Labeling of edges #175
Stop putting random strings as names in our basic test model
Add some real names to our Dutch waterways test model

evetion approved these changes

View reviewed changes

Member

evetion left a comment

I'm gonna miss my random names.

Besides, there's an Arhnem in the screenshot, what's up with that?


          refine edge labels

2db5484

visr force-pushed the label-edges branch from 6b962ce to 2db5484 Compare

October 25, 2023 19:46

visr merged commit 2d3b6d6 into main

15 checks passed

visr deleted the label-edges branch

October 25, 2023 20:21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet