Re: Review Request 108992: Simple optimizations in SignalPlotter

2014-05-05 Thread Raul Fernandes
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/108992/ --- (Updated May 5, 2014, 10:54 p.m.) Status -- This change has been dis

Re: Plasma 5?

2014-05-05 Thread Aleix Pol
On Mon, May 5, 2014 at 7:25 PM, Martin Graesslin wrote: > On Monday 05 May 2014 17:56:04 Ivan Čukić wrote: > > Jens: > > > the best option is to simply go with "Plasma by KDE" > > > > and > > > > > "So what version am I running?" is "Plasma 2", "Plasma 5", > > > > +1 > > > > I don't mind the vers

Re: Multi-screen on Plasma Shell

2014-05-05 Thread Martin Klapetek
I wanted to test it out a bit as I have a multi-screen setup too, but plasma-shell crashes on startup for me - backtrace -> http://paste.kde.org/pscwpjjnp Also as I mentioned earlier, please run astyle on it before merging to have a consistent coding style in the files. Please don't ignore that.

Re: Plasma 5?

2014-05-05 Thread Jens Reuterberg
I hate to play the devils (or in this case Martin G's ;) ) advocate but these are the issues that will crop up: People love names to be able to say "I use umptybump and not bippitbop". This will make people want to know what version they have in the beginning of the process. The upside with the

Re: Plasma 5?

2014-05-05 Thread Ivan Čukić
I meant +1 for "Plasma by KDE", and when someone wants the version, he can say KDE^WPlasma 5 :) On 5 May 2014 19:04, Jens Reuterberg wrote: > Well to be honest the same can be said of "Plasma 2014.5" - I mean as > long as that name is kept as a form of technical definer and never ever > near ma

Re: Plasma 5?

2014-05-05 Thread Martin Graesslin
On Monday 05 May 2014 17:56:04 Ivan Čukić wrote: > Jens: > > the best option is to simply go with "Plasma by KDE" > > and > > > "So what version am I running?" is "Plasma 2", "Plasma 5", > > +1 > > I don't mind the version 5.x (though, I didn't mind any of the proposals) > > Wondering what is

Re: Plasma 5?

2014-05-05 Thread Jens Reuterberg
Well to be honest the same can be said of "Plasma 2014.5" - I mean as long as that name is kept as a form of technical definer and never ever near marketing and promo work. (So Ivan, no dancing on a beach in black and white while a voice whispers "between beauty and desire, between 1337 and n00

Multi-screen on Plasma Shell

2014-05-05 Thread Aleix Pol
Hi, As some of you know, I've been trying to polish the plasma shell usage on two screens. It's a use-case I hit a lot given that I have 2 screens both in the office and at home and I think it's really important to be on par with Plasma 1 on this area, at least, especially if we want to be appealin

Re: Breeze QML window decoration

2014-05-05 Thread Andrew Lake
On Mon, May 5, 2014 at 6:54 AM, Martin Gräßlin wrote: > > some feedback after running the deco for most of today: > * in maximized state there is still a round corner in top left and top > right > * I experience a slight flicker when mouse over buttons > * there's a config dialog (which looks like

Re: Plasma 5?

2014-05-05 Thread Ivan Čukić
Jens: > the best option is to simply go with "Plasma by KDE" and > "So what version am I running?" is "Plasma 2", "Plasma 5", +1 I don't mind the version 5.x (though, I didn't mind any of the proposals) Wondering what is Martin's stance on this since the year.month was his child. Cheers ___

Re: Plasma 5?

2014-05-05 Thread Jens Reuterberg
Personally (and I've said this before) from my point of view a name is either a) a technical definition b) a form of promotion As for technical reasoning behind either name - I have little or no opinion. This is not my field to piss in as it where. BUT for the usage of promo work the best optio

Re: Plasma 5?

2014-05-05 Thread Shantanu Tushar Jha
+1 And, its easier to "say" as well :) On Mon, May 5, 2014 at 8:51 PM, Marco Martin wrote: > On Monday 05 May 2014, Sebastian Kügler wrote: > > > The baseline, to use "Plasma" as the brand, and only refer to the version > > as a technicality should of course stay the same. > > > > Why do I thi

