Skip to content

add chinese font in base contaimer #339

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

Closed
wants to merge 1 commit into from

Conversation

b4456609
Copy link

@b4456609 b4456609 commented Dec 2, 2016

The screenshots in selenium chrome node shows that Chinese characters display with square boxes. Install Chinese font could solve the problem.

@elgalu
Copy link
Member

elgalu commented Dec 2, 2016

Note installing fonts-noto-cjk will add 126.7 MB to the docker image, perhaps is better generating a separate image in this case.

@HackToday
Copy link

Adding Chinese support is good. however not sure if those containers could accept such locale support for general cases. As different language may have different requrements

@ddavison ddavison added this to the I18n milestone Dec 14, 2016
@ddavison
Copy link
Member

i'm going to keep this PR open just for reference, but i've assigned a milestone that i would like to achieve with these containers, and that is internationalization.

@diemol
Copy link
Member

diemol commented May 9, 2017

Someone sent a PR to elgalu/docker-selenium#153, with the same purpose.
The main difference is that the installed packages only need 30MB more. Maybe this is an alternative?

@Cyperwu
Copy link

Cyperwu commented Jun 14, 2017

Is this Issue going any further?

@ddavison
Copy link
Member

regarding your comment, @diemol, i tried a before-after using these packages:

  • fonts-ipafont-gothic
  • fonts-wqy-zenhei

Before and after had the same result - and the result looked like this:
screen shot 2018-01-22 at 1 59 24 pm

@diemol
Copy link
Member

diemol commented Mar 18, 2018

Closing this one since we merged #676

@diemol diemol closed this Mar 18, 2018
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.

6 participants