Bug 355679 - Add "Display Configuration" to Desktop desktop context menu
Summary: Add "Display Configuration" to Desktop desktop context menu
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Desktop Containment (show other bugs)
Version: 5.4.3
Platform: Other Linux
: NOR wishlist
Target Milestone: 1.0
Assignee: Marco Martin
URL:
Keywords:
: 358968 417188 443810 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-11-21 08:05 UTC by Gregor Mi
Modified: 2021-12-25 18:42 UTC (History)
11 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.24
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Gregor Mi 2015-11-21 08:05:01 UTC
Currently the "Display Configuration" (= "Manage and configure monitors and displays") dialog is accessible via the System Settings dialog or by using krunner and knowing its name.

To make the access to "Display Configuration" more intuitive, it could be added to the context menu of the desktop and/or to the Plasma toolbox menu.

I asked the VDG about this here: https://forum.kde.org/viewtopic.php?f=285&t=129088
Comment 1 Kai Uwe Broulik 2016-06-01 15:09:24 UTC
*** Bug 358968 has been marked as a duplicate of this bug. ***
Comment 2 Gregor Mi 2018-01-20 12:35:04 UTC
I recently helped someone to install openSUSE Tumbleweed on a laptop with highdpi display. Initially, the font was very small.

On the internet I found this article: https://unix.stackexchange.com/questions/219058/scaling-the-desktop-kde which helped to find the scaling option:

System Settings/Configure Desktop → Display and Monitor → Display Configuration → (Scroll down) -> Click "Scale Display" button. For German installations (which his was) the naming on each step is different and therefore harder to find out.

If not on the first level of the menu, one could add a submenu "Display" with the following options:
- Resolution (-> opens Displays dialog)
- Scaling (e.g. for HiDPI displays) (-> opens Displays dialog)
- Multiscreen setup (-> opens Displays dialog)

Each would open the same dialog but I think it helps getting the user quickly up and running with a properly configured display.

See also https://bugs.kde.org/show_bug.cgi?id=389249 (Make Desktop toolbox also available in System Tray).
Comment 3 Christoph Feck 2018-02-16 01:03:49 UTC
Code to open kcm-kscreen should be similar to code for the clock applet opening kcm-clock.
Comment 4 Björn Feber 2019-10-04 16:42:55 UTC
Patch: https://phabricator.kde.org/D14830
Comment 5 Nate Graham 2021-03-09 04:34:39 UTC
*** Bug 417188 has been marked as a duplicate of this bug. ***
Comment 6 Bug Janitor Service 2021-03-09 17:23:01 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/704
Comment 7 Nate Graham 2021-08-08 17:16:48 UTC
This has been rejected by a senior Plasma dev on multiple occasions; closing, sorry. Not my preferred outcome either, but sometimes that's life.
Comment 8 Nate Graham 2021-10-16 03:53:46 UTC
*** Bug 443810 has been marked as a duplicate of this bug. ***
Comment 9 Lonnie 2021-10-16 05:35:21 UTC
I respectfully request that the Senior Dev, who is rejecting this request, please reconsider.

I feel very strongly about this particular request and I consider it a practical requirement that every desktop environment make "Display Settings" accessible via each and every working monitor. This way, anytime there is an issue with the primary monitor's ability to display, you always have the ability to configure, as Primary, one of the monitors that remains working.

This is especially crucial for a Desktop Environment like KDE Plasma, whose default settings provide no panels (and therefore no Application Menu) for any other display than the primary one.

For example, if a laptop's monitor stops displaying, its not so easy to just "unplug" it. It remains the primary monitor even though it isn't displaying anything. The laptop itself may not even realize that there's a physical issue with its primary display. What if you have important unsaved work on this built-in primary monitor that's no longer displaying?

You don't need to have a bad monitor to reproduce this scenario. It takes only 3 steps

STEPS TO REPRODUCE