Re: Plasma 5?

2014-05-05 Thread Marco Martin
On Monday 05 May 2014, Sebastian Kügler wrote: > The baseline, to use "Plasma" as the brand, and only refer to the version > as a technicality should of course stay the same. > > Why do I think 5 is better than 2014.6, or .? After trying (with not much success) to use publicly the next/2014.06 s

Re: Plasma 5?

2014-05-05 Thread Martin Klapetek
On Mon, May 5, 2014 at 4:59 PM, Mark Gaiser wrote: > > I do wonder though, why didn't you folks decide this on the plasma > sprint earlier this year? > We did. That's where the original proposal came from. Cheers -- Martin Klapetek | KDE Developer __

Re: Plasma 5?

2014-05-05 Thread Mark Gaiser
On Mon, May 5, 2014 at 3:55 PM, Sebastian Kügler wrote: > Hi all, > > I am not happy with the 2014.6 name and naming scheme. There I said it. > > The reasons for this are multi-fold. First, and to me most importantly: It > feels awkward. Now that might be because it's new, but it also feels like n

Re: Locale settings in Plasma Next

2014-05-05 Thread Sebastian Kügler
On Sunday, February 23, 2014 20:10:31 John Layt wrote: > One question is when these changes to the envvars will get applied, and the > action required to apply them to the apps. I believe Gnome forces you to > log out and in again before the new envvars are applied. In KDE4 you could > simply re

Re: [plasma-devel] Plasma 5?

2014-05-05 Thread Jonathan Riddell
Plasma 5 certainly makes it a lot easier to name betas. It also means if the release date slips there's no need for a quick reversion. Frameworks want an extra beta so it may well slip. Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org http

Re: Plasma 5?

2014-05-05 Thread Eike Hein
On Monday 05 May 2014 15:55:27 Sebastian Kügler wrote: > Hi all, > > I am not happy with the 2014.6 name and naming scheme. There I said it. I'm not either. I think that going with the 5 scheme: (a) Provides a nice continuity for users, packagers and developers, who don't need to get used t

Re: Plasma 5?

2014-05-05 Thread Teo Mrnjavac
On Monday, May 05, 2014 15:55:27 Sebastian Kügler wrote: > Hi all, > > I am not happy with the 2014.6 name and naming scheme. There I said it. > > The reasons for this are multi-fold. First, and to me most importantly: It > feels awkward. Now that might be because it's new, but it also feels like

Re: Breeze QML window decoration

2014-05-05 Thread Martin Gräßlin
On Sunday 04 May 2014 22:28:23 Andrew Lake wrote: > Based on the design we came up with in the VDG forum, I completed a first > go at doing up a Aurorae QML window decoration. > > I added it to the 'working' branch of the Breeze repo. As far as I can tell > it works about as well as the Plastik QM

Plasma 5?

2014-05-05 Thread Sebastian Kügler
Hi all, I am not happy with the 2014.6 name and naming scheme. There I said it. The reasons for this are multi-fold. First, and to me most importantly: It feels awkward. Now that might be because it's new, but it also feels like no one else is going to understand it. My thinking goes towards a

Re: Breeze repo

2014-05-05 Thread Aleix Pol
On Mon, May 5, 2014 at 7:43 AM, Andrew Lake wrote: > I went ahead and added a 'working' branch and added the following: > > - Colors: This is the Breeze color scheme that has the both the style and > the window decoration colors. I think this is releasable, so if there are > no objections I'd lik

Re: Review Request 117894: Add a kscreenlocker_test test application

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

Re: Review Request 117894: Add a kscreenlocker_test test application

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

Re: Breeze repo

