Re: Review Request 125700: Drop remaning code for ShowDashboard

2015-10-19 Thread Martin Gräßlin
> On Oct. 19, 2015, 3:41 p.m., Martin Gräßlin wrote: > > This also needs a kconf update, but I fear my scripting capabilities are > > not sufficient for that. It should be doable with a simple sed call, though. > > Thomas Lübking wrote: > Humm? Looks like it's resolved to "None", what's goo

Re: Review Request 125678: Introduce User Switcher plasmoid

2015-10-19 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125678/ --- (Updated Okt. 19, 2015, 9:22 nachm.) Review request for Plasma and KDE Us

Re: Review Request 125706: Install kconf_update script in kdelibs4 directory

2015-10-19 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125706/ --- (Updated Oct. 19, 2015, 9:16 p.m.) Review request for Plasma, David Edmun

Re: Review Request 125706: Install kconf_update script in kdelibs4 directory

2015-10-19 Thread Jonathan Riddell
> On Oct. 19, 2015, 4:36 p.m., Hrvoje Senjan wrote: > > >because kmix is a kdelibs4 application > > > > well, only when it's built that way :D KMIX_KF5_BUILD will enable KF5 > > codepath/buildsystem. IOW, you need to make sure both cases are covered ok, updated - Jonathan -

Re: Review Request 125678: Introduce User Switcher plasmoid

2015-10-19 Thread Heiko Tietze
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125678/#review87099 --- Someone asked on the mailing list to keep the user icon. Agree

Re: Review Request 125706: Install kconf_update script in kdelibs4 directory

2015-10-19 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125706/ --- (Updated Oct. 19, 2015, 7:14 p.m.) Review request for Plasma, David Edmun

Re: Review Request 125705: Use LANG for month names in calendar applet

2015-10-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125705/#review87095 --- +1 you won't be able to use this trick for the tooltip date

Re: Review Request 125709: Ensure correct timezone is displayed when the selected ones change

2015-10-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125709/#review87094 --- can it be merged with the code in main.qml which is already bi

Re: Review Request 125709: Ensure correct timezone is displayed when the selected ones change

2015-10-19 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125709/ --- (Updated Oct. 19, 2015, 7:54 p.m.) Review request for Plasma. Changes -

Re: Review Request 125709: Ensure correct timezone is displayed when the selected ones change

2015-10-19 Thread Martin Klapetek
> On Oct. 19, 2015, 7:47 p.m., David Edmundson wrote: > > applets/digital-clock/package/contents/ui/DigitalClock.qml, line 638 > > > > > > do you need this still? Yeah, in fact this should be a function and shou

Re: Review Request 125709: Ensure correct timezone is displayed when the selected ones change

2015-10-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125709/#review87092 --- applets/digital-clock/package/contents/ui/DigitalClock.qml (l

Review Request 125709: Ensure correct timezone is displayed when the selected ones change

2015-10-19 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125709/ --- Review request for Plasma. Bugs: 353996 https://bugs.kde.org/show_bug

Re: Review Request 125706: Install kconf_update script in kdelibs4 directory

2015-10-19 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125706/#review87090 --- >because kmix is a kdelibs4 application well, only when it's

Review Request 125706: Install kconf_update script in kdelibs4 directory

2015-10-19 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125706/ --- Review request for Plasma, David Edmundson and Harald Sitter. Repository:

Review Request 125705: Use LANG for month names in calendar applet

2015-10-19 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125705/ --- Review request for KDE Frameworks and Plasma. Bugs: 353715 http://bug

Re: Review Request 125700: Drop remaning code for ShowDashboard

2015-10-19 Thread Thomas Lübking
> On Okt. 19, 2015, 1:41 nachm., Martin Gräßlin wrote: > > This also needs a kconf update, but I fear my scripting capabilities are > > not sufficient for that. It should be doable with a simple sed call, though. Humm? Looks like it's resolved to "None", what's good - isn't? +1 - Thomas -

[Powerdevil] [Bug 352497] laptop back light not coming back on after long period of lcd being in dpms power save mode.

2015-10-19 Thread Matteo Croce via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=352497 --- Comment #4 from Matteo Croce --- Created attachment 95045 --> https://bugs.kde.org/attachment.cgi?id=95045&action=edit login SDDM is active, I can login by typing blindly -- You are receiving this mail because: You are the assignee for the bug.

