Skip to content

Niord josm seachart#113

Open
nvasta wants to merge 4 commits intoNiordOrg:masterfrom
nvasta:niord-josm-seachart
Open

Niord josm seachart#113
nvasta wants to merge 4 commits intoNiordOrg:masterfrom
nvasta:niord-josm-seachart

Conversation

@nvasta
Copy link

@nvasta nvasta commented Feb 24, 2021

The current version of the josm-seachart does not support virtual AtoNs. I did find however, a newer version that does:

https://github.com/openstreetmap/josm-plugins/tree/master/seachart

I am not sure this is something that interests the main project but it has minimal code from my side and it does seems to do the job. That is it will generate icons when the "seamark:ref" is set to "virtual_aton".

SInce the virtual AtoN icons are larger though, I had to tweak the icon generation so that it centers and positions the icons a bit differently.

I need to be able to differentiate between AtoNs and VAtoNs to make the latter slightly smaller. Also the icons are now completely centered, so the buoys do appear slightly higher in the generated images. Finally I increased a bit the width of the generated images, so that they fit.

@nvasta
Copy link
Author

nvasta commented Feb 24, 2021

I think we might need a follow up to fix the centering for the selections in the web map view.

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot E 1 Security Hotspot
Code Smell A 126 Code Smells

No Coverage information No Coverage information
3.3% 3.3% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

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.

1 participant