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

Rendering for ringforts #4969

Open
b-unicycling opened this issue May 22, 2024 · 1 comment
Open

Rendering for ringforts #4969

b-unicycling opened this issue May 22, 2024 · 1 comment

Comments

@b-unicycling
Copy link

I would appreciate if ringforts could be rendered different than generic archaeological sites. There are now over 12,000 ringforts mapped, and the number is still rising. I've created an icon of how they could be rendered as a POI, however, I would prefer if they were rendered as an area with a hashed line.
The icon reflects the often (but not always!) circular nature of this archaeological site type, and I've tried to reflect that they can have multiple ditches.
Ringfort svg

If anyone wants to improve the design, be my guest.

@imagico
Copy link
Collaborator

imagico commented May 30, 2024

12k uses, almost all of them in Ireland, the vast majority on polygons.

At this stage this is very region specific in its use and we'd get significantly ahead of us if we'd specifically render it. More important at this stage would be rendering of historic=ruins (153k uses #331), possibly as part of a general concept of rendering ruined structures (#331 (comment)) and differentiated rendering of historic=archaeological_site (184k) based on archaeological_site=* (134k) and/or historic:civilization=* (89k). Fill/perimeter rendering of historic=archaeological_site polygons (52k) could also be considered (but would need to be done in a way that works decently in combination with micromapping details on larger sites).

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