Review Request 129872: [lookandfeel] Support changing window decoration

Anthony Fieroni bvbfan at abv.bg
Sat Jan 28 10:02:32 UTC 2017


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129872/
-----------------------------------------------------------

(Updated Ян. 28, 2017, 12:02 след обяд)


Review request for Plasma, Kai Uwe Broulik, Hugo Pereira Da Costa, and Marco Martin.


Repository: plasma-desktop


Description (updated)
-------

Applying a look and feel global theme will change visual changes in:
1. Widget style
2. Color scheme
3. Icons
4. Plasma theme
5. Mouse pointer
6. Window switcher
7. Desktop switcher
8. *Window decoration*
This patch aims to implement (or correct) applying of windows decortion. When user select Oxygen look and feel (before was other no matter what) it should read default configuration for oxygen look and feel. It's located in oxygen/lookandfeel/contents/defaults (https://git.reviewboard.kde.org/r/129867/), e.g. it's look like

[kdeglobals][KDE]
widgetStyle=oxygen

[kdeglobals][General]
ColorScheme=Oxygen

[kdeglobals][Icons]
Theme=oxygen

[plasmarc][Theme]
name=air

[kcminputrc][Mouse]
cursorTheme=Oxygen_Black

[kwinrc][WindowDecoration]
library=org.kde.oxygen

In KCMLookandFeel::load() we get what package support (you see oxygen does not support changing WindowSwitcher and DesktopSwitcher)
In KCMLookandFeel::save() we read what is supported now in value e.g. library=org.kde.oxygen and write it in place that will work and kwin expects in kwinrc by function
KCMLookandFeel::setWindowDecoration kwinrc does not have a theme -> i will remove it. In DBUS calls, after writing in right place of configuration, we apply all changes.


Diffs
-----

  kcms/lookandfeel/autotests/kcmtest.cpp 29d6ec0f 
  kcms/lookandfeel/autotests/lookandfeel/contents/defaults ef3165de 
  kcms/lookandfeel/kcm.h c27b7f3d 
  kcms/lookandfeel/kcm.cpp 6ea3d050 

Diff: https://git.reviewboard.kde.org/r/129872/diff/


Testing
-------


Thanks,

Anthony Fieroni

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170128/706742ad/attachment-0001.html>


More information about the Plasma-devel mailing list