FontEditor: Update GlyphMap on font type change

Since font type changes also change the amount of glyphs in a font, the glyph map has to be re-rendered to properly showcase the change.
abyesilyurt · Apr 18, 2021 · 60f82e0 · 60f82e0
1 parent f461ee7
commit 60f82e0
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 0 deletions.
diff --git a/Userland/Applications/FontEditor/FontEditor.cpp b/Userland/Applications/FontEditor/FontEditor.cpp
@@ -288,6 +288,7 @@ FontEditorWidget::FontEditorWidget(const String& path, RefPtr<Gfx::BitmapFont>&&
 
  m_type_combobox->on_change = [this](auto&, const auto& index) {
  m_edited_font->set_type(static_cast<Gfx::FontTypes>(index.row()));
+ m_glyph_map_widget->reprobe_font();
  };
 
  m_presentation_spinbox->on_change = [this, update_demo](int value) {

diff --git a/Userland/Applications/FontEditor/GlyphMapWidget.cpp b/Userland/Applications/FontEditor/GlyphMapWidget.cpp
@@ -90,6 +90,14 @@ void GlyphMapWidget::update_glyph(int glyph)
  update(get_outer_rect(glyph));
 }
 
+void GlyphMapWidget::reprobe_font()
+{
+ VERIFY(m_font);
+ m_glyph_count = m_font->glyph_count();
+ m_selected_glyph = 0;
+ update();
+}
+
 void GlyphMapWidget::paint_event(GUI::PaintEvent& event)
 {
  GUI::Frame::paint_event(event);

diff --git a/Userland/Applications/FontEditor/GlyphMapWidget.h b/Userland/Applications/FontEditor/GlyphMapWidget.h
@@ -47,6 +47,7 @@ class GlyphMapWidget final : public GUI::ScrollableWidget {
  const Gfx::BitmapFont& font() const { return *m_font; }
 
  void update_glyph(int);
+ void reprobe_font();
 
  Function<void(int)> on_glyph_selected;