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

Make maps of Balkans and Saint Helena consistent with main style #258

Merged
merged 1 commit into from
Apr 7, 2020

Conversation

aplaice
Copy link
Collaborator

@aplaice aplaice commented Apr 6, 2020

Saint Helena's map could do with a zoom box, but I don't think it's essential since there are no other islands in the immediate vicinity, and Saint Helena's exact shape isn't that important.

The Balkan Peninsula map uses a slightly different choice of definition of the Balkans than the existing map, but I don't think it's a worse definition (our current map uses the Danube-Sava-Kupa line, while my map uses the Soča–Vipava–Krka–Sava–Danube line (which was what was in Natural Earth Data)). According to Wikipedia, "The northern border of the peninsula is variously defined.". (The definition used for the map in the infobox is the Danube–Sava–Soča one which is different yet...) In any case the differences are quite slight.

(See #1.)

Comparisons of the old and new maps

Balkans

before

after

Saint Helena

before

after

@axelboc
Copy link
Collaborator

axelboc commented Apr 7, 2020

Amazing! ✨

@axelboc axelboc added the content Content changes, map improvements, translation fixes, etc. label Apr 7, 2020
@axelboc axelboc added this to the v3.3 milestone Apr 7, 2020
@axelboc axelboc merged commit 5c6bcaa into anki-geo:master Apr 7, 2020
@axelboc axelboc mentioned this pull request Apr 7, 2020
@lbolan136
Copy link

Culturally Vojvodina and Slavonia should be considered part of the Balkan Penninsula.

@aplaice aplaice deleted the consistent_map branch January 2, 2021 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Content changes, map improvements, translation fixes, etc.
Development

Successfully merging this pull request may close these issues.

3 participants