| Summary: | On X11, Plasma dialogs (applet popups, OSDs, notifications) should not be maximizable, or minimizable, or tilable | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] libplasma | Reporter: | Michail Vourlakos <mvourlakos> |
| Component: | libplasma | Assignee: | Marco Martin <notmart> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | andresbs2000, arthur, bugseforuns, cknauth119, fegul, jackyguo18, joshuag1000, katyaberezyaka, kde, kdebugs.wxdrr, leftcrane, maksatlego, mariusz.libera, mattyhoffman7, me, me, nate, ngompa13, o.malek.cz, plasma-bugs, poperigby, popov895, postix, rhiss, trmdi, yerrey |
| Priority: | VHI | ||
| Version: | 5.61.0 | ||
| Target Milestone: | --- | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| See Also: | https://bugs.kde.org/show_bug.cgi?id=445663 | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/commit/23c4e82cdcb6c7f251c27c6eefa643415c8c5927 | Version Fixed In: | 5.26 |
| Sentry Crash Report: | |||
|
Description
Michail Vourlakos
2019-08-31 06:56:44 UTC
*** Bug 411437 has been marked as a duplicate of this bug. *** *** Bug 439546 has been marked as a duplicate of this bug. *** *** Bug 441035 has been marked as a duplicate of this bug. *** *** Bug 416291 has been marked as a duplicate of this bug. *** *** Bug 419764 has been marked as a duplicate of this bug. *** *** Bug 426941 has been marked as a duplicate of this bug. *** *** Bug 443077 has been marked as a duplicate of this bug. *** *** Bug 444477 has been marked as a duplicate of this bug. *** *** Bug 444448 has been marked as a duplicate of this bug. *** *** Bug 445221 has been marked as a duplicate of this bug. *** Can't reproduce on Wayland; seems to be X11-only *** Bug 445854 has been marked as a duplicate of this bug. *** *** Bug 445663 has been marked as a duplicate of this bug. *** Raising priority due to number of duplicates. While I may not understand the severity of this problem, my initial thought is that these type of items should be at least allowed to be moved, resized, etc. When a display problem occurs such as a new install and something gets stuck in horrible resolution, being able to move or resize a popup/notification or whatever is in the way can be critical. So, while no one wants some window maximizing on a generic click, removing the ability altogether, I would advise against. Again, apologize if I misunderstand the context. (In reply to Jeff Tratt from comment #15) > While I may not understand the severity of this problem, my initial thought > is that these type of items should be at least allowed to be moved, resized, > etc. When a display problem occurs such as a new install and something gets > stuck in horrible resolution, being able to move or resize a > popup/notification or whatever is in the way can be critical. So, while no > one wants some window maximizing on a generic click, removing the ability > altogether, I would advise against. Again, apologize if I misunderstand the > context. I think that makes sense as well, the main thing is that the ability to adjust is either consistent or non-existent (and the ability to reset it to default should be somewhere as well if it's possible to adjust it) *** Bug 446379 has been marked as a duplicate of this bug. *** *** Bug 447274 has been marked as a duplicate of this bug. *** Bug already fixed in the Plasma Wayland session, so this only affects X11. *** Bug 385231 has been marked as a duplicate of this bug. *** *** Bug 453393 has been marked as a duplicate of this bug. *** Git commit 7892a18c26cad6881a1cddc5b1b722c8c5533049 by Nate Graham, on behalf of Luca Carlon. Committed on 27/05/2022 at 17:11. Pushed by ngraham into branch 'master'. Add a new window type named AppletPopup. See discussion in https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/500. Related: bug 332512 M +2 -0 autotests/kwindowinfox11test.cpp M +1 -1 autotests/netrootinfotestwm.cpp M +1 -0 autotests/netwininfotestclient.cpp M +5 -0 src/netwm_def.h M +1 -0 src/platforms/xcb/atoms_p.h M +16 -0 src/platforms/xcb/netwm.cpp https://invent.kde.org/frameworks/kwindowsystem/commit/7892a18c26cad6881a1cddc5b1b722c8c5533049 Git commit 590cc34c684b0c38ee895472a0e20f9787d5089d by Nate Graham, on behalf of Luca Carlon. Committed on 27/05/2022 at 17:12. Pushed by ngraham into branch 'master'. Add AppletPopup window type to PlasmaShellSurface See discussion in https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/500. Related: bug 332512 M +1 -0 autotests/client/test_plasmashell.cpp M +3 -0 src/client/plasmashell.cpp M +1 -0 src/client/plasmashell.h M +4 -1 src/server/plasmashell_interface.cpp M +1 -0 src/server/plasmashell_interface.h M +4 -0 tests/plasmasurfacetest.cpp https://invent.kde.org/frameworks/kwayland/commit/590cc34c684b0c38ee895472a0e20f9787d5089d Git commit da0dd1e367b638f7d7803f1eccfb8a36d54c3a43 by Nate Graham, on behalf of Luca Carlon. Committed on 27/05/2022 at 21:49. Pushed by ngraham into branch 'master'. Support new AppletPopup window type See the discussion in https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/500. Related: bug 332512 FIXED-IN: 5.25 M +2 -0 autotests/integration/internal_window.cpp M +2 -0 autotests/integration/plasma_surface_test.cpp M +2 -0 src/debug_console.cpp M +1 -0 src/effects.cpp M +1 -0 src/effects.h M +2 -1 src/effects/fadingpopups/package/contents/code/main.js M +4 -0 src/libkwineffects/kwineffects.h M +1 -0 src/wayland/autotests/client/test_plasmashell.cpp M +3 -0 src/wayland/plasmashell_interface.cpp M +1 -0 src/wayland/plasmashell_interface.h M +4 -0 src/wayland/tests/plasmasurfacetest.cpp M +5 -2 src/window.cpp M +11 -0 src/window.h M +1 -1 src/workspace.cpp M +12 -1 src/x11window.cpp M +10 -2 src/xdgshellwindow.cpp https://invent.kde.org/plasma/kwin/commit/da0dd1e367b638f7d7803f1eccfb8a36d54c3a43 Git commit ace8a4a3515de002b2605fbe382b825394fad28b by Nate Graham, on behalf of Luca Carlon. Committed on 27/05/2022 at 21:53. Pushed by ngraham into branch 'master'. Make widget popups resizable See the discussion in https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/500. Related: bug 332512 M +6 -2 applets/analog-clock/contents/ui/analogclock.qml M +6 -2 applets/batterymonitor/package/contents/ui/main.qml M +5 -2 applets/calendar/package/contents/ui/main.qml M +5 -2 applets/clipboard/contents/ui/clipboard.qml M +3 -0 applets/devicenotifier/package/contents/ui/FullRepresentation.qml M +6 -9 applets/digital-clock/package/contents/ui/CalendarView.qml M +1 -1 applets/kicker/plugin/submenu.cpp M +5 -2 applets/mediacontroller/contents/ui/ExpandedRepresentation.qml M +8 -3 applets/notifications/package/contents/ui/FullRepresentation.qml M +7 -13 applets/systemtray/package/contents/ui/ExpandedRepresentation.qml M +3 -0 applets/systemtray/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/commit/ace8a4a3515de002b2605fbe382b825394fad28b Git commit 23c4e82cdcb6c7f251c27c6eefa643415c8c5927 by Nate Graham, on behalf of Luca Carlon. Committed on 27/05/2022 at 21:53. Pushed by ngraham into branch 'master'. Make widget popups resizable See the discussion in https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/500. Related: bug 332512 M +4 -0 applets/kickoff/package/contents/ui/FullRepresentation.qml M +6 -0 desktoppackage/contents/applet/CompactApplet.qml https://invent.kde.org/plasma/plasma-desktop/commit/23c4e82cdcb6c7f251c27c6eefa643415c8c5927 *** Bug 455310 has been marked as a duplicate of this bug. *** *** Bug 455797 has been marked as a duplicate of this bug. *** Can reproduce with Kickoff. SOFTWARE/OS VERSIONS Operating System: Fedora Linux 36 KDE Plasma Version: 5.25.4 KDE Frameworks Version: 5.97.0 Qt Version: 5.15.5 Graphics Platform: X11 This seems to be already fixed in master and can't reproduce. KDE Plasma 5.25.80 KDE Frameworks 5.97.0 Yep, as the "version fixed in" field says, this is fixed in 5.26. How about stable? Plasma 5.26 will only be released in two months. It wasn't possible to backport the change to stable as it was a huge thing that depended on frameworks changes too. There is no need to backport from the master, just need to fix it in the stable. *** Bug 459918 has been marked as a duplicate of this bug. *** *** Bug 460296 has been marked as a duplicate of this bug. *** |