https://bugs.kde.org/show_bug.cgi?id=363804

--- Comment #9 from Thomas Lübking <thomas.luebk...@gmail.com> ---
I'd also frankly object the entire idea of the patch.
The typical bug would be that an inner client struts ie. +1920+432+32x768 sets
a left strut of 1952, isn't?

In that case you'd likely want to perform a spatial sanitation, ie. ignore the
strut for that area because it spans across the entire screen in strut
direction, not because it leaves "some" intersection (for a "proper" partial
strut broken this particular way usually will)

Also I'd remove the check from the ::adjustedClientArea() function and move it
to ::updateClientArea() where it belongs as the actual strutting is done there.
Yes, that defeats the present tests, but they've proven to be worthless and the
new magic condition in ::adjustedClientArea() is just a magic condition - the
approach damages code to support tests.
In doubt one could add a test for ::updateClientArea()

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to