Re: Setup build of kdeplasma-addons on CI system

2013-12-29 Thread Bhushan Shah
Hello! On Sun, Dec 29, 2013 at 6:49 PM, Sebastian Kügler wrote: > So please everybody, tackle bugs, do not port code that isn't really needed > for a basic functional system. It might be more fun, or maybe more rewarding > to pick the low-hanging fruit, but it's not what we need right now. Well,

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Marco Martin
On Sunday 29 December 2013, Sebastian Kügler wrote: > > formfactor does exist, but there aren't differences between being in > > apanel or in a systray... > > hmm, so when in a systray would make sense a "constrained" formfactor or > > something like that, ie not able to expand neither horizontall

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Sebastian Kügler
On Sunday, December 29, 2013 16:35:31 Marco Martin wrote: > > - (How) Can I determin whether the plasmoid is collapsed in systray or in > > a > > panel? Currently when there are eg. two batteries, the battery monitor > > shows two icons cramped next to eath other as it doesn't know it's only > > su

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Sebastian Kügler
On Sunday, December 29, 2013 14:37:25 Kai Uwe Broulik wrote: > > plasmoid.location should work, if it doesn't, let me know. (I haven't > > checked for the system tray, but it would be an easy fix.) > > Haven't tried so far but I guess I need to accomodate to the enum changes > using PlasmaCore.Ty

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Marco Martin
On Sunday 29 December 2013, Kai Uwe Broulik wrote: > > that said, i don't think should be something for the plasmoid to care to, > > that seems a thing that should be present in all plasmoids > Huh? I was talking about having the PM switch and brightness slider as > close to the panel as possible

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Kai Uwe Broulik
Hi, > yes, have to use PlasmaCore.Types prefix, everything in plasma.h is > available under there Okay. > in the systray having something not square breaks in many edge cases.. > so makes me think that we should really introduce a new formfactor for > square stuff, so the battery could behave dif

Review Request 114681: Removed all X-Plasma-DefaultSize declarations

2013-12-29 Thread Benjamin Kaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114681/ --- Review request for Plasma and Heena Mahour. Repository: kde-workspace D

Re: Review Request 114682: Remove 'X-Plasma-DefaultSize' declarations in metadata.desktop files

2013-12-29 Thread Benjamin Kaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114682/ --- (Updated Dec. 27, 2013, 1:56 p.m.) Review request for Plasma, Heena Mahou

Review Request 114682: Remove 'X-Plasma-DefaultSize' declarations in metadata.desktop files

2013-12-29 Thread Benjamin Kaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114682/ --- Review request for Plasma, Heena Mahour and Sebastian Kügler. Repository:

Re: Review Request 114682: Remove 'X-Plasma-DefaultSize' declarations in metadata.desktop files

2013-12-29 Thread Benjamin Kaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114682/ --- (Updated Dec. 27, 2013, 1:59 p.m.) Review request for Plasma, Heena Mahou

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Marco Martin
On Sunday 29 December 2013, Kai Uwe Broulik wrote: > Hi, > > > plasmoid.location should work, if it doesn't, let me know. (I haven't > > checked for the system tray, but it would be an easy fix.) > > Haven't tried so far but I guess I need to accomodate to the enum changes > using PlasmaCore.Type

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Marco Martin
On Saturday 28 December 2013, Kai Uwe Broulik wrote: > Hi there, > > first of all, I really like all the polish you put into Plasma 2, > especially system tray and the calendar :-) > > I have a few questions regarding the battery monitor, however: > - (How) Can I determin whether the panel the sy

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Kai Uwe Broulik
Hi, > plasmoid.location should work, if it doesn't, let me know. (I haven't > checked for the system tray, but it would be an easy fix.) Haven't tried so far but I guess I need to accomodate to the enum changes using PlasmaCore.Types for location then? > Please file a bug, this needs more detail

Re: Students to work on Plasma

2013-12-29 Thread Sebastian Kügler
On Saturday, December 28, 2013 09:21:50 Martin Graesslin wrote: > On Friday 27 December 2013 19:47:36 Artur Souza wrote: > > You probably have seen my post on planet.kde > > (http://blog.morpheuz.cc/26/12/2013/open-academy-and-kde/) about > > OpenAcademy and the possibility of sponsoring students

Re: Setup build of kdeplasma-addons on CI system

2013-12-29 Thread Sebastian Kügler
Hi, On Saturday, December 28, 2013 10:03:49 Bhushan Shah wrote: > Currently I am porting stuffs in kdeplasma-addons but as I see there > is no build of kdeplasma-addons on CI system. If someone can setup it > it will be very good. I know there are not much things in repo but it > is moving forward

Re: Battery Monitor on Plasma 2

2013-12-29 Thread Sebastian Kügler
Hey, On Saturday, December 28, 2013 17:13:24 Kai Uwe Broulik wrote: > first of all, I really like all the polish you put into Plasma 2, > especially system tray and the calendar Thanks. :) These layout cleanups do place a couple of extra restrictions (hard and soft) on the widgets in the panel

Re: Setup build of kdeplasma-addons on CI system

2013-12-29 Thread Ben Cooksley
On Sat, Dec 28, 2013 at 5:36 PM, Bhushan Shah wrote: > Hello! Hi, > > On Sat, Dec 28, 2013 at 10:05 AM, Ben Cooksley wrote: >> >> Please see http://build.kde.org/job/kdeplasma-addons_master/ and >> http://build.kde.org/job/kdeplasma-addons_stable/ > > Sorry I should have mentioned about framewo