-
Notifications
You must be signed in to change notification settings - Fork 220
[19277] Ensure the effective rect is fetched correctly when screen is locked #6815
[19277] Ensure the effective rect is fetched correctly when screen is locked #6815
Conversation
engine/src/desktop-stack.cpp
Outdated
{ | ||
MCRectangle t_content_rect, t_diff_rect; | ||
MCscreen->platform_getwindowgeometry(window, t_content_rect); | ||
t_diff_rect = MCU_subtract_rect(t_content_rect, t_frame_rect); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think MCU_subtract_rect is correc there - t_content_rect here is the window's current rect, and t_frame_rect is the window's current effective rect. The frame rect will contain the content rect, which gives you four margins - one for top, left, right and bottom. You then want to apply these margins to the rect of the stack - e.g. left of effective rect = rect.left + (t_content_rect.x - t_frame_rect.x).
@livecodepanos this looks testable when running in ui mode to me so I'll add the needs tests label |
@livecode-vulcan review ok e90667f |
💙 review by @montegoulding ok e90667f |
[19277] Ensure the effective rect is fetched correctly when screen is locked
😞 test failure e90667f
|
@livecode-vulcan review ok 0011c72 |
💙 review by @montegoulding ok 0011c72 |
[19277] Ensure the effective rect is fetched correctly when screen is locked
😎 test success 0011c72
|
No description provided.