[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2019-07-14 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=395262

Nate Graham  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |DUPLICATE

--- Comment #35 from Nate Graham  ---


*** This bug has been marked as a duplicate of bug 399680 ***

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2019-07-13 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=395262

tr...@yandex.com changed:

   What|Removed |Added

 CC||tr...@yandex.com
 Resolution|FIXED   |---
 Status|RESOLVED|REOPENED

--- Comment #34 from tr...@yandex.com ---
The issue is still not resolved completely.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-10-21 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #33 from Patrick Silva  ---
another related report: bug 400092

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-10-20 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #32 from Patrick Silva  ---
bug 399680 is related

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-10-05 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #31 from Alex Nemeth  ---
Fixing them individually is actually a workaround of the real problem.
I introduced the transparent menu feature. I'm sorry but I have no idea how to
really fix this. 
I looked at Kvantum and QSvgStyle where this actually works, but can't figure
out what their devs did to achieve this. 
I think currently our best option may be to fix this as workaround in the most
obvious places (like we did it with the desktop context menu).

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-10-05 Thread Ievgen Sobko
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #30 from Ievgen Sobko  ---
I created new issue but Nate marked it as duplicate of this and closed. Maybe
it is better to reopen this one.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-10-04 Thread Christoph Feck
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #29 from Christoph Feck  ---
I would suggest to file individual reports for the cases where it (still) does
not work correctly.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-19 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=395262

Nate Graham  changed:

   What|Removed |Added

 CC||ievgenso...@gmail.com

--- Comment #28 from Nate Graham  ---
*** Bug 398827 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-14 Thread Dr . Chapatin
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #27 from Dr. Chapatin  ---
Under Wayland the bug occurs with the context menu of window decoration. Both
neon dev unstable and Arch Linux (plasma 5.14 beta) are affected.

Under X11 the bug occurs with the context menu of plasma notification. For
example, press printscreen to open spectacle and save the screenshot. Right
click the notification and you can see the bug in the context menu.

Can we reopen this report or we need new reports?

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-14 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=395262

Kai Uwe Broulik  changed:

   What|Removed |Added

   Version Fixed In|5.14|

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-14 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=395262

Kai Uwe Broulik  changed:

   What|Removed |Added

 CC||k...@privat.broulik.de
   Version Fixed In||5.14

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-14 Thread Dr . Chapatin
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #26 from Dr. Chapatin  ---
(In reply to umer from comment #25)
> "drag and drop" copy/paste menu also have this broken blur effect in kde
> neon 5.13.5

Same thing in plasma 5.14 beta, Arch Linux.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-14 Thread umer
https://bugs.kde.org/show_bug.cgi?id=395262

umer  changed:

   What|Removed |Added

 CC||duzasa...@zep-hyr.com

--- Comment #25 from umer  ---
"drag and drop" copy/paste menu also have this broken blur effect in kde neon
5.13.5

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-12 Thread Dr . Chapatin
https://bugs.kde.org/show_bug.cgi?id=395262

Dr. Chapatin  changed:

   What|Removed |Added

 CC||iamsainisac...@gmail.com

--- Comment #24 from Dr. Chapatin  ---
*** Bug 398413 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-11 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #23 from Alex Nemeth  ---
(In reply to Alex Nemeth from comment #22)
> I also still have the problem.
> Here is an attempt to fix it for the desktop icon context menu: D15435

Full link: https://phabricator.kde.org/D15435

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-11 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #22 from Alex Nemeth  ---
I also still have the problem.
Here is an attempt to fix it for the desktop icon context menu: D15435

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-11 Thread Dr . Chapatin
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #21 from Dr. Chapatin  ---
Created attachment 114898
  --> https://bugs.kde.org/attachment.cgi?id=114898=edit
screencast recorded on neon dev unstable

Does anyone else confirm that the problem persists?

My Arch and Neon are running on different computers.
Arch computer has intel hd graphics, neon computer has intel hd 4000.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-10 Thread mthw0
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #20 from mthw0  ---
(In reply to Dr. Chapatin from comment #19)
> Where is the fix? Bug persists on both Arch Linux (plasma 5.13.5) and neon
> dev unstable under X11.

AFAIK The fix landed a while ago, and works just fine. 
The bug only persists in VLC. Arch Linux + Plasma 5.13.5 intel GPU

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-09-10 Thread Dr . Chapatin
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #19 from Dr. Chapatin  ---
Where is the fix? Bug persists on both Arch Linux (plasma 5.13.5) and neon dev
unstable under X11.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-17 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #18 from Alex Nemeth  ---
(In reply to David Edmundson from comment #17)
> Huzzah, I had some stray code setting that when trying different things :/
> 
> I'll have a look at whether we can do it in Breeze. I'm not convinced.

it works in QtCurve and Kvantum too, so we should be able to do it in Breeze
too, right?

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-17 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #17 from David Edmundson  ---
Huzzah, I had some stray code setting that when trying different things :/

I'll have a look at whether we can do it in Breeze. I'm not convinced.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-17 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #16 from Alex Nemeth  ---
Created attachment 113992
  --> https://bugs.kde.org/attachment.cgi?id=113992=edit
video of half-working

Adding this solves it for desktop icon menu:
menu->setAttribute(Qt::WA_TranslucentBackground);
However this is not the only case of using QMenu::popup() (see video)
Some 3rd party apps use it too.
Can this be done in Breeze theme itself?
Is there any technical possibility that doesn't allow it?

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-17 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #15 from David Edmundson  ---
Heh.
Does this work: 
menu->setAttribute(Qt::WA_TranslucentBackground);

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-17 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #14 from Alex Nemeth  ---
Created attachment 113989
  --> https://bugs.kde.org/attachment.cgi?id=113989=edit
desktop icon menu

It's very cool that you found a solution. Thank you for helping me out. I would
have never guessed this.
However even with the patch applied I still have the glitch.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-17 Thread mthw0
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #13 from mthw0  ---
(In reply to David Edmundson from comment #12)
> Git commit 42a74c9a79baafb222513f3ffad35b736b708454 by David Edmundson.
> Committed on 17/07/2018 at 09:34.
> Pushed by davidedmundson into branch 'master'.
> 
> Fix blur behind folderview context menus
> 
> Summary:
> Breeze has
> 
> if (!(widget->testAttribute(Qt::WA_WState_Created) ||
> widget->internalWinId()))
>KWindowSystem::blur(widget->winId(), ...)
> 
> which makes sense as the style should not be creating surfaces in polish
> and I don't want to remove it.
> 
> This is problematic with QMenu::polish which has a subtle behavioural
> change compared to QMenu::exec in that it calls ensurePolish before
> creating the surface.
> 
> Plasma::ContainmentInterface accidentally has this workaround already
> for parent window setting.
> 
> Test Plan:
> Right clicked on desktop icon
> Looked amazing
> 
> Reviewers: #plasma, hein
> 
> Reviewed By: #plasma, hein
> 
> Subscribers: hein, plasma-devel
> 
> Tags: #plasma
> 
> Differential Revision: https://phabricator.kde.org/D14174
> 
> M  +1-0containments/desktop/plugins/folder/foldermodel.cpp
> 
> https://commits.kde.org/plasma-desktop/
> 42a74c9a79baafb222513f3ffad35b736b708454

Will this change make it into Plasma 5.13.4? Or we will have to wait for 5.14?

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-17 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=395262

David Edmundson  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/pla
   ||sma-desktop/42a74c9a79baafb
   ||222513f3ffad35b736b708454
 Status|CONFIRMED   |RESOLVED
 Resolution|--- |FIXED

--- Comment #12 from David Edmundson  ---
Git commit 42a74c9a79baafb222513f3ffad35b736b708454 by David Edmundson.
Committed on 17/07/2018 at 09:34.
Pushed by davidedmundson into branch 'master'.

Fix blur behind folderview context menus

Summary:
Breeze has

if (!(widget->testAttribute(Qt::WA_WState_Created) ||
widget->internalWinId()))
   KWindowSystem::blur(widget->winId(), ...)

which makes sense as the style should not be creating surfaces in polish
and I don't want to remove it.

This is problematic with QMenu::polish which has a subtle behavioural
change compared to QMenu::exec in that it calls ensurePolish before
creating the surface.

Plasma::ContainmentInterface accidentally has this workaround already
for parent window setting.

Test Plan:
Right clicked on desktop icon
Looked amazing

Reviewers: #plasma, hein

Reviewed By: #plasma, hein

Subscribers: hein, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D14174

M  +1-0containments/desktop/plugins/folder/foldermodel.cpp

https://commits.kde.org/plasma-desktop/42a74c9a79baafb222513f3ffad35b736b708454

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-16 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #11 from David Edmundson  ---
@Alex please see https://phabricator.kde.org/D14174 and let me know if we need
this in other places

I don't know if it makes sense to fix Qt. QMenu is a horrible piece of code to
get changes into as it has so many platform specific behavioural things.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-16 Thread Dr . Chapatin
https://bugs.kde.org/show_bug.cgi?id=395262

Dr. Chapatin  changed:

   What|Removed |Added

 CC||william...@me.com

--- Comment #10 from Dr. Chapatin  ---
*** Bug 396540 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-07-06 Thread mthw0
https://bugs.kde.org/show_bug.cgi?id=395262

mthw0  changed:

   What|Removed |Added

 CC||jari...@hotmail.com

--- Comment #9 from mthw0  ---
This also happens on ringt-click-menu in vlc.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-22 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #8 from Alex Nemeth  ---
(In reply to David Edmundson from comment #7)
> But there shouldn't be a difference from a kwin side - so we need to figure
> out what's different from an X POV when the different method is called.

Kvantum and QtCurve themes that support transparent context menus work without
any problem, so this must be a Breeze issue then. I looked at their source code
but didn't find anything regarding handling these two types differently.

In breezestyle.cpp the correct code runs, regardless of popup() or exec() call,
although for popup() it gets called 3 or 4 times(??).

I'll investigate this further...

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-22 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=395262

David Edmundson  changed:

   What|Removed |Added

 CC||k...@davidedmundson.co.uk

--- Comment #7 from David Edmundson  ---
>But then changing every instance of popup() to exec() would be a big task.

This isn't an option for plasmashell.

But there shouldn't be a difference from a kwin side - so we need to figure out
what's different from an X POV when the different method is called.

Historically there was a difference with when QWidget::polish was called, which
affects when QStyle code is triggered, we used to have an issue with oxygen's
rounded corners.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-22 Thread Marek M
https://bugs.kde.org/show_bug.cgi?id=395262

Marek M  changed:

   What|Removed |Added

 CC||login_...@o2.pl

--- Comment #6 from Marek M  ---
At second screen works very well. But after use many days without restart, blur
effect start work also at first screen.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-21 Thread Dr . Chapatin
https://bugs.kde.org/show_bug.cgi?id=395262

Dr. Chapatin  changed:

   What|Removed |Added

 CC||bugsefor...@gmx.com

--- Comment #5 from Dr. Chapatin  ---
I can confirm the problem on Arch Linux running plasma 5.13.1 under X11.
Blur effect works correctly under Wayland.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-20 Thread Michael D
https://bugs.kde.org/show_bug.cgi?id=395262

Michael D  changed:

   What|Removed |Added

 CC||nortex...@gmail.com

--- Comment #4 from Michael D  ---
This bug also affects Dolphin's drag and drop context menu. If you drag and
drop an item, the context menu that pops up asking to move, copy, etc. exhibits
the blur bug. I'm on Manjaro testing, plasma 5.13.1.

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-14 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

--- Comment #3 from Alex Nemeth  ---
So after investigating I believe this visual bug happens when a QMenu is called
with popup() instead of exec()
Maybe this is a Qt bug?

In this file if I change menu->popup() to menu->exec() then there is no visual
bug, but after I close the menu plasmashell crashes and I don't know how to
prevent it from crashing.
https://github.com/KDE/plasma-desktop/blob/master/containments/desktop/plugins/folder/foldermodel.cpp#L1776

But then changing every instance of popup() to exec() would be a big task.

Can anyone with more experience please help me with this?

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-12 Thread Alex Nemeth
https://bugs.kde.org/show_bug.cgi?id=395262

Alex Nemeth  changed:

   What|Removed |Added

 Status|ASSIGNED|CONFIRMED

--- Comment #2 from Alex Nemeth  ---
I can confirm it.
Looking into it...

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-12 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=395262

Nate Graham  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |ASSIGNED

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-12 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=395262

Nate Graham  changed:

   What|Removed |Added

   Assignee|se...@kde.org   |alex19930...@gmail.com

--- Comment #1 from Nate Graham  ---
Alex, would you mind having a look here?

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

[plasmashell] [Bug 395262] Transparency/blur is corrupted on context menu of desktop files

2018-06-12 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=395262

Nate Graham  changed:

   What|Removed |Added

 CC||alex19930...@gmail.com,
   ||n...@kde.org

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