Change in plasma-framework[master]: Unify when tooltips are shown and kept alive.

2015-02-10 Thread David Edmundson (Code Review)
David Edmundson has uploaded a new change for review. https://gerrit.vesnicky.cesnet.cz/r/365 Change subject: Unify when tooltips are shown and kept alive. .. Unify when tooltips are shown and kept alive. A new tooltip would

Re: Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread David Edmundson
On Feb. 10, 2015, 1:33 p.m., David Edmundson wrote: lookandfeel/contents/lockscreen/LockScreen.qml, line 161 https://git.reviewboard.kde.org/r/122511/diff/1/?file=348176#file348176line161 Make sure you test this actually gets called when you switch VTs. It will require

Re: Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122511/ --- (Updated Feb. 10, 2015, 3:04 p.m.) Status -- This change has been

Re: Review Request 122512: [screenlocker] Add a KKeySequenceWidget to configuration interface

2015-02-10 Thread Bhushan Shah
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122512/#review75795 --- File Attachment: KCM with new item -

Re: Review Request 122400: Add timedated support into the clock KCM as an optional dependency

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122400/ --- (Updated Feb. 10, 2015, 3:55 p.m.) Review request for Plasma.

Review Request 122512: [screenlocker] Add a KKeySequenceWidget to configuration interface

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122512/ --- Review request for Plasma. Bugs: 325622

Re: Review Request 122400: Add timedated support into the clock KCM as an optional dependency

2015-02-10 Thread David Edmundson
On Feb. 9, 2015, 7:47 a.m., Martin Gräßlin wrote: kcms/dateandtime/main.cpp, line 57 https://git.reviewboard.kde.org/r/122400/diff/4/?file=347911#file347911line57 does it need to be a blocking dbus call? Any chance in getting it use a watcher? It's tricky as load() is called by

Re: Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122511/#review75791 --- Ship it! Ship It! - David Edmundson On Feb. 10, 2015,

Re: Review Request 122510: [screenlocker] Mark session as idle in logind while screen is locked

2015-02-10 Thread David Edmundson
On Feb. 10, 2015, 1:29 p.m., David Edmundson wrote: I'm not convinced this is right. From the doc you linked: This is necessary for the system to implement auto-suspend when all sessions are idle. When we lock the screen, powerdevil is still running, no? Powerdevil has

Re: Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread Martin Gräßlin
On Feb. 10, 2015, 2:33 p.m., David Edmundson wrote: lookandfeel/contents/lockscreen/LockScreen.qml, line 161 https://git.reviewboard.kde.org/r/122511/diff/1/?file=348176#file348176line161 Make sure you test this actually gets called when you switch VTs. It will require

Re: Review Request 122512: [screenlocker] Add a KKeySequenceWidget to configuration interface

2015-02-10 Thread Martin Gräßlin
On Feb. 10, 2015, 4:10 p.m., Bhushan Shah wrote: ksmserver/screenlocker/kcm/kcm.cpp, lines 96-97 https://git.reviewboard.kde.org/r/122512/diff/1/?file=348184#file348184line96 isn't ksmserver doing this already? I looked at kwin configuration modules for how it's handled and applied

Re: Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122511/#review75785 --- FWIW, I think that's a good idea. As David says, make sure to

Re: Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread Martin Gräßlin
On Feb. 10, 2015, 2:33 p.m., David Edmundson wrote: lookandfeel/contents/lockscreen/LockScreen.qml, line 161 https://git.reviewboard.kde.org/r/122511/diff/1/?file=348176#file348176line161 Make sure you test this actually gets called when you switch VTs. It will require

Re: Review Request 122510: [screenlocker] Mark session as idle in logind while screen is locked

2015-02-10 Thread Martin Gräßlin
On Feb. 10, 2015, 2:29 p.m., David Edmundson wrote: I'm not convinced this is right. From the doc you linked: This is necessary for the system to implement auto-suspend when all sessions are idle. When we lock the screen, powerdevil is still running, no? Powerdevil has

Re: Review Request 122513: Avoid comparing unintialised variable if BREEZE_HAVE_KSTYLE is not set

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122513/ --- (Updated Feb. 10, 2015, 4:48 p.m.) Status -- This change has been

