LibGUI: Fix 1px misalignment of ColorButton selections in ColorPicker

Also use shrunken() to calculate the selection rects.
lanmonster · Oct 2, 2020 · 18c67b4 · 18c67b4
1 parent a2ffe95
commit 18c67b4
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/Libraries/LibGUI/ColorPicker.cpp b/Libraries/LibGUI/ColorPicker.cpp
@@ -443,12 +443,12 @@ void ColorButton::paint_event(PaintEvent& event)
 
  Gfx::StylePainter::paint_button(painter, rect(), palette(), Gfx::ButtonStyle::Normal, is_being_pressed(), is_hovered(), is_checked(), is_enabled());
 
- painter.fill_rect({ 1, 1, rect().width() - 2, rect().height() - 2 }, m_color);
+ painter.fill_rect(rect().shrunken(2, 2), m_color);
 
  if (m_selected) {
- painter.fill_rect({ 3, 3, rect().width() - 6, rect().height() - 6 }, Color::Black);
- painter.fill_rect({ 5, 5, rect().width() - 10, rect().height() - 10 }, Color::White);
- painter.fill_rect({ 7, 6, rect().width() - 14, rect().height() - 14 }, m_color);
+ painter.fill_rect(rect().shrunken(6, 6), Color::Black);
+ painter.fill_rect(rect().shrunken(10, 10), Color::White);
+ painter.fill_rect(rect().shrunken(14, 14), m_color);
  }
 }