D14830: Add "Configure Displays" context menu entry on the desktop

[Nick Stefanov]

nstefanov added a comment.


  In D14830#541878 , @GB_2 wrote:
  
  > Is there a reason this got abandoned? :/
  
  
  I wonder too. This feature will be a great improvement. Many Windows users I 
migrate to Linux first ask where are the Display Settings. Is that much one 
context menu entry? Furthermore it's also far more convenient for us, Linux 
users. I'm tired to go to App Launcher and seek for Displa Settings. Yeah, I 
know there's a widget but I don't like many icons in the system tray. They sey 
KDE is the Swiss knife of Linux DEs. Let it be it :)

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: nstefanov, GB_2, gregormi, mart, davidedmundson, abetts, romangg, fabianr, 
ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, sebas, apol, ahiemstra

D14830: Add "Configure Displays" context menu entry on the desktop

[Björn Feber]

GB_2 added a comment.


  Is there a reason this got abandoned? :/

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: GB_2, gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, 
apol

D14830: Add "Configure Displays" context menu entry on the desktop

[Björn Feber]

GB_2 added inline comments.

INLINE COMMENTS

> GB_2 wrote in menu.cpp:117
> I'd change it to "Manage Displays..." to make the differentiate the name even 
> more from the "Configure Desktop..." menu entry.

*to differentiate

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: GB_2, gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, 
Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[Björn Feber]

GB_2 added inline comments.

INLINE COMMENTS

> menu.cpp:117
>  
> +m_displaySettingsAction = new 
> QAction(i18nc("plasma_containmentactions_contextmenu", "Configure 
> Displays..."), this);
> +
> m_displaySettingsAction->setIcon(QIcon::fromTheme(QStringLiteral("preferences-desktop-display-randr")));

I'd change it to "Manage Displays..." to make the differentiate the name even 
more from the "Configure Desktop..." menu entry.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: GB_2, gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, 
Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[Björn Feber]

GB_2 added a comment.


  I think we should definitely add this shortcut, because it is very useful for 
the people mentioned and for other situations. I agree that it should be on by 
default, then it's already there when you need it.
  
  In D14830#472177 , @mart wrote:
  
  > if this would be done, i would prefer for the kcm once ported to qml to be 
embedded in the desktop settings window
  
  
  D22468  ports the KCM to QML, but it 
doesn't integrate it into the desktop settings. Just having this shortcut is 
fine IMO, the desktop settings already a lot of categories and configuring the 
desktop does not mean managing displays.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: GB_2, gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, 
Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[Marco Martin]

mart added a comment.


  if this would be done, i would prefer for the kcm once ported to qml to be 
embedded in the desktop settings window

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[Kai Uwe Broulik]

broulik added a comment.


  > "back in Windows 7": you mean Windows 10?
  
  No, Windows 7. To quote from my favorite OS review page on Windows 7 [1] "The 
desktop right-click context menu also now contains a "Screen Resolution" 
option. […] I wonder if MS actually listened to consumers on this one or if it 
was just to stop the chair throwing when Balmer tried to use his laptop with 
the projector in meetings."
  
  [1] http://toastytech.com/guis/win7.html

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[gregormi]

gregormi added a comment.


  In D14830#345210 , @broulik wrote:
  
  > Note that Microsoft removed that option in Vista in favor of a 
"Personalize" option but brought a dedicated Screen configuration entry back in 
Windows 7 which has remained to this very day.
  
  
  "back in Windows 7": you mean Windows 10?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, 
sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[Kai Uwe Broulik]

broulik added a comment.


  Note that Microsoft removed that option in Vista in favor of a "Personalize" 
option but brought a dedicated Screen configuration entry back in Windows 7 
which has remained to this very day.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, 
sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[gregormi]

gregormi added a comment.


  In D14830#313645 , @davidedmundson 
wrote:
  
  > I am still waiting for a single justification.
  
  
  In https://phabricator.kde.org/T6875, under "Who should be our primary target 
groups?" we have "Windows 7 Power Users". Those users will benefit from this 
change (because there, the well-known way to reach the configure screens dialog 
is to right click on the desktop).

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, 
sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[Nathaniel Graham]

ngraham added a comment.


  In D14830#313645 , @davidedmundson 
wrote:
  
  > I am still waiting for a single justification.
  
  
  I think some were provided in the description:
  
  > One of LiMux client's requirements is for display configuration to be 
easily accessible. Adding a "Configure Displays..." context menu entry which is 
widely known by being present on another operating system aids in users finding 
those settings.
  
  
  
  > It also gets asked for occasionally by users on the internet.
  
  IMHO, "a major institutional user wants it" is a pretty fair justification 
all by itself.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: mart, davidedmundson, abetts, romangg, fabianr, ngraham, plasma-devel, 
ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[David Edmundson]

davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.


  I am still waiting for a single justification.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: mart, davidedmundson, abetts, romangg, fabianr, ngraham, plasma-devel, 
ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol

D14830: Add "Configure Displays" context menu entry on the desktop

[Kai Uwe Broulik]

broulik added a comment.


  Ping

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy
Cc: mart, davidedmundson, abetts, romangg, fabianr, ngraham, plasma-devel, 
ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol