organicmaps/imgui
13
Fork 0
mirror of https://github.com/ocornut/imgui.git synced 2025-04-05 05:25:08 +00:00
Code Issues 1 Projects Releases Wiki Activity Actions

WIP - Fonts: removed size rounding in AddFont() which breaks relative sizing of merged fonts (8502)

Browse source
This commit is contained in:
ocornut 2025-03-20 15:45:27 +01:00
parent 2f3c2a9a8c
commit 5071564562
3 changed files with 13 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
imgui.cpp
View file

@ -8513,7 +8513,12 @@ void ImGui::UpdateCurrentFontSize()
float final_size = g.FontSizeBeforeScaling * g.IO.FontGlobalScale;
if (ImGuiWindow* window = g.CurrentWindow)
final_size *= window->FontWindowScale * window->FontDpiScale;
final_size = ImMax(1.0f, IM_ROUND(final_size));
// Round font size
// - We started rounding in 1.90 WIP (18991) as our layout system currently doesn't support non-rounded font size well yet.
// - We may support it better later and remove this rounding.
final_size = GetRoundedFontSize(final_size);
final_size = ImMax(1.0f, final_size);
g.FontSize = final_size;
g.FontBaked = (g.Font != NULL) ? g.Font->GetFontBaked(g.FontSize) : NULL;

12
imgui_draw.cpp
View file

@ -3027,6 +3027,7 @@ ImFont* ImFontAtlas::AddFont(const ImFontConfig* font_cfg)
}
// Sanity check
// We don't round cfg.SizePixels yet as relative size of merged fonts are used afterwards.
if (font_cfg->GlyphExcludeRanges != NULL)
{
int size = 0;
@ -3042,12 +3043,6 @@ ImFont* ImFontAtlas::AddFont(const ImFontConfig* font_cfg)
}
IM_ASSERT(font_cfg->FontLoaderData == NULL);
// Round font size
// - We started rounding in 1.90 WIP (18991) as our layout system currently doesn't support non-rounded font size well yet.
// - Note that using io.FontGlobalScale or SetWindowFontScale(), with are legacy-ish, partially supported features, can still lead to unrounded sizes.
// - We may support it better later and remove this rounding.
new_font_cfg.SizePixels = ImTrunc(new_font_cfg.SizePixels);
// Pointers to Sources are otherwise dangling
ImFontAtlasBuildUpdatePointers(this);
if (!ImFontAtlasBuildAddFont(this, &new_font_cfg))
@ -5167,6 +5162,11 @@ ImGuiID ImFontAtlasBakedGetId(ImGuiID font_id, float baked_size)
ImFontBaked* ImFont::GetFontBaked(float size)
{
ImFontBaked* baked = LastBaked;
// Round font size
// - ImGui::PushFontSize() will already round, but other paths calling GetFontBaked() directly also needs it (e.g. ImFontAtlasBuildPreloadAllGlyphRanges)
size = ImGui::GetRoundedFontSize(size);
if (baked && baked->Size == size)
return baked;

1
imgui_internal.h
View file

@ -3340,6 +3340,7 @@ namespace ImGui
// Fonts, drawing
IMGUI_API void SetCurrentFont(ImFont* font, float font_size);
IMGUI_API void UpdateCurrentFontSize();
inline float GetRoundedFontSize(float size) { return IM_ROUND(size); }
inline ImFont* GetDefaultFont() { ImGuiContext& g = *GImGui; return g.IO.FontDefault ? g.IO.FontDefault : g.IO.Fonts->Fonts[0]; }
IMGUI_API void PushPasswordFont();
IMGUI_API void PopPasswordFont();