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

Japanese address scheme #3712

Closed
wants to merge 3 commits into from
Closed

Japanese address scheme #3712

wants to merge 3 commits into from

Conversation

MKuranowski
Copy link
Contributor

Just a temporary Japanese address scheme. Still country-specific placeholders should be defined, look at #3643.

@bhousel
Copy link
Member

bhousel commented Jan 2, 2017

Thanks, it looks ok, as far as I can tell..

I did notice something strange going on with address field combo boxes - it seems like d3 isn't always removing them. I'll dig into this today and try to find out what's happening. (It's not an issue with the code you added. I think it's an existing bug in the address field we didn't see before).

screenshot 2017-01-02 14 03 13

screenshot 2017-01-02 14 03 23

@bhousel
Copy link
Member

bhousel commented Jan 2, 2017

The weird field formatting has something to do with Helvetica Neue's handling of kanji glyphs. I don't understand the root cause, but I'm following some of the guidance here to use modern system fonts, which are all better than Helvetica Neue these days.

Most people probably will not notice the difference, but the new fontstack looks a little nicer..

@bhousel
Copy link
Member

bhousel commented Jan 2, 2017

This was merged

@bhousel bhousel closed this Jan 2, 2017
@bhousel
Copy link
Member

bhousel commented Jan 3, 2017

Whoops, I didn't see that you added postcode in fde26f8

I added it in aba1ab1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants