beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 23:50:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibGUI: Prevent a Painter's clip_rect from being outside of its target

Browse source 
Previously a Painter's m_clip_origin field was initialized to a
widget's window_relative_rect, which is not ensured to be within
the target rect.
m_clip_origin is normally not used for clipping, but after calling
clear_clip_rect the clip rect that IS used for clipping gets reset
to m_clip_origin (so an invalid state is entered).
Now the window_relative_rect will be clipped by the target rect
first, and will only then be used to initialize both the active
clip_rect and m_clip_origin.
This commit is contained in:
Mart G 2021-04-24 15:02:25 +02:00 committed by Linus Groh
parent 0754280214
commit 157cd7c819
Notes: sideshowbarker 2024-07-18 19:09:37 +09:00 
Author: https://github.com/Maato
Commit: https://github.com/SerenityOS/serenity/commit/157cd7c8196
Pull-request: https://github.com/SerenityOS/serenity/pull/6612
1 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
Userland/Libraries/LibGUI/Painter.cpp
View file

@ -21,9 +21,8 @@ Painter::Painter(Widget& widget)
state().font = &widget.font();
auto origin_rect = widget.window_relative_rect();
state().translation = origin_rect.location();
state().clip_rect = origin_rect;
m_clip_origin = origin_rect;
state().clip_rect.intersect(m_target->rect());
state().clip_rect = origin_rect.intersected(m_target->rect());
m_clip_origin = state().clip_rect;
}
}