Re: Review Request: Make “No multiscreen configuration” message prettier

Kai Uwe Broulik kde at privat.broulik.de
Thu Jun 30 21:45:49 BST 2011


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

(Updated June 30, 2011, 8:45 p.m.)


Review request for KDE Base Apps.


Changes
-------

Fixed indentation.
Sorry, I always forget switching indentation type (tabulator (goood one), space (baad one)) if I work on different projects ;)


Summary
-------

I always found Bluedevil’s notifications so nice, so I patched the Multi monitor configuration KCM to use something similar to indicate that this configuration is not available on the respective machine.

Comparison screenshot: http://privat.broulik.de/xineramapatch.png


Diffs (updated)
-----

  CMakeLists.txt 6709241 
  kcontrol/xinerama/kcmxinerama.h fc83e5a 
  kcontrol/xinerama/kcmxinerama.cpp 86fbf38 
  kwin/effects/CMakeLists.txt 0312ac2 
  kwin/effects/login/login.desktop 1a2cb47 
  libs/taskmanager/taskactions.cpp 0e6ba8e 

Diff: http://git.reviewboard.kde.org/r/101810/diff


Testing
-------


Thanks,

Kai Uwe

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110630/4ac5ab7e/attachment.htm>


More information about the kde-core-devel mailing list