Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126221/#review89070 --- I'm glad that you found a solution which works just in time fo

Jenkins-kde-ci: plasma-workspace Plasma-5.5 stable-kf5-qt5 » Linux,gcc - Build # 4 - Still Failing!

2015-12-02 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-workspace%20Plasma-5.5%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/4/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 03 Dec 2015 04:12:09 + Build duration: 2 min 3 sec CHANGE SET Revision 4a6f7db0018a2a7366ac7

Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126221/ --- (Updated Dec. 3, 2015, 4:11 a.m.) Status -- This change has been mar

Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread Bhushan Shah
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126221/#review89068 --- Ship it! This fixes bug for me :) - Bhushan Shah On Dec. 3

[Plasma Workspace Wallpapers] [Bug 354379] Wallpaper fails to rotate 90 on dual screen with external scr. on left. Stays horiz. and spills over onto right scr.

2015-12-02 Thread Martin Klapetek via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=354379 Martin Klapetek changed: What|Removed |Added CC||mklape...@kde.org Status|UNCONFIR

Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread Martin Klapetek
> On Dec. 3, 2015, 12:36 a.m., Mark Gaiser wrote: > > I had this issue for quite a while! > > It's in bug #349669, i have high hopes that this change fixes my case as > > well :) > > > > Will try this out somewhere this weekend. No, unfortunately this will not fix this issue, but it is related

Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126221/#review89064 --- Ship it! I had this issue for quite a while! It's in bug #349

[Plasma Workspace Wallpapers] [Bug 354379] Wallpaper fails to rotate 90 on dual screen with external scr. on left. Stays horiz. and spills over onto right scr.

2015-12-02 Thread PhillB via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=354379 --- Comment #15 from PhillB --- Latest update to Fedora 23 seems to have fixed my issue. Thanks. Please close. -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mail

Re: Review Request 126166: use stylesheets in breeze icons

2015-12-02 Thread andreas kainz
> On Nov. 26, 2015, 5 p.m., Marco Martin wrote: > > *please* VDG people, take a look at this asap. I already seen that new > > changes have been done to icons in master, which means that i'll have to > > redo this from scratch. > > Uri Herrera wrote: > This needs to be documented first. Pr

Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126221/#review89061 --- Ship it! Ship It! - David Edmundson On Dec. 2, 2015, 7:32

Re: Review Request 126166: use stylesheets in breeze icons

2015-12-02 Thread andreas kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126166/#review89060 --- Ship it! Ship It! - andreas kainz On Nov. 25, 2015, 1:54 p

Re: Review Request 126166: use stylesheets in breeze icons

2015-12-02 Thread andreas kainz
> On Nov. 26, 2015, 5 p.m., Marco Martin wrote: > > *please* VDG people, take a look at this asap. I already seen that new > > changes have been done to icons in master, which means that i'll have to > > redo this from scratch. > > Uri Herrera wrote: > This needs to be documented first. Pr

Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126221/ --- (Updated Dec. 2, 2015, 8:32 p.m.) Review request for Plasma. Changes --

Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread Martin Klapetek
> On Dec. 2, 2015, 5:17 p.m., David Edmundson wrote: > > applets/notifications/lib/notificationsapplet.cpp, line 46 > > > > > > what about when it's just "Left" or "Right" ? It's not, ever. http://paste.opensuse.

Re: Review Request 126126: kicker: always show arrows for items with children

2015-12-02 Thread Pino Toscano
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126126/ --- (Updated Dec. 2, 2015, 5:55 p.m.) Status -- This change has been mar

Re: 5.5 video

2015-12-02 Thread Martin Klapetek
On Wed, Dec 2, 2015 at 11:18 AM, Eike Hein wrote: > > > On 12/02/2015 03:05 PM, Jonathan Riddell wrote: > > Lucas has made a 5.5 video and is after comments for any changes > > > > https://youtu.be/rYUqgFiY5Go > > * The 3D titles as well as any on screen type should use > type faces from the Br

Re: 5.5 video

2015-12-02 Thread Eike Hein
On 12/02/2015 03:05 PM, Jonathan Riddell wrote: > Lucas has made a 5.5 video and is after comments for any changes > > https://youtu.be/rYUqgFiY5Go * The 3D titles as well as any on screen type should use type faces from the Breeze visual design * The colors should be from the Breeze palette

Re: Review Request 126221: Rework the notifications positioning a bit

2015-12-02 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126221/#review89049 --- applets/notifications/lib/notificationsapplet.cpp (line 46) <

Review Request 126221: Rework the notifications positioning a bit

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

Re: 5.5 video

2015-12-02 Thread Alfonso Hernandez
excellent, we chose an excellent desktop to ours and to participate in the developer of the same. I like :) I like for people to know our distribution based on Arch Linux. Coming soon see the light for download. Alfonso Hernandez CEOPhone: +573057496424Skype: Ponchale EasyLabs S.A.S El Mié

