-
Notifications
You must be signed in to change notification settings - Fork 50
Add regular and semibold typefaces of Inter font #49
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
Conversation
Concept ACK. To reduce risk of conflicts during syncing with the main repo I'd refrain from the first commit. We could add it later, after merging this repo into the main one. |
@hebasto good point, I'll remove it and maybe propose the prefix in main repo 1st. Edit: actually the 1st and 2nd commits can be proposed 1st. |
d54ec27 qt: Add helper to load font (João Barbosa) Pull request description: Originally submitted as bitcoin-core/gui-qml#49. ACKs for top commit: hebasto: re-ACK d54ec27 stratospher: Tested ACK d54ec27. Refactoring the code and defining `loadFont()` in `src/qt/guiutil.cpp` reduces redundant imports of the `QFontDatabase` and is a better design. shaavan: ACK d54ec27 Tree-SHA512: b156bb6ffb08dd57476f383a29bbb0a1108b62794d430debb77252f7d09df1409a7532b09d17d8836d1c2ab7c126a6618231164b9d0def1b8f361a81ef22d107
a5c1200
to
26de8b0
Compare
Pull request description: Sync with the main repo up to bitcoin-core/gui@ef59692. This sync includes: - bitcoin/bitcoin#23242 ported from #49 - bitcoin/bitcoin#23286 ported from #58 Parent commits: ``` $ git rev-parse 8967f19^@ 8cfb3cb ef59692 ``` ACKs for top commit: jarolrod: ACK 8967f19 Tree-SHA512: c387e638862afe5fd4711d9c1aad36165d04f328612c9b74b35a50b477933e6c6b8cf0674caf3b02e24a22d8598aee7e880baf4c3879c629f71a92af31714548
One more rebase? |
26de8b0
to
5071af5
Compare
5071af5
to
b36cf20
Compare
@hebasto rebased. |
b36cf20
to
3751fd8
Compare
3751fd8
to
b36cf20
Compare
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.
ACK b36cf20, tested on Linux Mint 20.2 (Qt 5.12.8):
Verified hashes of the font files in the src/qml/res/fonts/
sub-directory:
$ sha256sum *
a7e791e8f5a0fb02b65663f7fca73e1d1ca9543f772ad480cbd76f4e3fe3f8cc Inter-Regular.otf
8c1990b6012254ea2b487161697d107357dd0ee55811cfd91c8c11227bbef457 Inter-SemiBold.otf
Hashes match ones I get for files from the Inter-3.19.zip
archive which I've downloaded from https://rsms.me/inter/
0d624261ef Merge bitcoin-core/crc32c-subtree#2: Merge upstream cac7ca830b Merge commit 'fa5ade41ee480003d9c5af6f43567ba22e4e17e6' into bitcoin-fork fa5ade41ee Fix compilation warnings on ARM64 with old GCC versions. (bitcoin-core#52) db08d22129 Updated Travis-CI configuration. (bitcoin-core#51) e31619a5b7 Fix GitHub links. (bitcoin-core#50) 7fa4c263e8 Update Travis CI config. (bitcoin-core#49) a3d9e6d1a4 Updated third_party/ and Travis CI config. (bitcoin-core#48) git-subtree-dir: src/crc32c git-subtree-split: 0d624261ef83ab08c953c196540ed18f355add4c
d54ec27 qt: Add helper to load font (João Barbosa) Pull request description: Originally submitted as bitcoin-core/gui-qml#49. ACKs for top commit: hebasto: re-ACK d54ec27 stratospher: Tested ACK d54ec27. Refactoring the code and defining `loadFont()` in `src/qt/guiutil.cpp` reduces redundant imports of the `QFontDatabase` and is a better design. shaavan: ACK d54ec27 Tree-SHA512: b156bb6ffb08dd57476f383a29bbb0a1108b62794d430debb77252f7d09df1409a7532b09d17d8836d1c2ab7c126a6618231164b9d0def1b8f361a81ef22d107
As requested by @GBKS.