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

Add zoomboxes for disambiguation #357

Merged
merged 1 commit into from
Sep 13, 2020

Conversation

aplaice
Copy link
Collaborator

@aplaice aplaice commented Aug 19, 2020

Disambiguate Hong Kong and Macau, Guernsey and Jersey, Mayotte and
Comoros.

Fix #314, fix #344.

Maps before and after

Hong Kong

before

before

Macau

before

before

Guernsey

before

before

Jersey

before

before

Mayotte

before

before

Comoros

before

before

As usual, if the maps are OK, I'll upstream to Wikimedia and update sources.csv.

(Note that for Guernsey, in the zoombox I selected all the islands making up the Bailiwick of Guernsey, not just the island (or the jurisdiction) of Guernsey. (See also #241, which in-itself is no longer relevant due to the removal of the flags of small dependent territories, but which dealt with virtually the same issue.) I think that the issue is now more clear-cut since it's the Bailiwick which is a Crown Dependency, as stated in the Country info, not the Juridiction which is a sub-unit of the Bailiwick.)

@axelboc
Copy link
Collaborator

axelboc commented Aug 19, 2020

Love your work!! 😍

If I really had to nitpick, perhaps the zoom boxes of Macau, Hong Kong, Comoros, and Mayotte are a tiny bit too zoomed in... but it's really insignificant, so feel free to upload as is.

@aplaice
Copy link
Collaborator Author

aplaice commented Aug 19, 2020

The zoomed-in maps that I'm relying on for these four zoom boxes don't have much more context, so it'd be relatively hard to zoom out. Hence, if it's not that much of an issue, I'll leave them as they are!

@axelboc
Copy link
Collaborator

axelboc commented Aug 19, 2020

Fair enough, sounds good 👍

@axelboc axelboc added the content Content changes, map improvements, translation fixes, etc. label Aug 19, 2020
@axelboc axelboc added this to the v4.1 milestone Aug 19, 2020
@aplaice aplaice force-pushed the disambiguate_hong_kong_et_al branch from cb2ce46 to 8f8d3d6 Compare August 19, 2020 22:30
@aplaice aplaice changed the title [WIP] Add zoomboxes for disambiguation Add zoomboxes for disambiguation Aug 19, 2020
Disambiguate Hong Kong and Macau, Guernsey and Jersey, Mayotte and
Comoros.

Fix anki-geo#314, anki-geo#344.
@aplaice aplaice force-pushed the disambiguate_hong_kong_et_al branch from 8f8d3d6 to fa8196e Compare August 19, 2020 22:35
@axelboc
Copy link
Collaborator

axelboc commented Sep 13, 2020

Woops, sorry, forgot to merge this one.

@axelboc axelboc merged commit 7c6bf77 into anki-geo:master Sep 13, 2020
@aplaice aplaice deleted the disambiguate_hong_kong_et_al branch January 2, 2021 21:34
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.

Add zoom boxes to Guernsey and Jersey Disambiguate Macau & Hong Kong Maps
2 participants