Re: Review Request 122513: Avoid comparing unintialised variable if BREEZE_HAVE_KSTYLE is not set

2015-02-10 Thread Hugo Pereira Da Costa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122513/#review75799 --- Ship it! Nice spotting ! Thanks ! - Hugo Pereira Da Costa

Review Request 122513: Avoid comparing unintialised variable if BREEZE_HAVE_KSTYLE is not set

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122513/ --- Review request for Plasma and Hugo Pereira Da Costa. Repository: breeze

[Powerdevil] [Bug 343962] Add option to disable Backlight OSD

2015-02-10 Thread Antonis G .
https://bugs.kde.org/show_bug.cgi?id=343962 --- Comment #12 from Antonis G. gaanto...@civil.auth.gr --- (In reply to Kai Uwe Broulik from comment #11) The threads complaining about the OSD you have linked do not apply to the Plasma 5 world. In Plasma 4.x the OSD would show up whenever the

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Kai Uwe Broulik
On Feb. 10, 2015, 9:01 vorm., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). As I stated above, all the way down can mean

Re: Review Request 122506: Fix crash due to unsafe access to ShellCorona::m_screenConfiguration

2015-02-10 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122506/#review75759 --- shell/shellcorona.cpp

Re: Review Request 122339: Add KWorkSpace::isShuttingDown()

2015-02-10 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122339/ --- (Updated Feb. 10, 2015, 9:15 a.m.) Status -- This change has been

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Mark Gaiser
On feb 10, 2015, 9:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above, all

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122505/#review75756 --- I'm not quite sure if a user wants to see a warning message

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Martin Klapetek
On Feb. 10, 2015, 10:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122452: [shell] Emit dbus signals in org.kde.osdService

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122452/ --- (Updated Feb. 10, 2015, 10:07 a.m.) Status -- This change has been

Jenkins build is back to normal : plasma-workspace_master_qt5 #1292

2015-02-10 Thread KDE CI System
See http://build.kde.org/job/plasma-workspace_master_qt5/1292/changes ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 122509: [screenlocker] Perform KAuthorized check for org.freedesktop.ScreenSaver.Lock

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122509/ --- Review request for Plasma. Repository: plasma-workspace Description

Build failed in Jenkins: plasma-workspace_master_qt5 #1291

2015-02-10 Thread KDE CI System
See http://build.kde.org/job/plasma-workspace_master_qt5/1291/changes Changes: [mklapetek] Sanitize whitespace in notification popups [shafff] fix total system freeze on some systems like my 10 inch netbook [lukas] set the right widget theme [scripty] SVN_SILENT made messages (.desktop file)

Review Request 122508: [screenlocker] Support global shortcut for changing keyboard layout

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122508/ --- Review request for Plasma. Bugs: 331726

Re: Review Request 122452: [shell] Emit dbus signals in org.kde.osdService

2015-02-10 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122452/#review75763 --- Ship it! A few minor nitpicks. Nice work! We can fix and

Re: Review Request 122509: [screenlocker] Perform KAuthorized check for org.freedesktop.ScreenSaver.Lock

