D28152: KWinRules KCM Redesign

Ismael Asensio <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
iasensio updated this revision to Diff 80240.
iasensio marked 6 inline comments as done.
iasensio added a comment.


  Address some comments:
  
  - Select items by cliking on its row
  - Disable highlights in editor listview
  - Add hints when rules list is empty
  - Use minimumWidth for percentage label

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D28152?vs=79894&id=80240

BRANCH
  iasensio/kcm_kwinrules_xt

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

AFFECTED FILES
  kcmkwin/kwinrules/CMakeLists.txt
  kcmkwin/kwinrules/detectwidget.cpp
  kcmkwin/kwinrules/detectwidget.h
  kcmkwin/kwinrules/detectwidget.ui
  kcmkwin/kwinrules/editshortcut.ui
  kcmkwin/kwinrules/kcm.cpp
  kcmkwin/kwinrules/kcm.h
  kcmkwin/kwinrules/kcm_kwinrules.desktop
  kcmkwin/kwinrules/kcmrules.cpp
  kcmkwin/kwinrules/kcmrules.h
  kcmkwin/kwinrules/kwinrules.desktop
  kcmkwin/kwinrules/main.cpp
  kcmkwin/kwinrules/optionsmodel.cpp
  kcmkwin/kwinrules/optionsmodel.h
  kcmkwin/kwinrules/package/contents/ui/FileDialogLoader.qml
  kcmkwin/kwinrules/package/contents/ui/OptionsComboBox.qml
  kcmkwin/kwinrules/package/contents/ui/RuleItemDelegate.qml
  kcmkwin/kwinrules/package/contents/ui/RulesEditor.qml
  kcmkwin/kwinrules/package/contents/ui/RulesList.qml
  kcmkwin/kwinrules/package/contents/ui/ValueEditor.qml
  kcmkwin/kwinrules/package/metadata.desktop
  kcmkwin/kwinrules/rulebookmodel.cpp
  kcmkwin/kwinrules/rulebookmodel.h
  kcmkwin/kwinrules/ruleitem.cpp
  kcmkwin/kwinrules/ruleitem.h
  kcmkwin/kwinrules/rulesdialog.cpp
  kcmkwin/kwinrules/rulesdialog.h
  kcmkwin/kwinrules/ruleslist.cpp
  kcmkwin/kwinrules/ruleslist.h
  kcmkwin/kwinrules/ruleslist.ui
  kcmkwin/kwinrules/rulesmodel.cpp
  kcmkwin/kwinrules/rulesmodel.h
  kcmkwin/kwinrules/ruleswidget.cpp
  kcmkwin/kwinrules/ruleswidget.h
  kcmkwin/kwinrules/ruleswidgetbase.ui
  kcmkwin/kwinrules/yesnobox.cpp
  kcmkwin/kwinrules/yesnobox.h

To: iasensio, #plasma, #kwin, #vdg
Cc: ngraham, davidedmundson, hchain, broulik, zzag, kwin, dmenig, manueljlin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, konkinartem, ian, jguidon, Ghost6, jraleigh, zachus, fbampaloukas, squeakypancakes, alexde, IohannesPetros, GB_2, mkulinski, trickyricky26, ragreen, jackyalcine, iodelay, crozbo, ndavis, bwowk, ZrenBot, firef, alexeymin, skadinna, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, aaronhoneycutt, abetts, sebas, apol, ahiemstra, mbohlender, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.