[Powerdevil] [Bug 352497] laptop back light not coming back on after long period of lcd being in dpms power save mode.

2015-10-19 Thread Matteo Croce via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=352497 Matteo Croce changed: What|Removed |Added Attachment #95043|0 |1 is obsolete|

[Powerdevil] [Bug 352497] laptop back light not coming back on after long period of lcd being in dpms power save mode.

2015-10-19 Thread Matteo Croce via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=352497 --- Comment #2 from Matteo Croce --- Created attachment 95043 --> https://bugs.kde.org/attachment.cgi?id=95043&action=edit how to fix This is how I have to enable the backlight again every time, using a torch -- You are receiving this mail because:

[Powerdevil] [Bug 352497] laptop back light not coming back on after long period of lcd being in dpms power save mode.

2015-10-19 Thread Matteo Croce via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=352497 Matteo Croce changed: What|Removed |Added CC||rootki...@yahoo.it --- Comment #1 from Matteo Cr

Re: Review Request 125699: switch to Noto font and dejavu sans font

2015-10-19 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125699/ --- (Updated Oct. 19, 2015, 2:41 p.m.) Review request for Plasma. Repositor

Re: Review Request 125699: switch to Noto font and dejavu sans font

2015-10-19 Thread Jonathan Riddell
> On Oct. 19, 2015, 2:11 p.m., David Edmundson wrote: > > src/platformtheme/frameworksintegration-5.16-font.upd, line 5 > > > > > > if you add > > > > File=kdeglobals > > > > > > kconfupdate

Re: Review Request 125605: Resize lock screen windows if screen size changes

2015-10-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125605/ --- (Updated Oct. 19, 2015, 2:38 p.m.) Status -- This change has been ma

Re: Review Request 124267: Use KWin to lower/raiser panel in windows can cover mode with edge activation

2015-10-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124267/ --- (Updated Oct. 19, 2015, 2:21 p.m.) Status -- This change has been ma

Re: Review Request 125702: switch to noto sans

2015-10-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125702/#review87079 --- Ship it! probably best to wait for the other changes to be OK

Re: Review Request 125699: switch to Noto font and dejavu sans font

2015-10-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125699/#review87078 --- src/platformtheme/frameworksintegration-5.16-font.upd (line 5

Re: Review Request 125699: switch to Noto font and dejavu sans font

2015-10-19 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125699/ --- (Updated Oct. 19, 2015, 2:09 p.m.) Review request for Plasma. Repositor

Review Request 125702: switch to noto sans

2015-10-19 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125702/ --- Review request for Plasma. Repository: plasma-desktop Description -

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review87077 --- KWin supports loading scripts at runtime through a dbus interf

Re: Review Request 125699: switch to Noto font and dejavu sans font

2015-10-19 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125699/ --- (Updated Oct. 19, 2015, 2:05 p.m.) Review request for Plasma. Changes -

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review87074 --- FWIW, I know this was deliberately not implemented in the past

Re: Review Request 125700: Drop remaning code for ShowDashboard

2015-10-19 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125700/#review87072 --- This also needs a kconf update, but I fear my scripting capabi

Review Request 125700: Drop remaning code for ShowDashboard

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

Re: Review Request 125686: Drop the Dashboard Effect

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

Re: Review Request 125555: Port KAuth return code error checking in fontinst

2015-10-19 Thread Martin Gräßlin
> On Oct. 19, 2015, 12:40 p.m., Martin Gräßlin wrote: > > kcms/kfontinst/dbus/FontInst.cpp, line 957 > > > > > > that's an async call, isn't it? So somehow I doubt it will work that > > way. > > David Edmundson

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread Marco Martin
> On Oct. 19, 2015, 10:43 a.m., Marco Martin wrote: > > is true that for a shell script would already be possible to poke with the > > config file.. > > but wouldn't this be a bit too much of a security hole/open invitation for > > anybody, script or app to poke in the desktop configuration wit

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread David Edmundson
> On Oct. 19, 2015, 10:43 a.m., Marco Martin wrote: > > is true that for a shell script would already be possible to poke with the > > config file.. > > but wouldn't this be a bit too much of a security hole/open invitation for > > anybody, script or app to poke in the desktop configuration wit

