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

Mention ZIP Code in the "place" field placeholder #663

Open
WebworkrNet opened this issue Sep 11, 2021 · 9 comments
Open

Mention ZIP Code in the "place" field placeholder #663

WebworkrNet opened this issue Sep 11, 2021 · 9 comments
Labels
i18n Internationalization

Comments

@WebworkrNet
Copy link
Contributor

https://www.datenanfragen.de/generator/

In my opinion, the field for city should be supplemented by postcode.
Currently, the entry of the postcode only results from the context.

Screenshot_20210911-121220_Firefox

@zner0L
Copy link
Member

zner0L commented Sep 11, 2021

This is deliberate to avoid having to properly internationalize ZIP codes and city names/places. Often times, countries to this very differently and sometimes ZIP codes don't even correspond to city names or municipalities but other places entirely. Restricting this field means loosing a lot of necessary flexibility.

If you think this is confusing, I guess we could clarify by mentioning the ZIP code in the placeholder.

@mal-tee
Copy link
Member

mal-tee commented Sep 11, 2021

Once I got confused by this as well, I think we should clarify it. :D

@WebworkrNet
Copy link
Contributor Author

If you think this is confusing, I guess we could clarify by mentioning the ZIP code in the placeholder.

Yes, that's what I had imagined. :-)

@zner0L zner0L changed the title Generator / my data: field "place" Mention ZIP Code in the "place" field placeholder Oct 6, 2021
@zner0L zner0L added hacktoberfest help wanted Extra attention is needed i18n Internationalization good first issue Good for newcomers labels Oct 6, 2021
@zner0L
Copy link
Member

zner0L commented Oct 6, 2021

If anyone wants to do this: What needs to be done is basically changing the string for the placeholder to include the word "ZIP" or a translated version. So a new placeholder would read: "Place and ZIP" or something similar. If you can try to do this in at least German and English. The strings can be found in src/i18n. Please read our guide on translations before submitting.

@baltpeter
Copy link
Member

Do we really want to implement that? I'm pretty sure, this will break for some countries. For example, in the UK, the ZIP should be on a separate line from the city.

I do regret that we implemented this complex input for the addresses in the first place. I'm pretty sure we don't need the separation into the different components anywhere. We should have just used a textarea…

@mal-tee
Copy link
Member

mal-tee commented Oct 6, 2021

We can still move "back" to a textarea? :D

@baltpeter
Copy link
Member

baltpeter commented Oct 6, 2021

We could but someone would have to write code that migrates the old saved ID data to that. Probably not the end of the world but easy to do something wrong and mess up someone's saved ID data.

@zner0L
Copy link
Member

zner0L commented Oct 7, 2021

Not just that, quite a lot of code depends that. I think of we move to a textarea, then we need to enforce a limit of lines, because otherwise it would break the address field at some point.

@baltpeter
Copy link
Member

That too, good point. I'd say it would be possible if we were using Typescript and had a reliable way of finding uses of that. But alas we don't.

Anyway, the question of whether we want to change the placeholder remains. I'm voting "no". If anything, I think we should just rename them all to "Address line {1,2,…}".

@baltpeter baltpeter removed help wanted Extra attention is needed good first issue Good for newcomers hacktoberfest labels Oct 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n Internationalization
Development

No branches or pull requests

4 participants