Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[android] set localIdeographFontFamily default to sans-serif #13925

Merged
merged 2 commits into from
Feb 19, 2019
Merged

Conversation

Chaoba
Copy link
Contributor

@Chaoba Chaoba commented Feb 14, 2019

No description provided.

@Chaoba Chaoba self-assigned this Feb 14, 2019
@Chaoba Chaoba requested a review from tobrun February 14, 2019 04:50
Copy link
Contributor

@tmpsantos tmpsantos left a comment

Choose a reason for hiding this comment

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

I'm OK with this, but can you add a test to make sure only CJK are generated?

@LukasPaczos
Copy link
Contributor

Could you also add javadoc entries stating that this value defaults to "sans-serif"?

@Chaoba
Copy link
Contributor Author

Chaoba commented Feb 15, 2019

I'm OK with this, but can you add a test to make sure only CJK are generated?

I am not sure how to test this function, do you have any suggestion?

@tmpsantos
Copy link
Contributor

I am not sure how to test this function, do you have any suggestion?

Nevermind, this can be done on the core engine.

/cc @tobrun

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

Successfully merging this pull request may close these issues.

4 participants