Re: Review Request 118063: New Formats KCM

2014-05-12 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118063/ --- (Updated May 13, 2014, 2:15 a.m.) Review request for Plasma and John Layt

Re: Review Request 118063: New Formats KCM

2014-05-12 Thread Sebastian Kügler
> On May 11, 2014, 5:59 p.m., John Layt wrote: > > kcms/formats/kcmformats.cpp, line 204 > > > > > > Why are you writing all these entries here? If all they have set is > > the global then all we need to export

Re: Where we are now... (VDG)

2014-05-12 Thread Andrew Lake
On Mon, May 12, 2014 at 1:22 AM, Marco Martin wrote: > so of all that, let's see what we have, what we don't and how we can make > shippable > > * plasma theme: check > * wallpaper: once is ready, where we do put it? > * widget style: we have the qtcurve settings, i failed as well to contact > the

Re: Breeze QML window decoration

2014-05-12 Thread Andrew Lake
On Mon, May 5, 2014 at 6:54 AM, 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.

Re: Qt Quick Controls style

2014-05-12 Thread Andrew Lake
Oh and the style is now officially housed in the Breeze project repo: g...@git.kde.org:breeze.git Thanks much On Mon, May 12, 2014 at 7:43 AM, Jan Grulich wrote: > On Monday 12 of May 2014 16:32 David Edmundson wrote: > > On Mon, May 12, 2014 at 3:49 PM, Jan Grulich > wrote: > > > I just tes

Re: Qt Quick Controls style

2014-05-12 Thread Andrew Lake
Thanks for testing and reporting this Jan. The style needs to be tested like this to uncover any issues and so that we'll eventually have a fully functional, first class Qt Quick Controls style. Thanks again! Andrew On Mon, May 12, 2014 at 7:43 AM, Jan Grulich wrote: > On Monday 12 of May 201

Re: Dual Monitors

2014-05-12 Thread Lindsay Mathieson
On Mon, 12 May 2014 03:11:16 PM Marco Martin wrote: > > > NB - Plasma2 (via project neon) is unbelievably slow on my PC, a good > > > 10-20 seconds to bring up the KDE menu etc, though it is faster the > > > second time. All screens are very slow the first time round. > > > > > > > > > > > > Is th

Re: Dual Monitors

2014-05-12 Thread Lindsay Mathieson
On Mon, 12 May 2014 03:04:35 PM David Edmundson wrote: > Gone isn't the right word. It is not ported/not in Neon yet though. > > You can run > kcmshell4 kcm_kscreen to launch it That worked, thanks. Doesn't stick though, monitor alignment resets after login/logout, and panels/widgets seem to ra

Re: Review Request 118089: Fix build: add spaces in string concatenation

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

Re: Review Request 118089: Fix build: add spaces in string concatenation

2014-05-12 Thread Alexander Potashev
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118089/ --- (Updated May 12, 2014, 8:19 p.m.) Status -- This change has been mar

Re: Review Request 118092: Left and right movement in All music

2014-05-12 Thread Shantanu Tushar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118092/#review57797 --- components/listbrowser/ListBrowser.qml

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Martin Klapetek
> On May 12, 2014, 1:04 p.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to so

Re: RFC: Change Krunner's default key shortcut for Next

2014-05-12 Thread Vishesh Handa
On Mon, May 12, 2014 at 12:29 PM, Martin Klapetek wrote: > Collecting the input > > So Ctrl+space & Meta+space are already reserved, Alt+space is only problem > for Gnome as it's used there but we don't intend to run KRunner in Gnome so > we are fine. > > So if there are no objections, I'll chang

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread David Edmundson
> On May 12, 2014, 11:04 a.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to s

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Aleix Pol Gonzalez
> On May 12, 2014, 11:04 a.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to s

Re: Qt Quick Controls style

2014-05-12 Thread Jan Grulich
On Monday 12 of May 2014 16:32 David Edmundson wrote: > On Mon, May 12, 2014 at 3:49 PM, Jan Grulich wrote: > > I just tested it with our fresh QML version of kde-nm-connection editor > > and I found two issues: > > > > 1) Headers in TableView don't have visible sort indicators > > It seems [2]

