Re: [PROBLEM] New systray notifications and kwin
On Wednesday 22 October 2008 19:26:22 Sebastian Kügler wrote: Only the ones that open popups automatically. The power management control extender from the battery and the calendar should have focus. (But I think it's fine to make that explicit in the applet itself.) Ok, so we need some function in popup applet which will allow its subclasses to specify desired behavior. Before that I guess we need to find some combination of Qt::WindowFlags or NET::* flags which make popups focusless and don't break extenders at the same time :) Cheers, Dmitry. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [PROBLEM] New systray notifications and kwin
2008/10/22 Dmitry Suzdalev [EMAIL PROTECTED]: Hello! I just found a real problem with new notifications system. Let me describe it. New notifications are shown using a PopupApplet which is a full-blown window from the KWin POV. So what happens is the following: dimsuz is touch-typing with amazing speed in his kmail window. at the same time he often alt-tabs to kopete to chat with friends, and also types much in konversation in #plasma channel :) At the same time a lot of notifications arrive, and guess what - each of them steals focus from dimsuz's current input widget! Because each of them has a default window role. What do you think dimsuz thinks on each notification appearance? Yeah, he thinks: #%#!!!?#?!!! And then clicks back on the widget he is typing into :) I tried to fix this one, but quickly found that systray (being a PopupApplet) just uses PopupApplet::popup() function. That is systray has no control on the dialog that pops up. I think that it should be able to somehow tell the popup() that this particular popup should be of Qt::Tooltip type or something like that. So I'm thinking what's the best way to do this? A parameter to popup() function? A new function PopupApplet::setPopupAsTooltip(bool)? something else? Note, that i think this is a problem that really needs to be fixed, as it may potentially annoy a lot of our users :) Readers digest version for those of you that don't have lotsa free time: Popup notifications should not steal focus, ever. They are the new round of passivePopups and should behave as so. Dan, Cheers, Dmitry. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[PROBLEM] New systray notifications and kwin
Hello! I just found a real problem with new notifications system. Let me describe it. New notifications are shown using a PopupApplet which is a full-blown window from the KWin POV. So what happens is the following: dimsuz is touch-typing with amazing speed in his kmail window. at the same time he often alt-tabs to kopete to chat with friends, and also types much in konversation in #plasma channel :) At the same time a lot of notifications arrive, and guess what - each of them steals focus from dimsuz's current input widget! Because each of them has a default window role. What do you think dimsuz thinks on each notification appearance? Yeah, he thinks: #%#!!!?#?!!! And then clicks back on the widget he is typing into :) I tried to fix this one, but quickly found that systray (being a PopupApplet) just uses PopupApplet::popup() function. That is systray has no control on the dialog that pops up. I think that it should be able to somehow tell the popup() that this particular popup should be of Qt::Tooltip type or something like that. So I'm thinking what's the best way to do this? A parameter to popup() function? A new function PopupApplet::setPopupAsTooltip(bool)? something else? Note, that i think this is a problem that really needs to be fixed, as it may potentially annoy a lot of our users :) Cheers, Dmitry. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [PROBLEM] New systray notifications and kwin
On Wednesday 22 October 2008 15:27:18 Dmitry Suzdalev wrote: Hello! I just found a real problem with new notifications system. Let me describe it. New notifications are shown using a PopupApplet which is a full-blown window from the KWin POV. So what happens is the following: dimsuz is touch-typing with amazing speed in his kmail window. at the same time he often alt-tabs to kopete to chat with friends, and also types much in konversation in #plasma channel :) At the same time a lot of notifications arrive, and guess what - each of them steals focus from dimsuz's current input widget! Because each of them has a default window role. What do you think dimsuz thinks on each notification appearance? Yeah, he thinks: #%#!!!?#?!!! And then clicks back on the widget he is typing into :) I tried to fix this one, but quickly found that systray (being a PopupApplet) just uses PopupApplet::popup() function. That is systray has no control on the dialog that pops up. I think that it should be able to somehow tell the popup() that this particular popup should be of Qt::Tooltip type or something like that. Popup Dialogs used to be Qt::Tooltip, only that doesn't function correctly when you want to be able to drag widgets away from the window, as is possible with extender items. But I agree we should avoid the dialog stealing focus. I assume there is some window flag to accomplish that. I will look into that. So I'm thinking what's the best way to do this? A parameter to popup() function? A new function PopupApplet::setPopupAsTooltip(bool)? something else? I think the don't steal focus behavior should be default for all popupapplets. Note, that i think this is a problem that really needs to be fixed, as it may potentially annoy a lot of our users :) Agreed. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [PROBLEM] New systray notifications and kwin
On Wednesday 22 October 2008, Dmitry Suzdalev wrote: So I'm thinking what's the best way to do this? A parameter to popup() function? A new function PopupApplet::setPopupAsTooltip(bool)? something setPopupWindowFlags(Qt::WindowFlags)? -- Aaron J. Seigo humru othro a kohnu se GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 KDE core developer sponsored by Qt Software signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [PROBLEM] New systray notifications and kwin
On Wednesday 22 October 2008 18:54:28 Rob Scheepmaker wrote: Popup Dialogs used to be Qt::Tooltip, only that doesn't function correctly when you want to be able to drag widgets away from the window, as is possible with extender items. But I agree we should avoid the dialog stealing focus. I assume there is some window flag to accomplish that. I will look into that. That would be very nice, thanks! Perhaps use some NET:: flags would help? I think the don't steal focus behavior should be default for all popupapplets. Ah, this makes thing easier (no api modifications required) Cheers, Dmitry. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [PROBLEM] New systray notifications and kwin
On Wednesday 22 October 2008 16:54:28 Rob Scheepmaker wrote: So I'm thinking what's the best way to do this? A parameter to popup() function? A new function PopupApplet::setPopupAsTooltip(bool)? something else? I think the don't steal focus behavior should be default for all popupapplets. Only the ones that open popups automatically. The power management control extender from the battery and the calendar should have focus. (But I think it's fine to make that explicit in the applet itself.) -- sebas http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9 signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [PROBLEM] New systray notifications and kwin
On Wednesday 22 October 2008 19:13:01 Aaron J. Seigo wrote: On Wednesday 22 October 2008, Rob Scheepmaker wrote: So I'm thinking what's the best way to do this? A parameter to popup() function? A new function PopupApplet::setPopupAsTooltip(bool)? something else? I think the don't steal focus behavior should be default for all popupapplets. .. that popup automatically. for popups that are trigerred by user interaction, this could get annoying in all new ways. Ok, but workaround against wrong behaviour of Qt::Popup that Rob talked about still has to be found. That brings us to the question of introducing setPopupWindowFlags(Qt::WindowFlags) - if user will set Qt::Popup, he'll break extenders stuff... Cheers, Dmitry. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [PROBLEM] New systray notifications and kwin
On Wednesday 22 October 2008, Rob Scheepmaker wrote: Popup Dialogs used to be Qt::Tooltip, only that doesn't function correctly when you want to be able to drag widgets away from the window, as is possible with extender items. what problems does this cause, exactly? -- Aaron J. Seigo humru othro a kohnu se GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 KDE core developer sponsored by Qt Software signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [PROBLEM] New systray notifications and kwin
On Wednesday 22 October 2008 18:36:57 Aaron J. Seigo wrote: On Wednesday 22 October 2008, Rob Scheepmaker wrote: Popup Dialogs used to be Qt::Tooltip, only that doesn't function correctly when you want to be able to drag widgets away from the window, as is possible with extender items. what problems does this cause, exactly? IIRC, when dragging an item away, the dialog gets hidden when you move an item. Isn't one of the NET flag usable to avoid the dialog getting focus? Hmm, I'm starting to doubt if it wasn't Qt::Popup before, instead of Qt::Tooltip. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel