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

Added Roboto Font to the app #172

Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

Added Roboto Font to the app #172

wants to merge 1 commit into from

Conversation

batbrain7
Copy link

@batbrain7 batbrain7 commented Jan 5, 2018

Fixes #159

Added Roboto Font to the app using downloadable fonts as described in the link provided in the issue description.

@batbrain7
Copy link
Author

@therajanmaurya please review and merge this PR :)

@therajanmaurya
Copy link
Member

@batbrain7 We are not going to add fonts in our app. we will use downloadable fonts and how downloadable fonts work please see the second link. I think you missed to read issue description. Please check again and here the link to implement the downloadable font: https://developer.android.com/guide/topics/ui/look-and-feel/downloadable-fonts.htmt.

@therajanmaurya
Copy link
Member

The first reason to not use your present implementation. It's making code messier and it's increasing the APK size. we don't want it, not even a single kb.

@batbrain7
Copy link
Author

@therajanmaurya please merge this PR, I updated this with the suggested changes quite a time ago now.

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.

None yet

2 participants