Add Vazirmatn font for Farsi#3463
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds the Vazirmatn variable font and globally applies it to Farsi content, along with setting RTL direction for headings.
- Defines a new @font-face for Vazirmatn with variable weight range
- Scopes Farsi typography styles under a universal lang(fa) selector
- Applies RTL direction to headings within Farsi content
Comments suppressed due to low confidence (2)
assets/css/custom.scss:254
- [nitpick] It may help future maintainers to add a comment with the font source URL and license under the
@font-facedeclaration for clarity and attribution.
@font-face {
assets/css/custom.scss:256
- [nitpick] Check that the absolute URL
/assets/fonts/...resolves correctly in all deploy contexts; you might use a relative path (../fonts/...) or a build helper to ensure correct asset resolution.
src: url('/assets/fonts/Vazirmatn-Variable.woff2') format('woff2-variations');
| font-display: swap; | ||
| } | ||
|
|
||
| *:lang(fa) { |
There was a problem hiding this comment.
Using a universal selector here (*) can degrade rendering performance and may unintentionally affect non-text elements. Consider scoping to html:lang(fa) or body:lang(fa) instead.
| * { | ||
| font-family: 'Vazirmatn', sans-serif; |
There was a problem hiding this comment.
[nitpick] This nested universal selector will apply the font to every descendant, including icons or inline code. Restrict the selector to common text containers (e.g., body, p, h1, h2, blockquote).
| * { | |
| font-family: 'Vazirmatn', sans-serif; | |
| body, | |
| p, | |
| h1, | |
| h2, | |
| h3, | |
| blockquote { | |
| font-family: 'Vazirmatn', sans-serif; |
Replaces the universal selector (*) with a specific list of text-containing elements for applying the Vazirmatn font.
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
您好,請問,我接下來該怎麼做?
github-actions[bot] ***@***.***> 於 2025年7月30日 週三 20:14 寫道:
… *github-actions[bot]* left a comment (github/opensource.guide#3463)
<#3463 (comment)>
This pull request has been automatically marked as stale because it has
not had recent activity. It will be closed if no further activity occurs.
Thank you for your contributions.
—
Reply to this email directly, view it on GitHub
<#3463 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BVGNB26DTI3FHQOMP4YFLFD3LCZIZAVCNFSM6AAAAACAJQUHFWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTCMZWGAYDSNRVGA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
|
Can someone please review the pull request? |
|
const app = new Vue({ |
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.
This PR introduces the Vazirmatn font to improve the readability of Farsi (Persian) content.
This change adds the Vazirmatn variable font and applies it globally to all elements when the language is
fa. Vazirmatn is a free (as in freedom) Persian/Arabic font project that is extremely popular in Persian web content.Font source and license: https://github.com/rastikerdar/vazirmatn
Before / After
Before (left) and after (right) applying the Vazirmatn font. The new font improves readability for Persian text.