Re: Qt Quick Controls style

2014-05-12 Thread David Edmundson
On Mon, May 12, 2014 at 3:49 PM, Jan Grulich wrote: > I just tested it with our fresh QML version of kde-nm-connection editor and I > found two issues: > > 1) Headers in TableView don't have visible sort indicators > It seems [2] is showing the sort indicator. Does it only not work with the qtcur

Re: Qt Quick Controls style

2014-05-12 Thread David Edmundson
On Mon, May 12, 2014 at 3:49 PM, Jan Grulich wrote: > I just tested it with our fresh QML version of kde-nm-connection editor and I > found two issues: > Personally I think you're still far better off using QWidgets for anything outside plasma space. Your life will be much easier. > 1) Headers in

Re: Review Request 117984: Try to prioritize photos taken by a camera-like device

2014-05-12 Thread Shantanu Tushar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117984/ --- (Updated May 12, 2014, 2:14 p.m.) Status -- This change has been mar

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Martin Gräßlin
> On May 12, 2014, 1:04 p.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to so

Re: Qt Quick Controls style

2014-05-12 Thread Jan Grulich
I just tested it with our fresh QML version of kde-nm-connection editor and I found two issues: 1) Headers in TableView don't have visible sort indicators 2) Menu and Toolbar don't have icons, not sure whether this is an issue, but when I use default style, I have Icon + Text + Shortcut in each

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Martin Klapetek
> On May 12, 2014, 1:04 p.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to so

Re: Dual Monitors

2014-05-12 Thread Aleix Pol
On Mon, May 12, 2014 at 2:05 PM, Lindsay Mathieson < lindsay.mathie...@gmail.com> wrote: > Is there a tool for configuring multiple monitors in Plasma2? as kscreen > appears to be gone? > -- > Lindsay > ___ > Plasma-devel mailing list > Plasma-devel@kde.

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Martin Gräßlin
> On May 12, 2014, 1:04 p.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to so

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Aleix Pol Gonzalez
> On May 12, 2014, 11:04 a.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to s

Re: Plasma Next Beta 1

2014-05-12 Thread Jonathan Riddell
On Sat, May 10, 2014 at 08:24:30PM +0200, šumski wrote: > Yeah. Thus also kfilemetadata (so this baloo tar is unbuildable - not just > cause master branch is packaged instead of frameworks one ;-) > And since the WIP kde-baseapps is here (somehow this would fit more with > Applications releases t

Re: Review Request 118063: New Formats KCM

2014-05-12 Thread John Layt
> On May 11, 2014, 6:59 p.m., John Layt wrote: > > kcms/formats/kcmformats.cpp, line 204 > > > > > > Why are you writing all these entries here? If all they have set is > > the global then all we need to export

Re: Dual Monitors

2014-05-12 Thread Marco Martin
On Monday 12 May 2014, David Edmundson wrote: > > NB - Plasma2 (via project neon) is unbelievably slow on my PC, a good > > 10-20 seconds to bring up the KDE menu etc, though it is faster the > > second time. All screens are very slow the first time round. > > > > Is this normal at this stage? - I

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Aleix Pol Gonzalez
> On May 12, 2014, 11:04 a.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to s

Re: Dual Monitors

2014-05-12 Thread David Edmundson
On Mon, May 12, 2014 at 2:25 PM, Lindsay Mathieson wrote: > On Mon, 12 May 2014 02:09:38 PM Luca Beltrame wrote: >> Since recently, plasmashell uses libkscreen. Whether the user-facing UI was >> ported yet I don't know, I defer this to the more expert people in the >> list. > > Thanks, I'll see wh

Re: Where we are now... (VDG)

2014-05-12 Thread Marco Martin
On Monday 12 May 2014, Jens Reuterberg wrote: > There's an issue with the cursors though and that is that they only exist > in one size right now which is causing merry hell for people with HDPI > screens ok, now i automated the generation a bit and the cursors have also a doubled version. there

Re: Dual Monitors

2014-05-12 Thread David Edmundson
On Mon, May 12, 2014 at 2:05 PM, Lindsay Mathieson wrote: > Is there a tool for configuring multiple monitors in Plasma2? as kscreen > appears to be gone? Gone isn't the right word. It is not ported/not in Neon yet though. You can run kcmshell4 kcm_kscreen to launch it David ___

Re: Dual Monitors

2014-05-12 Thread Lindsay Mathieson
On Mon, 12 May 2014 02:09:38 PM Luca Beltrame wrote: > Since recently, plasmashell uses libkscreen. Whether the user-facing UI was > ported yet I don't know, I defer this to the more expert people in the > list. Thanks, I'll see what pops up :) NB - Plasma2 (via project neon) is unbelievably slo

