mirror of
https://github.com/stenzek/duckstation.git
synced 2026-02-04 05:04:33 +00:00
[PR #2867] [MERGED] fix AchievementSummary/Notification font issue for ja/zh-cn #1249
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/stenzek/duckstation/pull/2867
Author: @Guwange
Created: 8/13/2022
Status: ✅ Merged
Merged: 8/18/2022
Merged by: @stenzek
Base:
master← Head:fix_notification_font_issue📝 Commits (3)
ff09edffix AchievementSummary/Notification font issue for ja/zh-cn01f0c1ffix OSD message about ChallengeMode to be translatable0d03aa2generate GlyphRanges for ja/zh-cn based on *.ts files📊 Changes
4 files changed (+97 additions, -11 deletions)
View changed files
➕
scripts/generate_update_glyph_ranges.py(+44 -0)📝
src/duckstation-qt/qthost.cpp(+44 -2)📝
src/frontend-common/achievements.cpp(+3 -2)📝
src/frontend-common/imgui_manager.cpp(+6 -7)📄 Description
This PR reverts imgui_manager.cpp modification made by below commit:
commit
853e9a8a6aAuthor: Connor McLaughlin stenzek@gmail.com
Date: Fri Jul 22 21:51:00 2022 +1000
AchievementSummary (and Leaderboard submittion) uses Notification, and Notification is drawn using medium_font and large_font.
There are already some translations for AchievementSummary/Leaderboard.
This PR fixes issue that these translated messages are currently shown with ???? string.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.