2015-02-10 Thread Martin Gräßlin
On Feb. 10, 2015, 12:35 p.m., David Edmundson wrote: If an admin as root says to lock the screen via logind, should it not lock regarless? (I don't know the right answer, just raising a question) If an admin as root says to lock the screen via logind, should it not lock regarless?

Re: Review Request 122509: [screenlocker] Perform KAuthorized check for org.freedesktop.ScreenSaver.Lock

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122509/#review75768 --- If an admin as root says to lock the screen via logind,

Re: Review Request 122508: [screenlocker] Support global shortcut for changing keyboard layout

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122508/ --- (Updated Feb. 10, 2015, 12:06 p.m.) Status -- This change has been

Re: Review Request 122508: [screenlocker] Support global shortcut for changing keyboard layout

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122508/#review75767 --- Ship it! Did the little indicator in the main UI change? -

Re: Review Request 122508: [screenlocker] Support global shortcut for changing keyboard layout

2015-02-10 Thread Martin Gräßlin
On Feb. 10, 2015, 12:23 p.m., David Edmundson wrote: Did the little indicator in the main UI change? just tested: yes - Martin --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 122510: [screenlocker] Mark session as idle in logind while screen is locked

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122510/#review75777 --- I'm not convinced this is right. From the doc you linked:

Re: Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122511/#review75778 --- Ship it! lookandfeel/contents/lockscreen/LockScreen.qml

Re: Review Request 122502: Add a `X-Plasma-Can-Be-Windowed` option to applets

2015-02-10 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122502/ --- (Updated Feb. 10, 2015, 1:35 p.m.) Review request for Plasma. Changes

Re: Review Request 122500: Don't delete widgets directly when changing styles

2015-02-10 Thread David Edmundson
On Feb. 9, 2015, 5:34 p.m., Hugo Pereira Da Costa wrote: Although I have no objection against the change, I must admit I don't understand what's wrong with the current code, nor the actual description of the patch. registerWidget may take an existing widget as a parameter. If so, we

Re: Review Request 122502: Add a `X-Plasma-Can-Be-Windowed` option to applets

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122502/#review75780 --- +1 - David Edmundson On Feb. 10, 2015, 1:35 p.m., Vishesh

Re: Review Request 122502: Add a `X-Plasma-Can-Be-Windowed` option to applets

2015-02-10 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122502/#review75781 --- Ship it! Ship It! - Marco Martin On Feb. 10, 2015, 1:35

Re: Review Request 122500: Don't delete widgets directly when changing styles

2015-02-10 Thread Hugo Pereira Da Costa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122500/#review75782 --- Ship it! Ship It! - Hugo Pereira Da Costa On Feb. 10,

Re: Review Request 122502: Add a `X-Plasma-Can-Be-Windowed` option to applets

2015-02-10 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122502/ --- (Updated Feb. 10, 2015, 1:43 p.m.) Status -- This change has been

Re: Review Request 122500: Don't delete widgets directly when changing styles

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122500/ --- (Updated Feb. 10, 2015, 1:47 p.m.) Status -- This change has been

[Powerdevil] [Bug 189456] powerdevil should not suspend when computer is being shutdown

2015-02-10 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=189456 Kai Uwe Broulik k...@privat.broulik.de changed: What|Removed |Added Version Fixed In||5.3.0

Re: Get panel list and hidding them via C++ module

2015-02-10 Thread Evgeniy Alekseev
On Tuesday 10 February 2015 12:32:49 Sebastian Kügler wrote: Why not just add the ability to hide a panel with a shortcut? Just an idea that some related items may be collected by one item (e.g. Plasmoid) to prevent similar actions to configure item. (BTW I didn't find ability to hide panel by

Re: Review Request 122400: Add timedated support into the clock KCM as an optional dependency

2015-02-10 Thread David Edmundson
On Feb. 10, 2015, 7:51 p.m., Xuetian Weng wrote: kcms/dateandtime/main.cpp, line 146 https://git.reviewboard.kde.org/r/122400/diff/5/?file=348191#file348191line146 This part can be async. You can do it by chain two different callback if it need to be done one after

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Thomas Pfeiffer
On Feb. 10, 2015, 9:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122400: Add timedated support into the clock KCM as an optional dependency

2015-02-10 Thread Xuetian Weng
On Feb. 10, 2015, 7:51 p.m., Xuetian Weng wrote: kcms/dateandtime/main.cpp, line 146 https://git.reviewboard.kde.org/r/122400/diff/5/?file=348191#file348191line146 This part can be async. You can do it by chain two different callback if it need to be done one after

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Martin Klapetek
On Feb. 10, 2015, 10:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122400: Add timedated support into the clock KCM as an optional dependency

2015-02-10 Thread David Edmundson
On Feb. 10, 2015, 7:51 p.m., Xuetian Weng wrote: kcms/dateandtime/main.cpp, line 146 https://git.reviewboard.kde.org/r/122400/diff/5/?file=348191#file348191line146 This part can be async. You can do it by chain two different callback if it need to be done one after

Re: Review Request 122400: Add timedated support into the clock KCM as an optional dependency

2015-02-10 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122400/#review75801 --- kcms/dateandtime/main.cpp

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Mark Gaiser
On feb 10, 2015, 9:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above, all

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Thomas Pfeiffer
On Feb. 10, 2015, 9:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Thomas Pfeiffer
On Feb. 10, 2015, 9:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Martin Klapetek
On Feb. 10, 2015, 10:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Martin Klapetek
On Feb. 10, 2015, 10:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Martin Klapetek
On Feb. 10, 2015, 10:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Thomas Pfeiffer
On Feb. 10, 2015, 9:01 a.m., Mark Gaiser wrote: I'm not quite sure if a user wants to see a warning message at all. When i use my notebook in a dark environment i usually put the brightness all the way down (depending on the notebook). Kai Uwe Broulik wrote: As I stated above,

Re: Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread David Edmundson
On Feb. 10, 2015, 1:33 p.m., David Edmundson wrote: lookandfeel/contents/lockscreen/LockScreen.qml, line 161 https://git.reviewboard.kde.org/r/122511/diff/1/?file=348176#file348176line161 Make sure you test this actually gets called when you switch VTs. It will require

Review Request 122511: [screenlocker] Clear password field whenever the visibility changes

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122511/ --- Review request for Plasma. Bugs: 326633

Re: Review Request 122510: [screenlocker] Mark session as idle in logind while screen is locked

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122510/ --- (Updated Feb. 10, 2015, 2:21 p.m.) Review request for Plasma. Changes

Re: Review Request 122500: Don't delete widgets directly when changing styles

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122500/ --- (Updated Feb. 10, 2015, 1:25 p.m.) Review request for Plasma and Hugo

Re: Review Request 122500: Don't delete widgets we don't own when changing styles

2015-02-10 Thread David Edmundson
On Feb. 9, 2015, 5:34 p.m., Hugo Pereira Da Costa wrote: Although I have no objection against the change, I must admit I don't understand what's wrong with the current code, nor the actual description of the patch. registerWidget may take an existing widget as a parameter. If so, we

Re: Proposal regarding plasmate's release

2015-02-10 Thread Sebastian Kügler
On Tuesday, February 03, 2015 12:29:08 Philipp A. wrote: Sebastian Kügler se...@kde.org schrieb am Tue Feb 03 2015 at 12:59:22: If you're using imports, you have to install them first, no way around that. no way, that’s silly. how can we fix that? You may be able to work around that by

Re: Get panel list and hidding them via C++ module

2015-02-10 Thread Sebastian Kügler
On Monday, February 09, 2015 22:47:20 Evgeniy Alekseev wrote: Have you looked at desktop scripting to achieve what you are looking for? There is an applet on which I'm working now (rather, on porting) which provides some concepts from minimalistic WMs (predominantly tiling ones) which

Re: Review Request 122510: [screenlocker] Mark session as idle in logind while screen is locked

2015-02-10 Thread Lukáš Tinkl
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122510/#review75776 --- ksmserver/screenlocker/logind.cpp

Re: Review Request 122509: [screenlocker] Perform KAuthorized check for org.freedesktop.ScreenSaver.Lock

2015-02-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122509/#review75774 --- Ship it! Ship It! - David Edmundson On Feb. 10, 2015,

[plasma-mediacenter] browsingbackends: Merge branch 'bshah/pmc2'

2015-02-10 Thread Bhushan Shah
Git commit 5fcea4fd58b6799c785958cd6f1455c3865e1165 by Bhushan Shah. Committed on 10/02/2015 at 12:20. Pushed by bshah into branch 'master'. Merge branch 'bshah/pmc2' This merges the new design in master branch, Few things - free from kdelibs4support - requires clean rebuild and install dir -

Review Request 122510: [screenlocker] Mark session as idle in logind while screen is locked

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122510/ --- Review request for Plasma. Bugs: 271731

Re: Review Request 122509: [screenlocker] Perform KAuthorized check for org.freedesktop.ScreenSaver.Lock

2015-02-10 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122509/ --- (Updated Feb. 10, 2015, 12:35 p.m.) Status -- This change has been

Re: Review Request 122505: Warn about brightness getting too low

2015-02-10 Thread Thomas Pfeiffer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122505/#review75824 --- Basic rule from design for safety: Don't use a warning if you