Review Request 116626: Fix for "Bug 325012 - Playlist reordering works only in one direction"

2014-03-05 Thread Ashish Madeti
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116626/ --- Review request for Plasma, Shantanu Tushar and Sinny Kumari. Repository:

Re: Review Request 116625: Oxygen as default font

2014-03-05 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116625/#review52225 --- I don't like that the fonts are copied as binaries into our re

Re: Review Request 116625: Oxygen as default font

2014-03-05 Thread Martin Gräßlin
> On March 6, 2014, 3:06 a.m., Hrvoje Senjan wrote: > > startkde.cmake, line 76 > > > > > > Why not just change the defaults in frameworkintegration instead of > > writing directly into users settings? I also do

Re: Review Request 116620: Bug 322820 - Playlist is not reachable from home screen

2014-03-05 Thread Ashish Madeti
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116620/ --- (Updated March 6, 2014, 5:20 a.m.) Review request for Plasma, Akshay Rata

Re: Review Request 116620: Bug 322820 - Playlist is not reachable from home screen

2014-03-05 Thread Sinny Kumari
> On March 6, 2014, 5:07 a.m., Shantanu Tushar wrote: > > A better way would be to just show the playlist instead of showing the > > controller, because the controller is useless when nothing is playing. See > > if it works good if you simply show the playlist. Yes! This approach will be more

Re: Review Request 116620: Bug 322820 - Playlist is not reachable from home screen

2014-03-05 Thread Shantanu Tushar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116620/#review5 --- A better way would be to just show the playlist instead of sho

Re: Review Request 116625: Oxygen as default font

2014-03-05 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116625/#review52219 --- startkde.cmake

Review Request 116625: Oxygen as default font

2014-03-05 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116625/ --- Review request for Plasma. Repository: kde-workspace Description --

Re: Plasma Next alpha release engineering bits

2014-03-05 Thread Albert Astals Cid
El Dimecres, 5 de març de 2014, a les 13:10:12, Sebastian Kügler va escriure: > Hi, > > As we're planning to release the first alpha of Plasma Next next week, I'd > like to go over some details that need discussing. > > - promo preparations > - tagging / tarballing: can happen anytime on Thursday

Re: Review Request 116620: Bug 322820 - Playlist is not reachable from home screen

2014-03-05 Thread Ashish Madeti
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116620/ --- (Updated March 5, 2014, 7:38 p.m.) Review request for Plasma, Akshay Rata

Re: Review Request 116620: Bug 322820 - Playlist is not reachable from home screen

2014-03-05 Thread Sinny Kumari
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116620/#review52206 --- Good work! Neat and best solution as of whatever we came acros

Re: Review Request 116621: New activity switcher

2014-03-05 Thread Ivan Čukić
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116621/ --- (Updated March 5, 2014, 7:07 p.m.) Status -- This change has been ma

[kde-runtime/frameworks] kioexec: This basically makes kioexec work, by finishing the port

2014-03-05 Thread Àlex Fiestas
Git commit 84317812ee91ff504644aa999d2e681266b412a1 by Àlex Fiestas. Committed on 05/03/2014 at 18:54. Pushed by afiestas into branch 'frameworks'. This basically makes kioexec work, by finishing the port It is quite embarassing from our part (if you ask me) that we had this in this state. It cle

Review Request 116621: New activity switcher

2014-03-05 Thread Ivan Čukić
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116621/ --- Review request for Plasma, Marco Martin and Sebastian Kügler. Repository:

Re: Can I remove kde-workspace/plasma/desktop/qmlpackages/blank

2014-03-05 Thread Ivan Čukić
> so let's just kill it. > Ivan. do you still need it for tests? I guess no. Until we have a second shell (active or whatever) I don't really have anything to do there. -- Make your code readable. Pretend the next person who looks at your code is a psychopath and they know where you live. --

Re: Can I remove kde-workspace/plasma/desktop/qmlpackages/blank

2014-03-05 Thread Marco Martin
On Wednesday 05 March 2014 18:50:14 Aleix Pol wrote: > On Wed, Mar 5, 2014 at 6:04 PM, Marco Martin wrote: > > On Wednesday 05 March 2014 17:47:41 David Edmundson wrote: > > > AFAIK it's unused, and was just a clone for ivan's testing shell > > > switching testing > > > > > > We probably don't wa

