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

Don't use bundled AppCompat emojis. Instead download the font. #380

Merged
merged 2 commits into from
Sep 20, 2019

Conversation

vanniktech
Copy link
Owner

Much much better. Thanks to @stefanhaustein for providing the integration with Google's AppCompat.

Copy link
Collaborator

@rubengees rubengees left a comment

Choose a reason for hiding this comment

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

Nice

@mario
Copy link
Collaborator

mario commented Sep 20, 2019

Cool, but where are these certs from?

@vanniktech
Copy link
Owner Author

Google sample's everyone is using 'em.

@mario
Copy link
Collaborator

mario commented Sep 20, 2019

Generally saying I'm more for bundled option (privacy related), but oh well. Looks good otherwise.

@vanniktech vanniktech merged commit d544b2c into master Sep 20, 2019
@vanniktech vanniktech deleted the downloadable branch September 20, 2019 13:55
@vanniktech
Copy link
Owner Author

It's the sample app and everyone is showing how to use the bundled one so I'm for the font one. In general, people can choose what they want since we support both options.

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.

3 participants