-
Notifications
You must be signed in to change notification settings - Fork 164
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
Conversation
|
This would fix #404. |
There was a problem hiding this 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.
data/fields/crossing.json
Outdated
"strings": { | ||
"options": { | ||
"traffic_signals": "Traffic signals and marking", | ||
"uncontrolled": "Marking only", |
There was a problem hiding this comment.
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.
data/fields/crossing.json
Outdated
"options": { | ||
"traffic_signals": "Traffic signals and marking", | ||
"uncontrolled": "Marking only", | ||
"unmarked": "No Traffic Signals, no marking", |
There was a problem hiding this comment.
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.
data/fields/entrance.json
Outdated
"label": "Type", | ||
"strings": { | ||
"options": { | ||
"yes": "Unspecified", |
There was a problem hiding this comment.
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.
9d3204d
to
49f529e
Compare
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. |
There was a problem hiding this 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:
If this PR was merged, I'm going to create similar preset for Belarus.