mirrors/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-01-25 18:52:22 -05:00
Code Issues Projects Releases Packages Wiki Activity

LibGUI: Properly draw the background of the selected TreeView line

Browse source 
In the TreeView, the background of the selected line (or any background,
really) was only drawn until the frame's width. When the text was larger
than the frame's width, this caused the end of the text to be displayed
without background, making it unreadable if it was white (which is the
default text color for selected lines).

To compute the background width, we have a choice between :
- The inner frame width (the current behaviour which causes the issue)
- The total width of all columns (which causes the background to end
  early if the columns don't cover the full width)

The new algorithm uses the biggest of the above values, which gives us
exactly what we want in all cases :^)

Fixes #2134
This commit is contained in:
DexesTTP 2020-05-07 16:42:34 +02:00 • committed by Andreas Kling
parent 8d01fd9863
commit b478c5f86c
Notes: sideshowbarker 2024-07-19 06:54:31 +09:00
1 changed files with 11 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
Libraries/LibGUI/TreeView.cpp
View file

@ -224,7 +224,17 @@ void TreeView::paint_event(PaintEvent& event)
}
}
Gfx::Rect row_rect { 0, rect.y(), frame_inner_rect().width(), rect.height() };
int row_width = 0;
for (int column_index = 0; column_index < model.column_count(); ++column_index) {
if (is_column_hidden(column_index))
continue;
row_width += this->column_width(column_index) + horizontal_padding() * 2;
}
if (frame_inner_rect().width() > row_width) {
row_width = frame_inner_rect().width();
}
Gfx::Rect row_rect { 0, rect.y(), row_width, rect.height() };
painter.fill_rect(row_rect, background_color);
int x_offset = 0;