blakeridgway/supercell-wx
1
0
Fork 0
mirror of https://github.com/ciphervance/supercell-wx.git synced 2025-10-30 15:50:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Address ImGui clang-tidy comments, store font size as float

Browse source
This commit is contained in:
Dan Paulat 2025-08-28 11:16:23 -05:00
parent c82b9753c6
commit ffe1e7efaf
5 changed files with 20 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

4
scwx-qt/source/scwx/qt/gl/draw/placefile_text.cpp
View file

@ -62,10 +62,10 @@ public:
std::vector<std::shared_ptr<const gr::Placefile::TextDrawItem>> newList_ {};
std::vector<std::pair<std::shared_ptr<types::ImGuiFont>,
units::font_size::pixels<int>>>
units::font_size::pixels<float>>>
fonts_ {};
std::vector<std::pair<std::shared_ptr<types::ImGuiFont>,
units::font_size::pixels<int>>>
units::font_size::pixels<float>>>
newFonts_ {};
};

19
scwx-qt/source/scwx/qt/manager/font_manager.cpp
View file

@ -84,11 +84,11 @@ public:
boost::unordered_flat_map<std::string, std::vector<char>> rawFontData_ {};
std::mutex rawFontDataMutex_ {};
std::pair<std::shared_ptr<types::ImGuiFont>, units::font_size::pixels<int>>
std::pair<std::shared_ptr<types::ImGuiFont>, units::font_size::pixels<float>>
defaultFont_ {};
boost::unordered_flat_map<types::FontCategory,
std::pair<std::shared_ptr<types::ImGuiFont>,
units::font_size::pixels<int>>>
units::font_size::pixels<float>>>
fontCategoryImguiFontMap_ {};
boost::unordered_flat_map<types::FontCategory, QFont>
fontCategoryQFontMap_ {};
@ -160,13 +160,18 @@ void FontManager::InitializeFonts()
}
}
units::font_size::pixels<int>
units::font_size::pixels<float>
FontManager::ImFontSize(units::font_size::pixels<double> size)
{
static constexpr units::font_size::pixels<int> kMinFontSize_ {8};
static constexpr units::font_size::pixels<int> kMaxFontSize_ {96};
// Only allow whole pixels, and clamp to 6-72 pt
units::font_size::pixels<double> pixels {size};
units::font_size::pixels<int> imFontSize {
std::clamp(static_cast<int>(pixels.value()), 8, 96)};
const units::font_size::pixels<double> pixels {size};
const units::font_size::pixels<int> imFontSize {
std::clamp(static_cast<int>(pixels.value()),
kMinFontSize_.value(),
kMaxFontSize_.value())};
return imFontSize;
}
@ -223,7 +228,7 @@ int FontManager::GetFontId(types::Font font) const
return -1;
}
std::pair<std::shared_ptr<types::ImGuiFont>, units::font_size::pixels<int>>
std::pair<std::shared_ptr<types::ImGuiFont>, units::font_size::pixels<float>>
FontManager::GetImGuiFont(types::FontCategory fontCategory)
{
std::unique_lock lock {p->fontCategoryMutex_};

4
scwx-qt/source/scwx/qt/manager/font_manager.hpp
View file

@ -28,7 +28,7 @@ public:
std::shared_mutex& imgui_font_atlas_mutex();
int GetFontId(types::Font font) const;
std::pair<std::shared_ptr<types::ImGuiFont>, units::font_size::pixels<int>>
std::pair<std::shared_ptr<types::ImGuiFont>, units::font_size::pixels<float>>
GetImGuiFont(types::FontCategory fontCategory);
QFont GetQFont(types::FontCategory fontCategory);
std::shared_ptr<types::ImGuiFont>
@ -39,7 +39,7 @@ public:
void LoadApplicationFont(types::Font font, const std::string& filename);
void InitializeFonts();
static units::font_size::pixels<int>
static units::font_size::pixels<float>
ImFontSize(units::font_size::pixels<double> size);
static FontManager& Instance();

2
scwx-qt/source/scwx/qt/manager/placefile_manager.hpp
View file

@ -26,7 +26,7 @@ public:
using FontMap =
boost::unordered_flat_map<std::size_t,
std::pair<std::shared_ptr<types::ImGuiFont>,
units::font_size::pixels<int>>>;
units::font_size::pixels<float>>>;
bool placefile_enabled(const std::string& name);
bool placefile_thresholded(const std::string& name);

7
scwx-qt/source/scwx/qt/types/imgui_font.cpp
View file

@ -23,9 +23,8 @@ static const auto logger_ = scwx::util::Logger::Create(logPrefix_);
class ImGuiFont::Impl
{
public:
explicit Impl(const std::string& fontName,
const std::vector<char>& fontData) :
fontName_ {fontName}
explicit Impl(std::string fontName, const std::vector<char>& fontData) :
fontName_ {std::move(fontName)}
{
CreateImGuiFont(fontData);
}
@ -34,7 +33,7 @@ public:
void CreateImGuiFont(const std::vector<char>& fontData);
const std::string fontName_;
std::string fontName_;
ImFont* imFont_ {nullptr};
};