Re: Review Request 125694: Use the correct old screen to shift panels to new screen

2015-10-19 Thread David Kahles
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125694/ --- (Updated Oct. 19, 2015, 11:10 a.m.) Status -- This change has been m

Re: Review Request 125693: Prevent changing the primary screen when it doesn't have a view yet

2015-10-19 Thread David Kahles
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125693/ --- (Updated Oct. 19, 2015, 11:10 a.m.) Status -- This change has been m

Re: Review Request 125696: Fix calculation of available screen region

2015-10-19 Thread David Kahles
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125696/ --- (Updated Oct. 19, 2015, 11:10 a.m.) Status -- This change has been m

Re: Review Request 125555: Port KAuth return code error checking in fontinst

2015-10-19 Thread David Edmundson
> On Oct. 19, 2015, 10:40 a.m., Martin Gräßlin wrote: > > kcms/kfontinst/dbus/FontInst.cpp, line 957 > > > > > > that's an async call, isn't it? So somehow I doubt it will work that > > way. no, this is KJob::e

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review87061 --- is true that for a shell script would already be possible to p

Re: Review Request 125555: Port KAuth return code error checking in fontinst

2015-10-19 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/12/#review87060 --- kcms/kfontinst/dbus/FontInst.cpp (line 957)

Minutes Monday Plasma Hangout

2015-10-19 Thread Sebastian Kügler
Present: sebas, bhushan, david, jens, jonathan, kai uwe, marco, martin g Date: 21 Oct, 2015 Bhushan: - I will open review request for lockscreen on mobile today - My issues with the "password always accepted" seems to be fixed auto magically yes - I also did some mobile IMG testing and release

Re: Review Request 125698: Fix Button minimumWidth property

2015-10-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125698/ --- (Updated Oct. 19, 2015, 10:31 a.m.) Status -- This change has been m

Re: Review Request 125678: Introduce User Switcher plasmoid

2015-10-19 Thread Thomas Pfeiffer
> On Oct. 19, 2015, 8:47 a.m., Marco Martin wrote: > > Nice! But I wonder if wouldn't be better to have the whole plasmoid as a > > scrolling list instead of just the users scrolling and not the rest, it > > looks a bit werird. I know that technically the user list and the other > > functions

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-19 Thread Marco Martin
> On Oct. 13, 2015, 12:12 p.m., Marco Martin wrote: > > I tried the script on some elements, doesn't work with all of the svgs but > > the result seems pretty good, > > here is the theme with the questionable "wonton soup" color theme > > http://imgur.com/g5tsNs6 > > > > so it seems feasible, m

Review Request 125699: switch to Noto font and dejavu sans font

2015-10-19 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125699/ --- Review request for Plasma. Repository: frameworkintegration Description

Re: Review Request 125678: Introduce User Switcher plasmoid

2015-10-19 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125678/#review87054 --- Nice! But I wonder if wouldn't be better to have the whole pla

Re: Review Request 125669: Remove redundant code

2015-10-19 Thread Marco Martin
> On Oct. 17, 2015, 9:58 a.m., Kai Uwe Broulik wrote: > > Btw what should be do with the "Show Dashboard" entry in the desktop > > toolbox? I mean, if you can actually use the toolbox you can probably see > > your desktop anyway, so we could remove it altogether? > > David Edmundson wrote: >

Re: 5.5 module review

2015-10-19 Thread Jonathan Riddell
On Mon, Oct 19, 2015 at 10:23:31AM +0200, Harald Sitter wrote: > oxygen-icons I'll probably call the tar for this oxygen-icons-plasma to allow it to have a smaller version number. Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail

Re: Review Request 125698: Fix Button minimumWidth property

2015-10-19 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125698/#review87052 --- Ship it! Ship It! - Marco Martin On Oct. 18, 2015, 11:29 p

Re: 5.5 module review

2015-10-19 Thread Harald Sitter
On Thu, Oct 15, 2015 at 6:37 PM, Jonathan Riddell wrote: > Does anyone else have anything they want included in Plasma 5.5? breeze-icons (going to be split from breeze sometime closer to freeze to not cause headache for artists) oxygen-icons ___ Plasma-