Merge pull request #6 from ilya-fedin/fix-bold

This commit is contained in:
Eric Kotato 2020-12-26 02:43:25 +03:00 committed by GitHub
commit b9dab597d8
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 6 additions and 55 deletions

View file

@ -28,7 +28,6 @@ void SetCustomFonts(const CustomFont &regular, const CustomFont &bold) {
QFont ResolveFont(uint32 flags, int size) {
static auto Database = QFontDatabase();
const auto fontOverride = ParseFamilyName(GetFontOverride(flags));
const auto overrideIsEmpty = GetPossibleEmptyOverride(flags).isEmpty();
const auto bold = ((flags & FontBold) || (flags & FontSemibold));
@ -46,8 +45,10 @@ QFont ResolveFont(uint32 flags, int size) {
: sizes;
const auto point = good.isEmpty() ? size : good.front();
result = Database.font(custom.family, custom.style, point);
} else if (!UseSystemFont || !overrideIsEmpty) {
result.setFamily(fontOverride);
} else {
if (!UseSystemFont || !overrideIsEmpty) {
result.setFamily(GetFontOverride(flags));
}
if (bold) {
if (CustomSemiboldIsBold) {
result.setBold(true);
@ -70,14 +71,6 @@ QFont ResolveFont(uint32 flags, int size) {
}
}
}
if (IsRealSemibold(fontOverride)) {
if (flags & FontItalic) {
result.setStyleName("Semibold Italic");
} else {
result.setStyleName("Semibold");
}
}
}
if (italic) {
result.setItalic(true);

View file

@ -42,26 +42,13 @@ uint32 fontKey(int size, uint32 flags, int family) {
return (((uint32(family) << 12) | uint32(size)) << 6) | flags;
}
QString RemoveSemiboldFromName(const QString &familyName) {
auto removedSemibold = familyName;
removedSemibold.remove("Semibold", Qt::CaseInsensitive);
return removedSemibold.trimmed();
}
bool ValidateFont(const QString &familyName, int flags = 0) {
const auto parsedFamily = ParseFamilyName(familyName);
QFont checkFont(parsedFamily);
QFont checkFont(familyName);
checkFont.setBold(flags & style::internal::FontBold);
checkFont.setItalic(flags & style::internal::FontItalic);
checkFont.setUnderline(flags & style::internal::FontUnderline);
if (IsRealSemibold(familyName)) {
checkFont.setStyleName("Semibold");
}
auto realFamily = QFontInfo(checkFont).family();
if (realFamily.trimmed().compare(parsedFamily, Qt::CaseInsensitive)) {
if (realFamily.trimmed().compare(familyName, Qt::CaseInsensitive)) {
UI_LOG(("Font Error: could not resolve '%1' font, got '%2'.").arg(familyName).arg(realFamily));
return false;
}
@ -305,33 +292,6 @@ QString GetFontOverride(int32 flags) {
return result.isEmpty() ? "Open Sans" : result;
}
bool IsRealSemibold(const QString &familyName) {
const auto removedSemibold = RemoveSemiboldFromName(familyName);
QFont originalFont(familyName);
QFont withoutSemiboldFont(removedSemibold);
withoutSemiboldFont.setStyleName("Semibold");
QFontInfo originalFontInfo(originalFont);
QFontInfo withoutSemiboldInfo(withoutSemiboldFont);
if (originalFontInfo.family().trimmed().compare(familyName, Qt::CaseInsensitive) &&
!withoutSemiboldInfo.family().trimmed().compare(removedSemibold, Qt::CaseInsensitive) &&
!withoutSemiboldInfo.styleName().trimmed().compare("Semibold", Qt::CaseInsensitive)) {
return true;
} else {
return false;
}
}
QString ParseFamilyName(const QString &familyName) {
if (IsRealSemibold(familyName)) {
return RemoveSemiboldFromName(familyName);
} else {
return familyName;
}
}
QString MonospaceFont() {
static const auto family = [&]() -> QString {
if (TryFont(CustomMonospaceFont)) {

View file

@ -24,8 +24,6 @@ extern bool UseOriginalMetrics;
void StartFonts();
[[nodiscard]] QString GetPossibleEmptyOverride(int32 flags = 0);
[[nodiscard]] QString GetFontOverride(int32 flags = 0);
[[nodiscard]] bool IsRealSemibold(const QString &familyName);
[[nodiscard]] QString ParseFamilyName(const QString &familyName);
[[nodiscard]] QString MonospaceFont();
void destroyFonts();