Summary: | PanelView::setBackgroundHints doesn't update enabledBorders (which means it doesn't toggle the shadows) | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Chris Holland <zrenfire> |
Component: | general | Assignee: | David Edmundson <kde> |
Status: | NEEDSINFO WAITINGFORINFO | ||
Severity: | minor | CC: | kde, nate, niccolo, plasma-bugs-null |
Priority: | NOR | ||
Version First Reported In: | 5.10.5 | ||
Target Milestone: | 1.0 | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: |
2017 Panel.qml with blur
Video alternating between backgroundHint on/off every 2sec |
Description
Chris Holland
2017-09-06 04:03:46 UTC
There's also something wrong going on in PanelShadows. Even if I fix the issue you mentioned, the shadow doesn't re-appear after it's gone. It never calls addShadow again after initialization but it does removeShadow when background hints become "NoBackground", so updateShadows won't do anything. I then fixed that but it still wouldn't help :/ May I ask for more information about what's happening here? Why are you trying to disable shadows from Panel.qml? What's the user-facing issue here? Created attachment 185201 [details]
2017 Panel.qml with blur
Created attachment 185202 [details]
Video alternating between backgroundHint on/off every 2sec
After looking up the video in my archives since the streamable video is deleted, it looks like I was trying to make a fully transparent panel without shadows when windows weren't nearby, then apply the black panel with shadows when the windows were touching the panel. Eg: This effect from a gnome extension https://www.youtube.com/watch?v=P5Eqf65uVhs IIRC somebody already implemented "adaptive" transparency panels so this bug has probably already been fixed? Or maybe not. It looks like the panel is still drawing shadows in this "KDE Plasma 5.22: Adaptive Transparency" youtube video when it's in "transparent mode" since it's only "transluscent" not fully transparent. https://www.youtube.com/watch?v=4exHfb8Qjl0 Looks like I'll need to try making an adaptive plasma style sometime to figure out if the bug still exists. |