D23615: Improve the look of the KWin options KCM UIs

davidedmundson added a comment.


  I don't like that the detailed description of the focus policy is effectively lost. It's quite important.
  VDG decided in another thread no-one uses whatsthis, which I kinda agree with.

INLINE COMMENTS

> main.cpp:92
>      mMoving = new KMovingConfig(false, mConfig, this);
> -    mMoving->setObjectName(QLatin1String("KWin Moving"));
> -    tab->addTab(mMoving, i18n("&Moving"));
> +    mMoving->setObjectName(QLatin1String("KWin Movement"));
> +    tab->addTab(mMoving, i18n("Mo&vement"));

Where does this object name get used?

> mouse.cpp:261
>      "Alt",
> +    "Meta",
>      ""

why is this swapped?

REPOSITORY
  R108 KWin

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

To: GB_2, #kwin, #plasma, #vdg
Cc: davidedmundson, zzag, #vdg, #plasma, kwin, #kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart