-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Add new header font #13714
Add new header font #13714
Changes from all commits
377bbd0
b337c91
77a1bb9
f6b3c7b
3f86808
4e70ca0
2dcb02c
8ab77fd
f43c956
18ae005
9c8da8b
3900956
16196e4
c20b818
ed901aa
b1ad116
919311b
996555b
279d5f4
34a3d84
5642c2d
6b55520
f56685c
46f2c75
45b6919
483d58f
6ad3e10
b7fac7c
1c15ed5
4d66239
f253e09
4f35a78
3bc2229
3622e05
99a7d53
61b4e92
c3b1102
cb248a0
bb45131
33ea15b
bae0e12
bf75082
5e768b5
c750c71
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -27,11 +27,11 @@ | |
}, | ||
{ | ||
"path": "assets/fonts/native/ExpensifyNewKansas-Medium.otf", | ||
"sha1": "b07181fac8d0602d4b18c339bdbef9589c67c9f9" | ||
"sha1": "6c670679ccb67975edba5e8dd59dc883ff402b25" | ||
}, | ||
{ | ||
"path": "assets/fonts/native/ExpensifyNewKansas-MediumItalic.otf", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note: It seems that permission got changed for these files for the group level. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could you clarify what you mean by that? |
||
"sha1": "00636ab2038c9690a123985b70a944c42cf65b8a" | ||
"sha1": "42b8a320808e1f0aed6b65fd06edd51f6cf5515c" | ||
} | ||
] | ||
} |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Coming from #35435. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we have to change these files? I think they were already updated on the main. Please correct me if I am wrong.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated the font files with the new ones provided by Shawn here and then ran
npx react-native-asset
to link them. These changes were made by the script.