Re: Dual Monitors

2014-05-12 Thread Luca Beltrame
In data lunedì 12 maggio 2014 22:05:47, Lindsay Mathieson ha scritto: > Is there a tool for configuring multiple monitors in Plasma2? as kscreen > appears to be gone? Since recently, plasmashell uses libkscreen. Whether the user-facing UI was ported yet I don't know, I defer this to the more expe

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Martin Gräßlin
> On May 12, 2014, 1:04 p.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to so

Dual Monitors

2014-05-12 Thread Lindsay Mathieson
Is there a tool for configuring multiple monitors in Plasma2? as kscreen appears to be gone? -- Lindsay signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/

Re: Layout.js in shell package is not working

2014-05-12 Thread Bhushan Shah
On Mon, May 12, 2014 at 5:04 PM, Nowardev-Team wrote: > i guess notmart has changed something on plasma scripting , > > try to read this : > > http://mail.kde.org/pipermail/plasma-devel/2014-March/029614.html Yeah but thing is it is not updated in the plasma desktop shell.. notmart right? -- Bh

Re: Layout.js in shell package is not working

2014-05-12 Thread Nowardev-Team
i guess notmart has changed something on plasma scripting , try to read this : http://mail.kde.org/pipermail/plasma-devel/2014-March/029614.html 2014-05-12 13:20 GMT+02:00 Bhushan Shah : > Hello, > > In a desktop shell package layout.js is not working as it should. > > Its failing at this line,

[plasma-shell] [Bug 328593] Dual screen has regressed in plasma-shell

2014-05-12 Thread Martin Klapetek
https://bugs.kde.org/show_bug.cgi?id=328593 Bug 328593 depends on bug 334502, which changed state. Bug 334502 Summary: Panel positions in multiscreen are not remembered/placed depending on mouse position https://bugs.kde.org/show_bug.cgi?id=334502 What|Removed

[plasma-shell] [Bug 328593] Dual screen has regressed in plasma-shell

2014-05-12 Thread Aleix Pol
https://bugs.kde.org/show_bug.cgi?id=328593 Bug 328593 depends on bug 334502, which changed state. Bug 334502 Summary: Panel positions in multiscreen are not remembered/placed depending on mouse position https://bugs.kde.org/show_bug.cgi?id=334502 What|Removed

Layout.js in shell package is not working

2014-05-12 Thread Bhushan Shah
Hello, In a desktop shell package layout.js is not working as it should. Its failing at this line, var desktop = new Activity with error message telling Activity is not undefined. So changing wallpaper plugin of desktop or adding applet with addWidget is not possible. What is solution of this?

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Aleix Pol Gonzalez
> On May 12, 2014, 11:04 a.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to s

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Martin Klapetek
> On May 12, 2014, 1:04 p.m., Aleix Pol Gonzalez wrote: > > That's not correct. Primary, at least, needs to have special treatment and > > get the 0-named containment assigned always, which is what the current > > naming is about. > > > > As for the rest of the screens it's probably best to so

Re: Review Request 118089: Fix build: add spaces in string concatenation

2014-05-12 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118089/#review57759 --- Ship it! Ship It! - Sebastian Kügler On May 12, 2014, 12:4

Minutes Monday Plasma hangout

2014-05-12 Thread Sebastian Kügler
Present: Antonis, David, Ivan, Jens, Jonathan, Marco, Martin G., Martin K., Vishesh, Sebastian Antonis - has started the work to port Plasma Active to Plasma Next, - started with a "forked" shell package David: - Has been doing SDDM fixes, merge pending - Some files in plasma-framework need r

