emvaized / kde-snap-assist

KWin script for KDE Plasma which suggests other window thumbnails on snap
https://store.kde.org/p/1875687
GNU General Public License v3.0
97 stars 6 forks source link

Main window is not transparent #33

Closed vpsone closed 1 year ago

vpsone commented 1 year ago

image

OS info

image

emvaized commented 1 year ago

Do you have KWin effects disabled in your system? As it may result in transparency effects not being applied.

If not, what happens if you try to change the background color and it's opacity from the script settings? It seem to me that you may have your script configs not being saved/restored correctly, as the default background color should not be completely black, but rather a somewhat gray-ish.

vpsone commented 1 year ago

I am not able to change any color. image

vpsone commented 1 year ago

In my kwinrc file, this is present.

[Script-kdesnapassist]
backdropColor=false
borderRadius=8
cardColor=false
hoveredCardColor=false
textColor=false
emvaized commented 1 year ago

Then it must be the same issue as reported before. I have no idea why a boolean value false gets set for the string color values, and never saw it happen on my computer.

Maybe it's some Plasma bug, as the logic behind connecting settings with corresponding stored config gets handled by the system itself under the hood.

emvaized commented 1 year ago

I just added small change in the dev branch, which ensures that the default colors will be used in case the stored values were read as false. It does not solve the issue of these configs being set this way in the first place, but at least will ensure the script remains usable.

vpsone commented 1 year ago

If i delete this part from kwinrc https://github.com/emvaized/kde-snap-assist/issues/33#issuecomment-1229226767 After rebooting, Main view gets transparent. image But if i open setting and try to change anything after saving. it's get black as previously image I also tried your solution https://github.com/emvaized/kde-snap-assist/issues/33#issuecomment-1229283450 but it is not working.

emvaized commented 1 year ago

You mean, after installing the dev version and rebooting the "black background" issue still happens?

emvaized commented 1 year ago

I rewrote this patch now, so now it should be def. working