فهرست منبع

LibGfx: Remove Gfx::FontDatabase::default_bold_fixed_width_font()

Ask for a bold_variant() of the default_fixed_width_font() instead.
Andreas Kling 4 سال پیش
والد
کامیت
8a6c37deef

+ 0 - 5
Tests/LibGfx/TestFontHandling.cpp

@@ -46,11 +46,6 @@ TEST_CASE(test_default_fixed_width_font)
     EXPECT(!Gfx::FontDatabase::default_fixed_width_font().name().is_null());
 }
 
-TEST_CASE(test_default_bold_fixed_width_font)
-{
-    EXPECT(!Gfx::FontDatabase::default_bold_fixed_width_font().name().is_null());
-}
-
 TEST_CASE(test_clone)
 {
     u8 glyph_height = 1;

+ 1 - 1
Userland/Demos/LibGfxDemo/main.cpp

@@ -149,7 +149,7 @@ void Canvas::draw()
     painter.draw_text({ 520, 415, 240, 20 }, "Normal text", Gfx::FontDatabase::default_font(), Gfx::TextAlignment::CenterLeft, Color::Red);
     painter.draw_text({ 520, 430, 240, 20 }, "Bold text", Gfx::FontDatabase::default_font().bold_variant(), Gfx::TextAlignment::CenterLeft, Color::Green);
     painter.draw_text({ 520, 450, 240, 20 }, "Normal text (fixed width)", Gfx::FontDatabase::default_fixed_width_font(), Gfx::TextAlignment::CenterLeft, Color::Blue);
-    painter.draw_text({ 520, 465, 240, 20 }, "Bold text (fixed width)", Gfx::FontDatabase::default_bold_fixed_width_font(), Gfx::TextAlignment::CenterLeft, Color::Yellow);
+    painter.draw_text({ 520, 465, 240, 20 }, "Bold text (fixed width)", Gfx::FontDatabase::default_fixed_width_font().bold_variant(), Gfx::TextAlignment::CenterLeft, Color::Yellow);
 
     auto font = Gfx::BitmapFont::load_from_file("/res/fonts/PebbletonBold14.font");
     painter.draw_rect({ 520, 510, 240, 30 }, Color::DarkGray);

+ 1 - 1
Userland/Games/Snake/SnakeGame.cpp

@@ -15,7 +15,7 @@
 
 SnakeGame::SnakeGame()
 {
-    set_font(Gfx::FontDatabase::default_bold_fixed_width_font());
+    set_font(Gfx::FontDatabase::default_fixed_width_font().bold_variant());
     m_fruit_bitmaps.append(*Gfx::Bitmap::load_from_file("/res/icons/snake/paprika.png"));
     m_fruit_bitmaps.append(*Gfx::Bitmap::load_from_file("/res/icons/snake/eggplant.png"));
     m_fruit_bitmaps.append(*Gfx::Bitmap::load_from_file("/res/icons/snake/cauliflower.png"));

+ 0 - 10
Userland/Libraries/LibGfx/FontDatabase.cpp

@@ -44,16 +44,6 @@ Font& FontDatabase::default_fixed_width_font()
     return *font;
 }
 
-Font& FontDatabase::default_bold_fixed_width_font()
-{
-    static Font* font;
-    if (!font) {
-        font = FontDatabase::the().get_by_name("Csilla 10 700");
-        VERIFY(font);
-    }
-    return *font;
-}
-
 struct FontDatabase::Private {
     HashMap<String, RefPtr<Gfx::Font>> full_name_to_font_map;
     Vector<RefPtr<Typeface>> typefaces;

+ 0 - 2
Userland/Libraries/LibGfx/FontDatabase.h

@@ -35,9 +35,7 @@ public:
     static FontDatabase& the();
 
     static Font& default_font();
-
     static Font& default_fixed_width_font();
-    static Font& default_bold_fixed_width_font();
 
     RefPtr<Gfx::Font> get(const String& family, unsigned size, unsigned weight);
     RefPtr<Gfx::Font> get(const String& family, const String& variant, unsigned size);

+ 1 - 2
Userland/Libraries/LibVT/TerminalWidget.cpp

@@ -1063,8 +1063,7 @@ void TerminalWidget::did_change_font()
     GUI::Frame::did_change_font();
     m_line_height = font().glyph_height() + m_line_spacing;
 
-    // TODO: try to find a bold version of the new font (e.g. CsillaThin7x10 -> CsillaBold7x10)
-    const Gfx::Font& bold_font = Gfx::FontDatabase::default_bold_fixed_width_font();
+    const Gfx::Font& bold_font = font().bold_variant();
 
     if (bold_font.glyph_height() == font().glyph_height() && bold_font.glyph_width(' ') == font().glyph_width(' '))
         m_bold_font = &bold_font;

+ 1 - 1
Userland/Libraries/LibWeb/CSS/StyleProperties.cpp

@@ -194,7 +194,7 @@ void StyleProperties::load_font() const
 RefPtr<Gfx::Font> StyleProperties::font_fallback(bool monospace, bool bold) const
 {
     if (monospace && bold)
-        return Gfx::FontDatabase::default_bold_fixed_width_font();
+        return Gfx::FontDatabase::default_fixed_width_font().bold_variant();
 
     if (monospace)
         return Gfx::FontDatabase::default_fixed_width_font();