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

American English and roundabouts #256

Closed
wants to merge 2 commits into from
Closed

Conversation

1ec5
Copy link
Member

@1ec5 1ec5 commented Jul 23, 2018

Issue

Added an American English localization that is programmatically derived from the international English localization. The English localization remains the default. The two localizations are identical, except for circular junction terminology. The English localization calls all circular junctions “roundabouts” – which is a change in favor of non-American usage – whereas the American English localization distinguishes between classical roundabouts as “traffic circles” and modern roundabouts as “roundabouts”. I think we could justify calling them all roundabouts in American English, but this is a slightly more conservative change that respects East Coast tradition.

This isn’t exactly a backwards-incompatible change, but clients that wish to retain some “traffic circle” usage should ensure the use of the en-US locale instead of relying on the en default.

Fixes #188.

Tasklist

  • Align to international English usage
  • Add American English localization
  • Add changelog entry
  • Test with osrm-frontend (?)
  • Review

/cc @danpaz @bsudekum

1ec5 added 2 commits July 23, 2018 15:19
Added an American English localization that is programmatically derived from the international English localization. The two are identical except for circular junction terminology.
@1ec5 1ec5 added the english label Jul 23, 2018
@1ec5 1ec5 self-assigned this Jul 23, 2018
@1ec5 1ec5 requested a review from danpaz July 23, 2018 23:09
@1ec5
Copy link
Member Author

1ec5 commented Aug 13, 2018

This isn’t exactly a backwards-incompatible change, but clients that wish to retain some “traffic circle” usage should ensure the use of the en-US locale instead of relying on the en default.

@danpaz reminded me that the language=en default (when the parameter is omitted) is quite entrenched in the Mapbox Directions API and perhaps other clients as well. I overlooked this possibility because the Mapbox Navigation SDK automatically specifies a full locale code including a region, such as en-US.

One possibility would be to fold the remaining “traffic circle” usage back into the en locale and create localizations for each of the major English locales (including GB, CA, AU, NZ, IN, and ZA). But even if we automate those locales using a transformation script, as this PR attempts to do, that’s a lot of duplication in the final package for relatively little gain.

#155 would enable us to avoid duplicating the entire English localization and ship the package with “sparse” localizations that only override individual strings. In the meantime, we might want to consider paring back this PR to just replace “traffic circle” with “roundabout” everywhere until #155 gives us that extra flexibility.

@andrewharvey
Copy link

One possibility would be to fold the remaining “traffic circle” usage back into the en locale and create localizations for each of the major English locales (including GB, CA, AU, NZ, IN, and ZA). But even if we automate those locales using a transformation script, as this PR attempts to do, that’s a lot of duplication in the final package for relatively little gain.

Not sure if you could just create an en-gb, then symlink en-ca, en-au, etc all to en-gb until such time as they need to diverge?

@1ec5
Copy link
Member Author

1ec5 commented Jul 23, 2019

Per chat with @danpaz, we’re going to change the wording to roundabout for all English locales, making an en-US locale unnecessary.

@1ec5 1ec5 mentioned this pull request Jul 23, 2019
5 tasks
@1ec5
Copy link
Member Author

1ec5 commented Jul 23, 2019

Circling back on this fix in #285.

@1ec5 1ec5 closed this Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename traffic circles to roundabouts
2 participants