2014-05-05 Thread Marco Martin
On Monday 05 May 2014, Andrew Lake wrote: > Yay, thanks! > > One thing I noticed, does the top level cmake include the Breeze color > scheme? Both the controls style and the QML windec use the system color gah, forgot it. now should be ok :) -- Marco Martin

Re: Breeze repo

2014-05-05 Thread Andrew Lake
On May 5, 2014 5:55 AM, "Marco Martin" wrote: > ok, > so now everything is installed by the toplevel cmake, hopefully in the proper > place. Yay, thanks! One thing I noticed, does the top level cmake include the Breeze color scheme? Both the controls style and the QML windec use the system color

Re: Review Request 117894: Add a kscreenlocker_test test application

2014-05-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117894/#review57311 --- Ship it! Ship It! - David Edmundson On April 30, 2014, 9:4

Re: Breeze repo

2014-05-05 Thread Marco Martin
On Monday 05 May 2014, Andrew Lake wrote: > On May 5, 2014 4:32 AM, "Marco Martin" wrote: > > On Monday 05 May 2014, Andrew Lake wrote: > > > - Window Decoration: I added the newly implemented Aurorae QML window > > > decoration and the current Aurorae SVG theme (which has a some buttons > > > miss

Re: Review Request 117839: [kglobalaccel] Update X11 appTime from key press events

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

Re: Review Request 117839: [kglobalaccel] Update X11 appTime from key press events

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

i18n in Plasma [Was Re: Minutes Monday Plasma hangout]

2014-05-05 Thread Martin Gräßlin
On Monday 05 May 2014 13:10:44 Sebastian Kügler wrote: > - Discovered problems in Kwin and Plasma localization (to be discussed on > mailinglist) ok, so with frameworks i18n changed a bit. I highly recommend to read the documentation in [1]. Things to be done are: * set KLocalizedString::setAp

Re: Review Request 112208: KMix qml applet

2014-05-05 Thread Sebastian Kügler
> On March 22, 2014, 11:09 p.m., Mark Gaiser wrote: > > Is there still an intention on getting this in KDE 4.xx? > > Just wondering since it seems to be much better then the current kmix popup. > > Christian Esken wrote: > I also haven't heard about further development here. Diego as origina

Re: Breeze repo

2014-05-05 Thread Andrew Lake
On May 5, 2014 4:32 AM, "Marco Martin" wrote: > > On Monday 05 May 2014, Andrew Lake wrote: > > - Window Decoration: I added the newly implemented Aurorae QML window > > decoration and the current Aurorae SVG theme (which has a some buttons > > missing for now). > > I am seeing a "qml" folder with

Review Request 117995: [screenlocker] Add a unit test for KSldApp

2014-05-05 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117995/ --- Review request for Plasma and David Edmundson. Repository: plasma-workspa

Re: Breeze repo

2014-05-05 Thread Marco Martin
On Monday 05 May 2014, Andrew Lake wrote: > - Window Decoration: I added the newly implemented Aurorae QML window > decoration and the current Aurorae SVG theme (which has a some buttons > missing for now). I am seeing a "qml" folder with the actual aurorae decoration and an svg folder with grap

Re: Review Request 117839: [kglobalaccel] Update X11 appTime from key press events

2014-05-05 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117839/#review57299 --- Ship it! Code looks good. - Àlex Fiestas On April 28, 2014

Re: Fwd: [kconfig] src/core: Store app config file in ~/.config//rc

2014-05-05 Thread Sebastian Kügler
On Monday, May 05, 2014 11:13:31 Marco Martin wrote: > On Sunday 04 May 2014, David Faure wrote: > > please note this change in KConfig. > > > > I would recommend making sure your applications call > > app.setOrganizationDomain("kde.org") so that the config files go into the > > right subdir right

Re: Minutes Monday Plasma hangout

2014-05-05 Thread Sebastian Kügler
On Monday, May 05, 2014 12:30:15 Ivan Čukić wrote: > Ivan: > - Resource linking is almost back. A few kinks to work out. Will be merged > after the first release of kf5. > > Cheerio, > Ivan > > p.s. I'm not properly online to join in the hangout :/ Thanks for prenatal and proactive hijacking of

Re: Review Request 117839: [kglobalaccel] Update X11 appTime from key press events

2014-05-05 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117839/#review57298 --- maybe add kdeframeworks group? - Marco Martin On April 28,

Minutes Monday Plasma hangout

2014-05-05 Thread Ivan Čukić
Ivan: - Resource linking is almost back. A few kinks to work out. Will be merged after the first release of kf5. Cheerio, Ivan p.s. I'm not properly online to join in the hangout :/ ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.

Re: Review Request 117813: Make the system tray faster

2014-05-05 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117813/#review57296 --- Ship it! applets/systemtray/package/contents/ui/StatusNotifi

Re: Fwd: [kconfig] src/core: Store app config file in ~/.config//rc

2014-05-05 Thread Marco Martin
On Monday 05 May 2014, David Faure wrote: > > does KSharedConfig::openConfig("confignamerc") now tries in ~/.config or > > in ~/.config/kde.org ? (supposing the domain is set) > > I reverted the change for now. ok, so i will delay any local change in plasma > Let's discuss whether the alternativ

Re: Review Request 117946: Track screen changes in the containment when the containment is inside an applet

2014-05-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117946/ --- (Updated May 5, 2014, 9:39 a.m.) Status -- This change has been mark

Re: Review Request 117946: Track screen changes in the containment when the containment is inside an applet

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

Re: Fwd: [kconfig] src/core: Store app config file in ~/.config//rc

2014-05-05 Thread David Faure
On Monday 05 May 2014 11:13:31 Marco Martin wrote: > On Sunday 04 May 2014, David Faure wrote: > > Hello, > > > > please note this change in KConfig. > > > > I would recommend making sure your applications call > > app.setOrganizationDomain("kde.org") so that the config files go into the > > righ

Re: Fwd: [kconfig] src/core: Store app config file in ~/.config//rc

2014-05-05 Thread Marco Martin
On Sunday 04 May 2014, David Faure wrote: > Hello, > > please note this change in KConfig. > > I would recommend making sure your applications call > app.setOrganizationDomain("kde.org") so that the config files go into the > right subdir right now. Otherwise you'll face migration issues later wh

Re: Re: Breeze QML window decoration

2014-05-05 Thread Martin Gräßlin
On Monday 05 May 2014 10:33:33 Marco Martin wrote: > On Monday 05 May 2014, Martin Gräßlin wrote: > > On Sunday 04 May 2014 22:28:23 Andrew Lake wrote: > > > Based on the design we came up with in the VDG forum, I completed a > > > first > > > go at doing up a Aurorae QML window decoration. > > >

Re: Breeze QML window decoration

2014-05-05 Thread Marco Martin
On Monday 05 May 2014, Martin Gräßlin wrote: > On Sunday 04 May 2014 22:28:23 Andrew Lake wrote: > > Based on the design we came up with in the VDG forum, I completed a first > > go at doing up a Aurorae QML window decoration. > > > > I added it to the 'working' branch of the Breeze repo. As far a

Re: Breeze repo

2014-05-05 Thread Marco Martin
On Monday 05 May 2014, Andrew Lake wrote: > I went ahead and added a 'working' branch and added the following: > later i'll check and see to make everything built and installed -- Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https:/

Re: Fwd: [kconfig] src/core: Store app config file in ~/.config//rc

2014-05-05 Thread David Faure
On Monday 05 May 2014 08:29:07 Martin Gräßlin wrote: > On Monday 05 May 2014 08:17:49 David Faure wrote: > > On Monday 05 May 2014 07:25:37 Martin Gräßlin wrote: > > > On Sunday 04 May 2014 23:53:40 David Faure wrote: > > > > Hello, > > > > > > > > please note this change in KConfig. > > > > > >

Re: Review Request 117900: Cleanup of screenlocker

2014-05-05 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117900/ --- (Updated May 5, 2014, 9 a.m.) Review request for Plasma and David Edmunds