Bug 338946

Summary: Notifications and system tray popup interfere
Product: [Plasma] plasmashell Reporter: Kai Uwe Broulik <kde>
Component: System TrayAssignee: Sebastian Kügler <sebas>
Status: RESOLVED FIXED    
Severity: normal CC: enoopt.adams, eugene.shalygin+bugzilla.kde, kde, mklapetek, neptunecat34, thomas.pfeiffer
Priority: NOR Flags: thomas.pfeiffer: Usability+
kde: VisualDesign+
Version: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.2.1
Sentry Crash Report:
Attachments: Video illustrating the issue

Description Kai Uwe Broulik 2014-09-09 14:02:13 UTC
When turning off WiFi in the network plasmoid, a notification saying the connection is lost, appears. Clicking on the close button will close the systray popup, you have to click the X a second time to actually close it.

Reproducible: Always

Steps to Reproduce:
1. Open Plasma NM
2. Turn off WiFi
3. Click on the resulting notification's close button

Actual Results:  
System tray popup closes

Expected Results:  
Notification closes
Comment 1 Kai Uwe Broulik 2014-09-09 14:13:53 UTC
Created attachment 88631 [details]
Video illustrating the issue
Comment 2 David Edmundson 2014-09-23 01:18:01 UTC
..and what do you think should happen?

Tagging VDG.
Comment 3 Martin Klapetek 2014-09-25 22:37:04 UTC
> ..and what do you think should happen?

Given the close button on the notification popup has focus (from hover), it should close only the notification popup or at least close it first.

The systray popup closes immediately after it looses focus, so clicking the notification popup actually makes the systray popup loose focus, then it animates any maaaybee kwin puts it over the notification, so your click actually ends up in the systray popup (wild guessing here however).
Comment 4 Thomas Pfeiffer 2014-09-30 15:13:08 UTC
This is just a symptom of a more general problem: That notifications appear at the same place as the systray popup.

We could solve this either by positioning the notifications elsewhere. Personally I'd like to put them on the top-right corner by default, but allow users to move them wherever they want them. If the user clicks the notifications icon in the systray, however, they should of course still be shown in the sytray popup.
Comment 5 Kai Uwe Broulik 2014-09-30 15:43:13 UTC
What about moving the popups out of the way when the systray popup is opened? Ie. they move up by whatever height the systray popups size is? Don't know if/how that is possible.
Comment 6 Philipp Stefan 2014-12-29 12:39:16 UTC
Since this bug has its age I opened a thread in the forum to discuss this matter:

https://forum.kde.org/viewtopic.php?f=285&t=124249
Comment 7 Martin Klapetek 2014-12-29 13:26:11 UTC
Nooo, please let's keep the discussion at one place only, having to jump back and forth is hard.

Basically there's not much to discuss, this report is confirmed, we would like to move the popups, but it's not as easy and may require changes in more components.
Comment 8 Philipp Stefan 2014-12-29 14:09:53 UTC
To copy my response in the forum:

I am sorry, I misunderstood the meaning of "confirmed". For me "confirmed" meant "yes it's a problem", not "yes, we want implement X".

We have decided that we would like to discuss VDG matters here [the forum] in future if someone wants to know the VDG's stance on an issue. The bugtracker is not the best place to have such discussions.
Comment 9 Thomas Pfeiffer 2014-12-29 14:15:26 UTC
Just to make sure there is no misunderstanding: We don't plan to spread discussions out over several places. 
If there is a clear question with a clear answer, or technical questions have to be discussed, we're happy to do that here on BKO.
However, if there is an open purely design-related question where we want to brainstorm with everyone int he wider VDG community, we want to allow people to do that in the environment they are used to, which is the forum.
Then if we either come up with a proposed design solution or we have further technical questions, we take it back here.
Comment 10 Martin Klapetek 2015-01-12 15:09:48 UTC
*** Bug 342180 has been marked as a duplicate of this bug. ***
Comment 11 Kai Uwe Broulik 2015-02-02 18:31:12 UTC
Git commit 522a76ea2fc00fa6b6271e2dd77b0419a16e52bc by Kai Uwe Broulik.
Committed on 02/02/2015 at 18:28.
Pushed by broulik into branch 'Plasma/5.2'.

Don't let notification popup accept focus

This fixes for example the system tray popup closing when dismissing a notification.

REVIEW: 122342
FIXED-IN: 5.2.1

M  +1    -0    applets/notifications/package/contents/ui/NotificationPopup.qml

http://commits.kde.org/plasma-workspace/522a76ea2fc00fa6b6271e2dd77b0419a16e52bc