D22803: Improved OSD behavior

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  Don't ask the user for a screen configuration on login, instead, apply a sane default.
  When connecting a new screen, apply a default configuration so the user gets immediate feedback "something's happening" and then prompt for a configuration via the OSD in case the chosen default doesn't fit.

TEST PLAN
  The login case is especially annoying as it would usually choose a sane default and then the user would choose Cancel which won't save a config and then kscreen asks again next time until the user choses anyhting but Cancel.
  
  Nuked my kscreen config, restarted kded, two screens attached, had kscreen apply a default config for my two screens without asking.
  Nuked my kscreen config, unplugged one monitor, restarted kded, plugged in second screen, KScreen applied a sane default (though it cannot know that the screens are swapped), and showed the OSD which I then used to swap the monitors "extend left" to how they should be

REPOSITORY
  R104 KScreen

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

AFFECTED FILES
  kded/daemon.cpp
  kded/daemon.h

To: broulik, #plasma, #vdg
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190729/578925da/attachment.html>