GNOME Bugzilla – Bug 746323
The legacyTray should stay below the modalDialogGroup
Last modified: 2015-03-23 19:21:28 UTC
I don't see any reason why it shouldn't as any other ui element.
Sorry, I find that legacyTray and modalDialogGroup are two js file in the source, but I do not understand what do you mean.
If you enter a modal state - such as the select wifi network of the the logout panel - you will see that you can still interact with the legacytray, which is again the modal way. All other elements get covered by a gray actor and the interaction with them is blocked. My understanding is that what is stacked below the modalDialogGroup is blocked, what's above is not, but there might be another way to achieve the desired behaviour.
OK, I understand what is a modal state now, but what is legacytray? If it is that bar in the picture[1]. I do not unserstand how to interact with it in a modal state, I have tried. You should describe how to reproduce it, thank you! [1]: http://img.vim-cn.com/08/50b4164f6f6aeb382d1582c0d128b20cfc3896.png
That's the message tray and has been removed in 3.16. The "legacy tray" is where status icons are now. http://fedoramagazine.org/wp-content/uploads/2015/03/new-notifications-system-tray.png.jpg
Well... I understand it finally now, as what Michele said, I can interact with the legacy tray in a modal state ion 3.15.92. So what do gnome-shell developer think? Should I fix this "bug"?
Sure - the current behavior is clearly not intentional and doesn't make sense.
(In reply to Florian Müllner from comment #6) > Sure - the current behavior is clearly not intentional and doesn't make sense. I have read the code a lot time... and still do not know how to fix it. Could you give any tips please?
Created attachment 299801 [details] [review] legacyTray: Stack tray below modal dialogs System modal dialogs should disable other UI while open, so make sure the legacy tray does not appear on top of them. (In reply to Isaac Ge from comment #7) > Could you give any tips please? Considering it's a one-liner - not really without writing the actual patch :-)
Review of attachment 299801 [details] [review]: OK. I would much prefer an explicit legacyTrayBox in layout.js, but this is fine too.
(In reply to Florian Müllner from comment #8) > Considering it's a one-liner - not really without writing the actual patch > :-) So the problem is solved and I have nothing to do now?
Attachment 299801 [details] pushed as 2ce7a3b - legacyTray: Stack tray below modal dialogs