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

Created translatable strings for fuel types, crossings, entrances and substations #467

Closed
wants to merge 16 commits into from

Conversation

Miroff
Copy link
Contributor

@Miroff Miroff commented May 24, 2022

  1. Created list of translatable values for fuel types based on Wiki
  2. Created limited set of values for RU based on discussion in local community. This set contains only fuel types used in the Russia.

If this PR was merged, I'm going to create similar preset for Belarus.

@Miroff Miroff changed the title Created translatable strings for fuel types Created translatable strings for fuel types, crossings, entrances and substations May 24, 2022
@Miroff
Copy link
Contributor Author

Miroff commented May 24, 2022

  1. Created list of translatable values for crossing types based on Crossings wiki page
  2. Created list of translatable values for entrance types based on Entrance wiki page
  3. Created list of translatable values for substation types based on Substation wiki page

@1ec5
Copy link
Contributor

1ec5 commented May 28, 2022

This would fix #404.

Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few things that jumped out at me at a glance, but this would benefit from a more detailed review.

"strings": {
"options": {
"traffic_signals": "Traffic signals and marking",
"uncontrolled": "Marking only",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, crosswalk classification is being discussed in #408.

"options": {
"traffic_signals": "Traffic signals and marking",
"uncontrolled": "Marking only",
"unmarked": "No Traffic Signals, no marking",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These values are using a mix of Title Case and Sentence case. I agree that title case can be annoying for long values, but the fields should at least be internally consistent.

"label": "Type",
"strings": {
"options": {
"yes": "Unspecified",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some possibly relevant naming considerations for entrance types in streetcomplete/StreetComplete#3949.

data/fields/fuel/fuel_multi.json Outdated Show resolved Hide resolved
data/fields/fuel/fuel_multi.json Outdated Show resolved Hide resolved
data/fields/substation.json Outdated Show resolved Hide resolved
@tyrasd
Copy link
Member

tyrasd commented Oct 17, 2022

I've tweaked this quite a bit and finally merged as 95c206b (sorry for the extra commits in your main branch @Miroff). I excluded the crossings field, as that is now handled in #590. I also changed the way the localized fuel field works: instead of duplicating the preset for every region, I made the field region-specific and let it inherit the strings from the global field. That avoids that translators have to translate the same string more than once, and should scale pretty nicely if you want to add more countries. Further, I removed lesser used fuel values (cut-off approx ~1000 usages with exceptions for the H2 types) and reworded the substation value strings slightly.

@tyrasd tyrasd closed this Oct 17, 2022
Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple minor things I noticed belatedly:

data/fields/fuel/fuel_multi.json Show resolved Hide resolved
data/fields/fuel/fuel_multi.json Show resolved Hide resolved
data/fields/substation.json Show resolved Hide resolved
tyrasd added a commit that referenced this pull request Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants