D23615: Improve the look of the KWin options KCM UIs

Vlad Zagorodniy noreply at phabricator.kde.org
Fri Sep 6 09:33:01 BST 2019


zzag added inline comments.

INLINE COMMENTS

> windows.cpp:176
>  {
> +    switch (m_ui->windowFocusPolicyCombo->currentIndex())
> +    {

Style: a switch statement should look like

  switch (foo) {
  case Foo:
      break;
  case Bar:
      break;
  }

i.e. `case`s are not indented and opening brace is kept on the same line as `switch` keyword.

REPOSITORY
  R108 KWin

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

To: GB_2, #kwin, #plasma, #vdg
Cc: ngraham, davidedmundson, zzag, #vdg, #plasma, kwin, #kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190906/4231923d/attachment-0001.html>


More information about the kwin mailing list