1. Log into the KDE Plasma Desktop in a dual Monitor Set up.
2. Turn you primary monitor's display away from you, so that you cannot see anything displayed at all.
3. Attempt to access "Display Setting" on the remaining non-primary monitor (so that you can disable the primary monitor that theoretically isn't working and make the working monitor primary).

Source: https://bugs.kde.org/show_bug.cgi?id=443810

I'm typically not one who advocates the imitation MS Windows, but in this case, I see practical reasons as to why they put "Display Settings" into the right-click-context-menu of every display connected to the PC.

However, even without these practical considerations, I'm a big advocate of putting settings "at the place where they're used" as a matter of convenience. Since "Display Settings" are used to configure Displays, I can't think of a more pertinent menu-option-candidate for the right-click-menu of each monitor's desktop.

Please reconsider.
Comment 10 Nate Graham 2021-10-18 14:24:06 UTC
^^ Seems reasonable to me, maybe we can reconsider?
Comment 11 David Edmundson 2021-10-18 16:11:16 UTC
Recoverability is a sound argument.  

(FWIW, meta+p would also resolve this case, but I am aware that's not discoverable)

It would require other changes to exist before that helps anything.

If we added this menu right now. You right click, you select display settings, where that window appears is not explicit. If it appears on the primary monitor you still wouldn't be able to access it.
Comment 12 Nate Graham 2021-10-18 16:12:40 UTC
I thought windows opened on the window that has the cursor on it by default now
Comment 13 Nate Graham 2021-10-18 16:12:50 UTC
er, on the *screen
Comment 14 Nate Graham 2021-10-19 14:08:41 UTC
Indeed, active screen follows mouse is default now, so that's not a problem. System Settings will appear on the screen you right-click on to open it since the mouse is there.
Comment 15 David Edmundson 2021-10-19 14:33:33 UTC
I don't think so.

src/kcmkwin/kwinoptions/kwinoptions_settings.kcfg:
    <entry key="FocusPolicy" type="Enum">
        <default>ClickToFocus</default>

I know it changed but also got reverted as it broke all the unit tests, I don't think it changed again.


But to make sure I'm seen as compromising, here's a list of pre-requisites, then I promise I won't block anything:

 - Window should appear on the same screen (I won't be a pedant and argue about "what if it's already opened" or "what if the user changed...")

 - I would like other solutions to at least be considered. i.e integrating into edit mode It already has some KCM shortcuts, screen layout is loosely related to other layouts. 

 - A commit message that is phrased properly and problem driven like #9 (Nate, you know me well enough to know what I'm after:) )
Comment 16 David Edmundson 2021-10-19 14:34:16 UTC
Edit, that was obviously the wrong line of config

    <entry name="Placement" type="Enum">
        <default>Smart</default>
Comment 17 Nate Graham 2021-10-19 15:11:12 UTC
I see 

    <entry key="ActiveMouseScreen" type="Bool">
        <default>true</default>
    </entry>

at https://invent.kde.org/plasma/kwin/-/blob/master/src/kcmkwin/kwinoptions/kwinoptions_settings.kcfg#L139

That's different from what you're thinking of.
Comment 18 Nate Graham 2021-10-19 15:11:22 UTC
> Nate, you know me well enough to know what I'm after:)
I do. :)
Comment 19 Nate Graham 2021-10-19 15:13:49 UTC
I don't mind the idea of adding it into Edit mode, though. However there's a limited amount of horizontal space there, and I don't think it would necessarily solve the discoverability problem:

The two ways that you enter edit mode are long-press on the desktop and right-click on the desktop. In the case of long press, we're not solving the problem because that's also not discoverable, and in the case of right-click, well, if the user knows how to right-click on the desktop to go into edit mode, then they will surely see a "Configure Displays" item in the context menu. :)
Comment 20 Nate Graham 2021-10-19 15:35:34 UTC
Still, it makes more sense than the button to open the Activity Switcher, which has nothing to do with global editing.
Comment 21 Bug Janitor Service 2021-10-19 15:37:46 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/613
Comment 22 Bug Janitor Service 2021-10-19 17:45:57 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1129
Comment 23 Nate Graham 2021-10-20 16:19:00 UTC
Git commit c911754653af3d59cda63f6cb3b741fbe41b37e9 by Nate Graham.
Committed on 20/10/2021 at 16:18.
Pushed by ngraham into branch 'master'.

Edit mode toolbar: Replace Activity Switcher button with KScreen KCM button

The Activity Switcher has nothing to do with editing global settings, so
it is not an appropriate thing to be on this toolbar.

Let's replace it with a button to open the KScreen KCM, which does
involve editing global things, and letting people access it from here
may help in cases where something's gone wrong and the screen with their
panel isn't showing it anymore so they can't access a means to launch
apps to get to System Setitngs to get to the KScreen KCM to juggle
things around and try to fix the problem.

M  +3    -4    toolboxes/desktoptoolbox/contents/ui/ToolBoxButton.qml

https://invent.kde.org/plasma/plasma-desktop/commit/c911754653af3d59cda63f6cb3b741fbe41b37e9
Comment 24 Lonnie 2021-10-22 22:17:48 UTC
I have no clue how to access the "Activity Switcher" without Googling it. The only thing I know how to do, on a completely blank desktop, is to right-click it. From the context-menu produced, the closest thing I see to an "Activity Switcher" is "Activities". Clicking that, produces this:
https://i.imgur.com/ihZ6A5s.png

My request, and the submitter's request, was to put the "Display Settings" menu-item into the right-click-context-menu of every monitor's desktop. And in my own bug report, I additionally requested (with foresight) that you should also ensure that it opens on the very desktop that was right-clicked.

I hope someone at KDE appreciates directness: because anything that compromises the exactness of my prior sentence, compromises perfection. Assuming the "Activity Switcher" was something I used, and that it was easily "discoverable" from my "blank screen" (while ignoring panels to simulate default), was a false assumption. How, someone considers "Activity Switcher" a more pertinent location for "Display Settings", than the desktop's right-click menu, is beyond me.

Compared to GNOME, I'm thankful for any compromise from development. However, I consider this compromise less discoverable than what was requested.

Additionally, and separately, I also consider it less than optimal that panel-creations, created on an non-primary display, by default get place onto the primary desktop (sometimes overlaying an existing panel --terrible placement)! The oversight, of not putting display settings into the right-click context menu of each monitor's desktop, would have been easier to overcome had I been able to create a panel (with an Application Menu) onto the the very desktop that I right clicked to do so.

Again, I hope you appreciate directness: because I have plenty of it when I'm drinking.
Comment 25 Lonnie 2021-10-22 22:52:08 UTC
My apologies: clicking "Activities" from the right-click-context-menu produced this:
http://neartalk.com/ss/ActivitySwitcher.png
Comment 26 David Edmundson 2021-10-22 22:58:46 UTC
>My request, and the submitter's request, was 

We don't take requests. We discuss problems.
If you have a problem we explore all solutions. Judging a proposal and only going with the route-zero approach is exactly the sort of thing that made me close the previous proposal.
Comment 27 Lonnie 2021-10-22 23:20:01 UTC
"Zero route" approach? You closed the previous proposal because you didn't consider all the ramification of not granting it.

Now that you see more ramifications, you impose ridiculous requirements on Nate: causing him to implement a solution that's less than optimal.

Sometimes, the route propose by the reporter, is exactly optimal. Anything less leads to impertinence, as I see here.

If you want to surprise me, admit the proposal (as proposed) is optimal and don't try to make Nate shoehorn the request into an "Activity Switcher", when the more pertinent placement is EXACTLY as requested.

I don't expect my directness to persuade you at all. If anything you'll be completely put off by my words despite what truth they may hold. Again, I've been drinking. And my tact right now is deficient at best. I'm proud that I've at least restrained my profanity.
Comment 28 David Edmundson 2021-10-22 23:51:05 UTC
1) that MR did not put it in the activity switcher, it's somewhere else. 

2) I did say I wouldn't block it going in the context menu.
Comment 29 Lonnie 2021-10-23 00:28:57 UTC
You know, I have to admit, you didn't pre-reject Nate from putting this into the context-menu. I'm not sure why I developed the perception that he was scared of not meeting your expectations of not doing so. I guess it was due to the utmost respect he obvious has for you.

I have a high regard for Nate, from the other bugs I've submitted. I guess I just detected from him, an apprehension to not disappoint you in this regard, but reviewing your words I see how this was probably unfounded.

Now, I will direct my drunk anger to Nate. Why didn't you put this in the right-click-context-menu as requested by multiple people?
Comment 30 Nate Graham 2021-10-23 01:54:17 UTC
I did. ...Well, at least, I proposed it: https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1129

It still needs to be reviewed and approved before it'll be merged.
Comment 31 Nate Graham 2021-10-23 01:56:41 UTC
And yes, I do have the utmost respect for David. His reasoning that this needs to solve a real problem isn't unreasonable. I think we have found one though, which is why I submitted that merge request.

This is despite the fact that personally I think the desktop context menu is too cluttered and needs fewer things in it by default, not more. So submitting that work actually makes my future decluttering work harder, not easier, if it gets merged. That's how you know I think it does actually solve a real-world issue some people are having. :)
Comment 32 Lonnie 2021-10-26 06:46:33 UTC
Gentlemen, please accept my apologies for being contentious and for jumping to false conclusions.
Comment 33 Nate Graham 2021-12-19 17:40:32 UTC
Git commit 4c8217008593f34fdd70ce8b579d1979a6b994a4 by Nate Graham, on behalf of Ezike Ebuka.
Committed on 19/12/2021 at 17:32.
Pushed by ngraham into branch 'master'.

Add "Configure Displays" action to desktop context menu

With a multi-screen setup, it's possible to get into a situation where
all of your means to launch apps have gone missing because a screen
forgot its containmewt or moved the only panel onto a disabled output,
or any number of other issues that unfortunately have not yet been
resolved. In such a situation, it is common for people to try to fix it
by visiting the KScreen KCM to switch around their primary screens,
enable disabled outputs, turn it off and on again, etc.

However if your only visible means to launch apps has gone missing, you
may have a hard time accessing the KScreen KCM. In such a circumstance,
it can be helpful to have the action in the desktop context menu so that
people can right-click on the wallpaper of any screen that *is* working
and make System Settings launch with the KScreen KCM on that screen.

Co-authored-by: Nate Graham <nate@kde.org>
FIXED-IN: 5.24

M  +1    -0    containmentactions/contextmenu/CMakeLists.txt
M  +35   -0    containmentactions/contextmenu/menu.cpp
M  +2    -0    containmentactions/contextmenu/menu.h

https://invent.kde.org/plasma/plasma-workspace/commit/4c8217008593f34fdd70ce8b579d1979a6b994a4
Comment 34 Nick Stefanov 2021-12-25 18:23:24 UTC
Great news! Thanks for that!
Comment 35 Nate Graham 2021-12-25 18:42:02 UTC
You're welcome!