Skip to content
This repository was archived by the owner on Jun 28, 2021. It is now read-only.

Added Nafees font for Urdu and Punjabi translation #486

Merged
merged 1 commit into from
Nov 9, 2016

Conversation

naveed-ahmad
Copy link
Contributor

Added ability to configure fonts for translation. This PR is adding fonts for Urdu, and Punjabi language for now. But it has opened the door to add fonts for any language :)

Default font:
2016-11-09_0407

New Nafees font:
2016-11-09_0407-after

@ahmedre
Copy link
Contributor

ahmedre commented Nov 8, 2016

Deployed to: http://staging.quran.com:32857

@naveed-ahmad naveed-ahmad changed the title added Nafees font for Urdu and Punjabi translation Added Nafees font for Urdu and Punjabi translation Nov 8, 2016
Copy link
Contributor

@thabti thabti left a comment

Choose a reason for hiding this comment

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

Awesome PR, LGTM

@thabti
Copy link
Contributor

thabti commented Nov 8, 2016

@naveed-ahmad what are your thoughts on the font size? Looks pretty small when using the new font.

@naveed-ahmad
Copy link
Contributor Author

@sabeurthabti default fonts actually looks small to me :) any average Urdu reader can read the new 'Nafees' pretty easily. Bottom line, font size looks good to me. Urdu is my mother tongue btw.

@thabti
Copy link
Contributor

thabti commented Nov 9, 2016

LGTM

@thabti thabti merged commit 2f292b1 into quran:master Nov 9, 2016
@mmahalwy
Copy link
Contributor

mmahalwy commented Nov 9, 2016

this is awesome!! thank you thank you thank you @naveed-ahmad

@mmahalwy
Copy link
Contributor

mmahalwy commented Nov 9, 2016

@ahmedre we may need to add these font files to the CDN please!

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