[Konsole-devel] [konsole] [Bug 198175] Konsole should set blur region for the new kwin effect

[Martin Sandsmark]

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

--- Comment #50 from Martin Sandsmark <martin.sandsmark at kde.org> ---
(In reply to comment #46)
> 1. blurring can be unwanted depending on the application that carries the
> konsolepart (though the scheme might be sufficient here)

Which is why it is optional, off by default, etc., as Eike explained.

> 2. the client needs to aggregate various blurring requests, denies into one
> region - the plasma function does not provide that since it was probably
> only meant to blur docks and panels.

This is not possible with the current API, and as I said, kdelibs is frozen so
we can't fix this. The idea I had was to remove the existing "enableBlurBehind"
API and add two functions; "addBlurBehind" and "removeBlurBehind" which adds to
a static list of regions that are unified before being put into the property.

In the meantime, I believe this gives us a good compromise, especially looking
at which apps actually use the konsole kpart.

> => I assume "mask()" in the patch refers to an atually usable region and not just QWidget::mask() - which is usually just an empty region and will not do anything usable in the plasma implementation?

No, I was being stupid. :-P

I changed it to using rect() locally, but KWin still doesn't update the blurred
region locally.

-- 
You are receiving this mail because:
You are the assignee for the bug.