Re: 5.5 video

2015-12-02 Thread Martin Klapetek
On Wed, Dec 2, 2015 at 9:05 AM, Jonathan Riddell wrote: > Lucas has made a 5.5 video and is after comments for any changes > > https://youtu.be/rYUqgFiY5Go Is this going to be an official video or something like that? Cheers -- Martin Klapetek | KDE Developer _

Re: 5.5 video

2015-12-02 Thread Marco Martin
On Wed, Dec 2, 2015 at 3:05 PM, Jonathan Riddell wrote: > Lucas has made a 5.5 video and is after comments for any changes > > https://youtu.be/rYUqgFiY5Go > It seems really well done! only things i'm not sure is the "powered by KDE" text, and in general that 3d effect for the titles -- Marco Mar

Re: Review Request 126016: fix: properly recognise Plasma 5 KCM modules (wmClass=kcmshell5)

2015-12-02 Thread Johan Ouwerkerk
> On Nov. 11, 2015, 7:14 a.m., Martin Gräßlin wrote: > > personal comment from X world: this is horrible, horrible ;-) What we > > should try is to make the desktop file available to the window. With KF > > 5.16 we will have all that's needed available. Let's try to improve this in > > Plasma

5.5 video

2015-12-02 Thread Jonathan Riddell
Lucas has made a 5.5 video and is after comments for any changes https://youtu.be/rYUqgFiY5Go Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 126216: Do not produce negative struts on switching screens

2015-12-02 Thread Jan Kundrát
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126216/ --- (Updated Dec. 2, 2015, 12:34 p.m.) Status -- This change has been ma

Re: Review Request 126126: kicker: always show arrows for items with children

2015-12-02 Thread Eike Hein
> On Nov. 30, 2015, 1:50 p.m., Marco Martin wrote: > > ping? what's the status of this? > > Eike Hein wrote: > I was waiting for Pino, but here's a shot of the proposed variant: > > ![All arrows everywhere](http://i.imgur.com/FwAmgGZ.png) > > Marco Martin wrote: > it looks aa b

Re: Review Request 126166: use stylesheets in breeze icons

2015-12-02 Thread Marco Martin
> On Nov. 26, 2015, 5 p.m., Marco Martin wrote: > > *please* VDG people, take a look at this asap. I already seen that new > > changes have been done to icons in master, which means that i'll have to > > redo this from scratch. > > Uri Herrera wrote: > This needs to be documented first. Pr

Re: Review Request 126216: Do not produce negative struts on switching screens

2015-12-02 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126216/#review89038 --- Ship it! - David Edmundson On Dec. 2, 2015, 11:29 a.m., Jan

Re: Review Request 126216: Do not produce negative struts on switching screens

2015-12-02 Thread Jan Kundrát
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126216/ --- (Updated Dec. 2, 2015, 11:29 a.m.) Review request for Plasma and Martin G

Re: Review Request 126166: use stylesheets in breeze icons

2015-12-02 Thread andreas kainz
> On Nov. 26, 2015, 5 p.m., Marco Martin wrote: > > *please* VDG people, take a look at this asap. I already seen that new > > changes have been done to icons in master, which means that i'll have to > > redo this from scratch. > > Uri Herrera wrote: > This needs to be documented first. Pr

Re: Review Request 126216: Do not produce negative struts on switching screens

2015-12-02 Thread David Edmundson
> On Dec. 1, 2015, 10:28 p.m., David Edmundson wrote: > > shell/panelview.cpp, line 918 > > > > > > This basically says: > > if the whole desktop workarea is null assume we only have one screen > > > >

Re: Review Request 126216: Do not produce negative struts on switching screens

2015-12-02 Thread Jan Kundrát
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126216/ --- (Updated Dec. 2, 2015, 10:53 a.m.) Review request for Plasma and Martin G

Re: Review Request 126216: Do not produce negative struts on switching screens

2015-12-02 Thread Jan Kundrát
> On Dec. 1, 2015, 10:28 p.m., David Edmundson wrote: > > shell/panelview.cpp, line 918 > > > > > > This basically says: > > if the whole desktop workarea is null assume we only have one screen > > > >

Re: Review Request 126216: Do not produce negative struts on switching screens

2015-12-02 Thread Jan Kundrát
> On Dec. 1, 2015, 11:30 p.m., Aleix Pol Gonzalez wrote: > > Looking at the documentation, it seems like it should never return invalid > > (at least if ::geometry also isn't invalid). > > > > I'll add some debug locally and try to reproduce the issue. Thanks for > > looking into this! For th

Re: Review Request 126203: Disable ptrace for kcheckpass and the greeter

2015-12-02 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126203/ --- (Updated Dec. 2, 2015, 8:11 a.m.) Status -- This change has been mar