Re: Can I remove kde-workspace/plasma/desktop/qmlpackages/blank

2014-03-05 Thread Aleix Pol
On Wed, Mar 5, 2014 at 6:04 PM, Marco Martin wrote: > On Wednesday 05 March 2014 17:47:41 David Edmundson wrote: > > AFAIK it's unused, and was just a clone for ivan's testing shell > > switching testing > > > > We probably don't want it in the release. > yes please. > > maybe moving it in exam

Review Request 116620: Bug 322820 - Playlist is not reachable from home screen

2014-03-05 Thread Ashish Madeti
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116620/ --- Review request for Plasma, Akshay Ratan, Shantanu Tushar, Sinny Kumari, and

Re: Can I remove kde-workspace/plasma/desktop/qmlpackages/blank

2014-03-05 Thread Ivan Čukić
> maybe moving it in examples? +1 -- The problem with object-oriented languages is they’ve got all this implicit environment that they carry around with them. You wanted a banana but what you got was a gorilla holding the banana. -- Joe Armstrong _

Re: Can I remove kde-workspace/plasma/desktop/qmlpackages/blank

2014-03-05 Thread Marco Martin
On Wednesday 05 March 2014 17:47:41 David Edmundson wrote: > AFAIK it's unused, and was just a clone for ivan's testing shell > switching testing > > We probably don't want it in the release. maybe moving it in examples? -- Marco Martin ___ Plasma-dev

Can I remove kde-workspace/plasma/desktop/qmlpackages/blank

2014-03-05 Thread David Edmundson
AFAIK it's unused, and was just a clone for ivan's testing shell switching testing We probably don't want it in the release. David ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 116618: Port FrameSVGItem to paint directly

2014-03-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116618/#review52097 --- This review has been submitted with commit 6599c2508738d82989

Re: Review Request 116618: Port FrameSVGItem to paint directly

2014-03-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116618/ --- (Updated March 5, 2014, 4:23 p.m.) Status -- This change has been ma

Re: Review Request 116618: Port FrameSVGItem to paint directly

2014-03-05 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116618/#review52096 --- Ship it! Ship It! - Marco Martin On March 5, 2014, 3:20 p.

Re: Move plasmate to git-next?

2014-03-05 Thread Giorgos Tsiapaliokas
Since nobody disagrees I will make the request to the sysadmins. I will inform you again when the move is completed. -- Giorgos Tsiapaliokas (terietor) terietor.org ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listin

Re: Review Request 116618: Port FrameSVGItem to paint directly

2014-03-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116618/ --- (Updated March 5, 2014, 3:20 p.m.) Review request for Plasma. Repositor

Review Request 116618: Port FrameSVGItem to paint directly

2014-03-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116618/ --- Review request for Plasma. Repository: plasma-framework Description ---

Re: Review Request 116614: Reduce glitches when moving panels around

2014-03-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116614/ --- (Updated March 5, 2014, 3:09 p.m.) Status -- This change has been ma

Re: Review Request 116614: Reduce glitches when moving panels around

2014-03-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116614/#review52089 --- This review has been submitted with commit f8be8ab0fddbbf684d

Re: Review Request 116614: Reduce glitches when moving panels around

2014-03-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116614/#review52090 --- This review has been submitted with commit 78c6e452dfc2bbbdec

[plasma-shell] [Bug 328651] Theme is not updated when composite changes

2014-03-05 Thread Alex Fiestas
https://bugs.kde.org/show_bug.cgi?id=328651 --- Comment #2 from Alex Fiestas --- Created attachment 85423 --> https://bugs.kde.org/attachment.cgi?id=85423&action=edit image showing composite with opaque theme. -- You are receiving this mail because: You are on the CC list for the bug. ___

[plasma-shell] [Bug 328651] Theme is not updated when composite changes

2014-03-05 Thread Alex Fiestas
https://bugs.kde.org/show_bug.cgi?id=328651 Alex Fiestas changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED

Re: Review Request 116554: Implementation of autoplay when the PlayAll button in pressed

2014-03-05 Thread Sujith Haridasan
> On March 3, 2014, 6:01 a.m., Sujith Haridasan wrote: > > Harshit, thanks for picking up the bug and I appreciate your effort. I > > would like to know every single step you had done as test case. I am pretty > > sure this is not a one liner fix. Only when the playlist frame gets into > > the

Re: Review Request 116614: Reduce glitches when moving panels around

2014-03-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116614/#review52087 --- Ship it! Makes sense. src/shell/panelview.cpp

Re: Bug 322820 - Playlist is not reachable from home screen

2014-03-05 Thread Shantanu Tushar Jha
On Wed, Mar 5, 2014 at 2:53 PM, Ashish Madeti wrote: > Hi Shantanu, > If you think the 1st solution bring unnecessary clutter to the home screen > , we can make it auto-hide like on the now playing screen. > > Another way to implement Solution 1, like if we presently click the empty > area in the

Re: Oxygen as default font

2014-03-05 Thread Sebastian Kügler
On Wednesday, March 05, 2014 13:34:36 Jonathan Riddell wrote: > On Wed, Mar 05, 2014 at 02:06:43PM +0100, Marco Martin wrote: > > On Wednesday 05 March 2014 13:36:33 Sebastian Kügler wrote: > > > picked up by fontconfig, though. This means, installing it to a non-/usr > > > prefix will likely mean

Re: Oxygen as default font

2014-03-05 Thread Sebastian Kügler
On Wednesday, March 05, 2014 14:16:06 Kai Uwe Broulik wrote: > Yes I've been changing the default font to Oxygen an all machines I have set > up. > > However, we should also enable font anti-aliasing and subpixel hinting (I > use Medium) to make the font really shine. That's actually part of my c

Re: Review Request 115922: Drop taskbar thumbnail effect

2014-03-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115922/#review52079 --- This review has been submitted with commit 53f83bfddc6209a36b

Re: Review Request 115922: Drop taskbar thumbnail effect

2014-03-05 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115922/ --- (Updated March 5, 2014, 1:46 p.m.) Status -- This change has been ma

Review Request 116614: Reduce glitches when moving panels around

2014-03-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116614/ --- Review request for Plasma. Repository: plasma-framework Description ---

Re: Oxygen as default font

2014-03-05 Thread Jonathan Riddell
On Wed, Mar 05, 2014 at 02:06:43PM +0100, Marco Martin wrote: > On Wednesday 05 March 2014 13:36:33 Sebastian Kügler wrote: > > picked up by fontconfig, though. This means, installing it to a non-/usr > > prefix will likely mean the font is never going to be found for the user. > > There's a trick

Re: Review Request 116613: Fix dis/connects in PanelView

2014-03-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116613/#review52075 --- This review has been submitted with commit 017387a0b279976ba6

Re: Review Request 116613: Fix dis/connects in PanelView

2014-03-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116613/ --- (Updated March 5, 2014, 1:29 p.m.) Status -- This change has been ma

Re: Oxygen as default font

2014-03-05 Thread Sebastian Kügler
On Wednesday, March 05, 2014 14:06:43 Marco Martin wrote: > > picked up by fontconfig, though. This means, installing it to a non-/usr > > prefix will likely mean the font is never going to be found for the user. > > There's a trick to fix this: fonts are picked up from XDG_DATA_HOME/fonts > > and

Re: Oxygen as default font

2014-03-05 Thread Kai Uwe Broulik
Hi, Yes I've been changing the default font to Oxygen an all machines I have set up. However, we should also enable font anti-aliasing and subpixel hinting (I use Medium) to make the font really shine. Cheers, Kai Uwe Am 05.03.2014 13:36 schrieb =?ISO-8859-1?Q?Sebastian_K=FCgler?= : > > Hey,

Re: Review Request 116613: Fix dis/connects in PanelView

2014-03-05 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116613/#review52074 --- Ship it! ouch, i missed it when the signal was renamed :/ tha

Re: Oxygen as default font

2014-03-05 Thread Marco Martin
On Wednesday 05 March 2014 13:36:33 Sebastian Kügler wrote: > picked up by fontconfig, though. This means, installing it to a non-/usr > prefix will likely mean the font is never going to be found for the user. > There's a trick to fix this: fonts are picked up from XDG_DATA_HOME/fonts > and ~/.fon