Re: RFC: Change Krunner's default key shortcut for Next

2014-05-12 Thread David Edmundson
Do it. Sooner we do it, the earlier we'll know if there's any problems. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118094/#review57757 --- That's not correct. Primary, at least, needs to have special t

Re: Review Request 118061: Plasma-mobile: Add an initial shell package

2014-05-12 Thread Aleix Pol Gonzalez
> On May 10, 2014, 12:15 a.m., Aleix Pol Gonzalez wrote: > > Wouldn't it be better to let the Desktop settle down a bit before we start > > to fork things out? Actually we should find ways to share code and not > > having to actually fork these, which is really counter-productive. > > Marco Ma

Re: Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118094/ --- (Updated May 12, 2014, 12:39 p.m.) Review request for Plasma, Aleix Pol G

Re: Review Request 118061: Plasma-mobile: Add an initial shell package

2014-05-12 Thread Bhushan Shah
> On May 10, 2014, 5:45 a.m., Aleix Pol Gonzalez wrote: > > Wouldn't it be better to let the Desktop settle down a bit before we start > > to fork things out? Actually we should find ways to share code and not > > having to actually fork these, which is really counter-productive. > > Marco Mar

Re: RFC: Change Krunner's default key shortcut for Next

2014-05-12 Thread Martin Klapetek
Collecting the input So Ctrl+space & Meta+space are already reserved, Alt+space is only problem for Gnome as it's used there but we don't intend to run KRunner in Gnome so we are fine. So if there are no objections, I'll change the primary shortcut to Alt+space while keeping Alt+F2 as a secondary

Review Request 118094: Sort screens in plasma from left to right

2014-05-12 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118094/ --- Review request for Plasma, Aleix Pol Gonzalez and Martin Gräßlin. Reposit

Re: Where we are now... (VDG)

2014-05-12 Thread Marco Martin
On Monday 12 May 2014, Jens Reuterberg wrote: > There's an issue with the cursors though and that is that they only exist > in one size right now which is causing merry hell for people with HDPI > screens i'm looking to see if generating a double size version may be scriptable enoough -- Marco

Re: Where we are now... (VDG)

2014-05-12 Thread Jens Reuterberg
There's an issue with the cursors though and that is that they only exist in one size right now which is causing merry hell for people with HDPI screens On Monday 12 May 2014 10.48.05 Marco Martin wrote: > On Monday 12 May 2014, Martin Klapetek wrote: > > > (supposedly new) maintainers > > > * ic

Re: Where we are now... (VDG)

2014-05-12 Thread Marco Martin
On Monday 12 May 2014, Martin Klapetek wrote: > > (supposedly new) maintainers > > * icons: those should also be imported somewhere > > * am i missing something? > > windeco - that's also done for aurorae, I'm unsure where though (breeze > repo?) breze repo (so already ok for release) contains:

Re: Where we are now... (VDG)

2014-05-12 Thread Martin Klapetek
On Mon, May 12, 2014 at 10:22 AM, Marco Martin wrote: > > so of all that, let's see what we have, what we don't and how we can make > shippable > > * plasma theme: check > * wallpaper: once is ready, where we do put it? > * widget style: we have the qtcurve settings, i failed as well to contact >

Re: Where we are now... (VDG)

2014-05-12 Thread Marco Martin
On Sunday 11 May 2014, Jens Reuterberg wrote: > (ok third time trying to send this email - so I'm loosing confidence in my > technical ability here) > > Ok so, via Andrew "The Mythical Design Machine" Lake - heres a screen shot > of where Plasma Next is now. Yes - we in the VDG are the awesomeness

Re: RFC - Removing System Bell KCM

2014-05-12 Thread Marco Martin
On Monday 12 May 2014, Martin Gräßlin wrote: > > I don't think it's something relevant anymore, and I'm not entirely > > convinced this code even works. > > I thought it's still used by the accessibility code. If that's not the > case: go, go, go! yes, good point,should be checked if is used in a