Re: Minutes Monday Plasma hangout

2014-03-05 Thread Sebastian Kügler
On Wednesday, March 05, 2014 13:38:04 Marco Martin wrote: > > I have even thought of making it a separate executable so that people can > > switch activities even when plasma crashes, but that would take much more > > memory than if it was inside plasma-shell. Some parts of it need to be > > always

Re: Review Request 116613: Fix dis/connects in PanelView

2014-03-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116613/#review52070 --- Ship it! Ship It! - David Edmundson On March 5, 2014, 12:4

Review Request 116613: Fix dis/connects in PanelView

2014-03-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116613/ --- Review request for Plasma. Repository: plasma-framework Description ---

Re: Minutes Monday Plasma hangout

2014-03-05 Thread Marco Martin
On Monday 03 March 2014 15:20:03 Ivan Čukić wrote: > Ivan Cukic: > - Keyboard support for activity switcher. > - Some fixes/improvements to kactivities/imports (did anyone notice we are > in master now :) finally) > > Questions: > - The time to merge the switcher into plasma is approaching. I'm t

Oxygen as default font

2014-03-05 Thread Sebastian Kügler
Hey, I've been trying to change our default font to Oxygen, with some mixed success so far. Let me explain a bit: There's two components to this, we need to set the default font to Oxygen, and we need to install the Oxygen font. Default Setting The easiest way to do this is to write out a kdeg

Re: Plasma Next alpha release engineering bits

2014-03-05 Thread Martin Gräßlin
On Wednesday 05 March 2014 13:10:12 Sebastian Kügler wrote: > Hi, > > As we're planning to release the first alpha of Plasma Next next week, I'd > like to go over some details that need discussing. > > - promo preparations > - tagging / tarballing: can happen anytime on Thursday next week > - smo

Plasma Next alpha release engineering bits

2014-03-05 Thread Sebastian Kügler
Hi, As we're planning to release the first alpha of Plasma Next next week, I'd like to go over some details that need discussing. - promo preparations - tagging / tarballing: can happen anytime on Thursday next week - smoke-testing of packages - rolling out packages on Monday, or sooner dependin

Re: Review Request 116075: Provide an implementation for QPlatformSystemTrayIcon

2014-03-05 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116075/ --- (Updated March 5, 2014, 11:49 a.m.) Status -- This change has been m

Re: Review Request 116075: Provide an implementation for QPlatformSystemTrayIcon

2014-03-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116075/#review52056 --- This review has been submitted with commit a6c75f16514a2f8bb5

Re: Review Request 116075: Provide an implementation for QPlatformSystemTrayIcon

2014-03-05 Thread Kevin Ottens
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116075/#review52055 --- Ship it! Ship It! - Kevin Ottens On March 5, 2014, 11:14 a

Re: Minutes Monday Plasma hangout

2014-03-05 Thread Sebastian Kügler
On Monday, March 03, 2014 15:20:03 Ivan Čukić wrote: > Ivan Cukic: > - Keyboard support for activity switcher. > - Some fixes/improvements to kactivities/imports (did anyone notice we are > in master now finally) > > Questions: > - The time to merge the switcher into plasma is approaching. I'm

Re: Review Request 116075: Provide an implementation for QPlatformSystemTrayIcon

2014-03-05 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116075/ --- (Updated March 5, 2014, 12:14 p.m.) Review request for KDE Frameworks, Pl

Re: Bug 322820 - Playlist is not reachable from home screen

2014-03-05 Thread Ashish Madeti
Hi Shantanu, If you think the 1st solution bring unnecessary clutter to the home screen , we can make it auto-hide like on the now playing screen. Another way to implement Solution 1, like if we presently click the empty area in the top and there is a media player instance it goes to it, we can ma

Re: Review Request 116599: Set minimum version of Wayland to 1.2

2014-03-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116599/#review52030 --- This review has been submitted with commit d61b0e229fb429d515

Re: Review Request 116599: Set minimum version of Wayland to 1.2

2014-03-05 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116599/ --- (Updated March 5, 2014, 9:02 a.m.) Status -- This change has been ma