Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Martin Graesslin
On Monday, July 4, 2016 11:06:05 PM CEST Aleix Pol wrote:
> On Mon, Jul 4, 2016 at 6:37 PM, Martin Gräßlin  wrote:
> > Am 2016-07-04 14:43, schrieb Thomas Pfeiffer:
> >> Hi everyone,
> >> every now and then, distributions approach us asking which
> >> applications they should ship by default with Plasma, or they complain
> >> about us not providing such information.
> >> Although the Plasma team of course does not have to provide such
> >> information, it may still be helpful also for us because we can try to
> >> make sure that these applications work well in Plasma.
> >> Choosing such applications is not an easy task, but to get things
> >> started, a group of people who were stranded in Bielefeld waiting for
> >> their trains after a meeting sat together to come up with an initial
> >> suggestion. Here is the result:
> >> 
> >> File manager: Dolphin
> >> Music player: Cantata
> > 
> > I think Cantata is unsuited as it requires an mpd running. Given that it's
> > out of scope for simple usage.
> > 
> >> Video player: VLC
> >> Document viewer: Okular
> > 
> > Here we need to be careful given that there is no release based on Qt 5
> > (note that some distros ship with it but master has a terrible and
> > annoying
> > warning in your face dialog about that) and Qt 4 is EOL. Given that
> > viewing
> > pdfs is something which has been exploited in the past and is network
> > attackable in worst case, I think it's not a good choice. As long as there
> > is no Qt5-maintained release I would say it needs to be evince or none.
> 
> That's absurd, if anything we can say it's none and set it as a
> priority to have Okular ported.

Why is that absurd? Currently KDE does not have a pdf viewer which is 
suitable. There is a Qt 4 version - that's no-no, and there's a Qt 5 version 
with a big fat in your face warning - that's no-no-no-no.

evince does a good job and I'm using it regularly.

> 
> >> Software center: Discover
> >> Communication: Konversation, KDE Telepathy (cautiously, because while
> >> it works well at the moment, it is also looking for a maintainer)
> >> Password storage: KWalletmanager, kwallet-pam
> > 
> > While KWalletmanager gives a good integration in some KDE applications
> > it's
> > nothing I would recommend as a wallet manager. It is not well integrated
> > into Plasma, it is not secure, it has a terrible first run experience with
> > recommending to use a GPG key and then telling you that you don't have one
> > and does not have any concept of synchronization. In the area of password
> > storage there are way better solutions available in the FLOSS world
> > 
> >> Hardware support: Skanlite, Print manager
> >> Utilities/system tools: KCalc, KDE Connect, Konsole, KSysguard, Kate,
> >> Kamoso (if a distro wants to ship a webcam app at all)
> >> Office suite: We do not recommend one at the moment
> >> Pim suite: We do not recommend one at the moment.
> >> Browser: We do not recommend one at the moment
> > 
> > for browser I would turn the recommendation the other way: let's
> > explicitly
> > recommend to not use any of the Qt browsers.
> 
> That's just negative speech, not something we want to be responsible
> for. I'm sure KDE could have a good web browser.

KDE might be able to have a good web browser. Currently we don't. If we 
explicitly recommend software we need to be honest first of all with ourselves. 
Just because there is a tool which does job foo, doesn't mean it's a good 
tool. 

> 
> >> If an applicaiton does not show up in this list, this does of course
> >> not mean we don't like the application or the team behind it, it just
> >> means that we _currently_ don't feel confident to recommend it to
> >> users.
> >> 
> >> This is our initial proposal, now we'd like to get the input from the
> >> rest of the Plasma team!
> > 
> > Thanks for starting that thread, very important
> 
> Let's remember that communicating is important.
> Team building is important.
> Community building is important.

Being honest to ourselves is important. Yes I see that a "this software is not 
good enough" can be discouraging for the developers and work against community 
building. But having good recommendations which we can actually recommend to 
our users is more important. We shouldn't be afraid of saying that a software 
is not good enough for a recommendation. If we recommend bad software this 
also directly reflects on us and our community.

Cheers
Martin

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128362: Add unclutter/cascade window actions

2016-07-04 Thread Martin Gräßlin

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128362/#review97112
---



I don't like the hard coding of the dbus service name. But I'm also lacking 
ideas how we could not hard code it. On X11 we could use the root-window 
property ORG_KDE_DBUS_SERVICE_NAME, but for Wayland we have no way to figure 
out the name.


src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h (line 156)


@since 5.25



src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h (line 163)


@since 5.25


- Martin Gräßlin


On July 4, 2016, 8:38 p.m., Eike Hein wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128362/
> ---
> 
> (Updated July 4, 2016, 8:38 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kdeclarative
> 
> 
> Description
> ---
> 
> This is the remaining useful code from the Tasks engine, which is now 
> scheduled for removal. It will be used by the Window List plasmoid.
> 
> As discussed on IRC.
> 
> 
> Diffs
> -
> 
>   src/qmlcontrols/kwindowsystemplugin/CMakeLists.txt ce0ea74 
>   src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h a9db965 
>   src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.cpp 4850011 
> 
> Diff: https://git.reviewboard.kde.org/r/128362/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Eike Hein
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7 video

2016-07-04 Thread Kai Uwe Broulik
Perfect! :)


  Ursprüngliche Nachricht  
Von:kde...@gmail.com
Gesendet:5. Juli 2016 12:17 vorm.
An:plasma-devel@kde.org
Antworten:plasma-devel@kde.org
Betreff:Re: Plasma 5.7 video

Hi,

Since I do not see any objections, here is the link to the final video
https://youtu.be/A9MtFqkRFwQ

Regards
Łukasz
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128375: kcm color preview on top and buttons on bottom.

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128375/
---

(Updated Juli 5, 2016, 1:03 vorm.)


Review request for Plasma.


Repository: plasma-desktop


Description
---

the previews moved to top and the buttons to bottom.


Diffs
-

  kcms/colors/colorsettings.ui 3fe8410 
  kcms/colors/preview.ui 8dde5c1 

Diff: https://git.reviewboard.kde.org/r/128375/diff/


Testing
---


File Attachments (updated)


new
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/05/460fb505-fcf7-473a-a619-5e9e7d930f30__color.png


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 128375: kcm color preview on top and buttons on bottom.

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128375/
---

Review request for Plasma.


Repository: plasma-desktop


Description
---

the previews moved to top and the buttons to bottom.


Diffs
-

  kcms/colors/colorsettings.ui 3fe8410 
  kcms/colors/preview.ui 8dde5c1 

Diff: https://git.reviewboard.kde.org/r/128375/diff/


Testing
---


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128374: solid kcm small change of the entire ux file

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128374/
---

(Updated Juli 5, 2016, 12:21 vorm.)


Review request for Plasma.


Repository: plasma-desktop


Description
---

the button size is in all kcm's defined by the text not extreme like here.

I'd like to make some changes in the other files, but I don't know how this kcm 
work.


Diffs
-

  kcms/emoticons/emoticonslist.ui be2ac6d 

Diff: https://git.reviewboard.kde.org/r/128374/diff/


Testing
---


File Attachments (updated)


new
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/05/2efb7ff0-0731-4f3c-9228-ea5bb2efc028__solid.png


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 128374: solid kcm small change of the entire ux file

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128374/
---

Review request for Plasma.


Repository: plasma-desktop


Description
---

the button size is in all kcm's defined by the text not extreme like here.

I'd like to make some changes in the other files, but I don't know how this kcm 
work.


Diffs
-

  kcms/emoticons/emoticonslist.ui be2ac6d 

Diff: https://git.reviewboard.kde.org/r/128374/diff/


Testing
---


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128373: re-arrange of the emoticon kcm buttons

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128373/
---

(Updated Juli 4, 2016, 11:46 nachm.)


Review request for Plasma.


Repository: plasma-desktop


Description
---

re-arrange of the emoticon kcm buttons.

I'm not sure if the checkbox is really needed or what it will do but this has 
to define the developer.


Diffs
-

  kcms/emoticons/emoticonslist.ui be2ac6d 

Diff: https://git.reviewboard.kde.org/r/128373/diff/


Testing
---


File Attachments


new layout
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/04/3195acf5-0a24-43b7-9b63-c5779cdb5dcf__emoticons.png


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 128373: re-arrange of the emoticon kcm buttons

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128373/
---

Review request for Plasma.


Repository: plasma-desktop


Description
---

re-arrange of the emoticon kcm buttons.

I'm not sure if the checkbox is really needed or what it will do but this has 
to define the developer.


Diffs
-

  kcms/emoticons/emoticonslist.ui be2ac6d 

Diff: https://git.reviewboard.kde.org/r/128373/diff/


Testing
---


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128373: re-arrange of the emoticon kcm buttons

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128373/
---

(Updated Juli 4, 2016, 11:46 nachm.)


Review request for Plasma.


Repository: plasma-desktop


Description
---

re-arrange of the emoticon kcm buttons.

I'm not sure if the checkbox is really needed or what it will do but this has 
to define the developer.


Diffs
-

  kcms/emoticons/emoticonslist.ui be2ac6d 

Diff: https://git.reviewboard.kde.org/r/128373/diff/


Testing
---


File Attachments (updated)


layout now
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/04/3195acf5-0a24-43b7-9b63-c5779cdb5dcf__emoticons.png


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 128372: date and time kcm remove current location time zone

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128372/
---

Review request for Plasma.


Repository: plasma-desktop


Description
---

in the kcm date and time -> timezone tab you have a description what you select 
in the list and than what was selected now. when you start the kcm, than you 
see the selected item so you know what it is now and can say. the setting is 
wrong I'd like to change, but when you want to change it, you don't need to 
know what it is now cause it is wrong otherwise you wouldn't change it and in 
any other kcm you don't get the information what was selected before you change 
it.


Diffs
-

  kcms/dateandtime/dateandtime.ui 1a2a308 

Diff: https://git.reviewboard.kde.org/r/128372/diff/


Testing
---


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128372: date and time kcm remove current location time zone

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128372/
---

(Updated Juli 4, 2016, 11:28 nachm.)


Review request for Plasma.


Repository: plasma-desktop


Description
---

in the kcm date and time -> timezone tab you have a description what you select 
in the list and than what was selected now. when you start the kcm, than you 
see the selected item so you know what it is now and can say. the setting is 
wrong I'd like to change, but when you want to change it, you don't need to 
know what it is now cause it is wrong otherwise you wouldn't change it and in 
any other kcm you don't get the information what was selected before you change 
it.


Diffs
-

  kcms/dateandtime/dateandtime.ui 1a2a308 

Diff: https://git.reviewboard.kde.org/r/128372/diff/


Testing
---


File Attachments (updated)


layout now
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/04/52c342cc-469e-452f-a2e4-b38d11bb23aa__dateandtime.png


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128371: baloo kcm layout update

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128371/
---

(Updated Juli 4, 2016, 11:11 nachm.)


Review request for Plasma.


Repository: plasma-desktop


Description
---

first you want to say desktop index yes/no so I move it on top

the baloo icon in the description is unique no other kcm had it so it was 
removed.


Diffs
-

  kcms/baloo/configwidget.ui 24a02e5 

Diff: https://git.reviewboard.kde.org/r/128371/diff/


Testing
---

edited with qt creator


File Attachments (updated)


new layout
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/04/0385cc21-9740-4b60-a18a-84fd6777ecbf__baloo.png


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 128371: baloo kcm layout update

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128371/
---

Review request for Plasma.


Repository: plasma-desktop


Description
---

first you want to say desktop index yes/no so I move it on top

the baloo icon in the description is unique no other kcm had it so it was 
removed.


Diffs
-

  kcms/baloo/configwidget.ui 24a02e5 

Diff: https://git.reviewboard.kde.org/r/128371/diff/


Testing
---

edited with qt creator


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7 video

2016-07-04 Thread Łukasz Sawicki
Hi,

Since I do not see any objections, here is the link to the final video
https://youtu.be/A9MtFqkRFwQ

Regards
Łukasz
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 163 - Failure!

2016-07-04 Thread no-reply

GENERAL INFO

BUILD FAILURE
Build URL: 
https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/163/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Mon, 04 Jul 2016 21:46:50 +
Build duration: 5 min 35 sec

CHANGE SET
Revision 0d2dbd04dba1222c798e6da3857fec718744bbc6 by kainz.a: (use the same 
layout for all kcm#039;s like plasma theme kcm REVIEW: 128310)
  change: edit kcms/cursortheme/xcursor/themepage.ui
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-04 Thread Andreas Kainz


> On Juli 2, 2016, 11:06 nachm., David Edmundson wrote:
> > Requires two tiny changes in the cpp, which I've done.
> > 
> > Are you happy for me to push with the compilation fixes?
> 
> Andreas Kainz wrote:
> so you already ship the request? or the change cpp file?
> 
> David Edmundson wrote:
> I applied your patch, then I made it compile.
> 
> Do you want me to merge it?
> 
> Andreas Kainz wrote:
> if possible yes, cause my system crashed so I have to redo it.

done, please update the cpp file david. thanks


- Andreas


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128310/#review97041
---


On Juli 4, 2016, 9:40 nachm., Andreas Kainz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128310/
> ---
> 
> (Updated Juli 4, 2016, 9:40 nachm.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> use the same layout for all kcm's like in plasma theme kcm.
> 
> 
> Diffs
> -
> 
>   kcms/cursortheme/xcursor/themepage.ui 
> 6efe60b97792cebb907159342ab66f1c9f34802a 
> 
> Diff: https://git.reviewboard.kde.org/r/128310/diff/
> 
> 
> Testing
> ---
> 
> no test's done cause I don't compile anything.
> 
> 
> File Attachments
> 
> 
> final solution
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/06/28/36f6dc94-e43d-4742-bde0-f0681a358f63__07_MouseCursor2.png
> new
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/07/02/2da0a43d-949a-4b90-90c8-0609f7543747__new.png
> 
> 
> Thanks,
> 
> Andreas Kainz
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128310/
---

(Updated July 4, 2016, 11:40 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma.


Changes
---

Submitted with commit 0d2dbd04dba1222c798e6da3857fec718744bbc6 by andreas kainz 
to branch master.


Repository: plasma-desktop


Description
---

use the same layout for all kcm's like in plasma theme kcm.


Diffs
-

  kcms/cursortheme/xcursor/themepage.ui 
6efe60b97792cebb907159342ab66f1c9f34802a 

Diff: https://git.reviewboard.kde.org/r/128310/diff/


Testing
---

no test's done cause I don't compile anything.


File Attachments


final solution
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/06/28/36f6dc94-e43d-4742-bde0-f0681a358f63__07_MouseCursor2.png
new
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/02/2da0a43d-949a-4b90-90c8-0609f7543747__new.png


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128311: standardize kcm layout - autostart

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128311/
---

(Updated July 4, 2016, 9:38 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma.


Changes
---

Submitted with commit 981a39a09b95c89112c1c962d1b323cb13e8ee1f by andreas kainz 
to branch master.


Repository: plasma-desktop


Description
---

rearrange buttons according to other kcm's


Diffs
-

  kcms/autostart/autostartconfig.ui 6dfdb8b 

Diff: https://git.reviewboard.kde.org/r/128311/diff/


Testing
---


File Attachments


before
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/02/67d0047a-124e-4c7a-b14f-b0656b67019c__before.png
after
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/07/02/90dcdd71-e769-4116-adb3-1adcfe45fcbd__after.png


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Aleix Pol
On Mon, Jul 4, 2016 at 6:37 PM, Martin Gräßlin  wrote:
> Am 2016-07-04 14:43, schrieb Thomas Pfeiffer:
>>
>> Hi everyone,
>> every now and then, distributions approach us asking which
>> applications they should ship by default with Plasma, or they complain
>> about us not providing such information.
>> Although the Plasma team of course does not have to provide such
>> information, it may still be helpful also for us because we can try to
>> make sure that these applications work well in Plasma.
>> Choosing such applications is not an easy task, but to get things
>> started, a group of people who were stranded in Bielefeld waiting for
>> their trains after a meeting sat together to come up with an initial
>> suggestion. Here is the result:
>>
>> File manager: Dolphin
>> Music player: Cantata
>
>
> I think Cantata is unsuited as it requires an mpd running. Given that it's
> out of scope for simple usage.
>
>> Video player: VLC
>> Document viewer: Okular
>
>
> Here we need to be careful given that there is no release based on Qt 5
> (note that some distros ship with it but master has a terrible and annoying
> warning in your face dialog about that) and Qt 4 is EOL. Given that viewing
> pdfs is something which has been exploited in the past and is network
> attackable in worst case, I think it's not a good choice. As long as there
> is no Qt5-maintained release I would say it needs to be evince or none.

That's absurd, if anything we can say it's none and set it as a
priority to have Okular ported.

>> Software center: Discover
>> Communication: Konversation, KDE Telepathy (cautiously, because while
>> it works well at the moment, it is also looking for a maintainer)
>> Password storage: KWalletmanager, kwallet-pam
>
>
> While KWalletmanager gives a good integration in some KDE applications it's
> nothing I would recommend as a wallet manager. It is not well integrated
> into Plasma, it is not secure, it has a terrible first run experience with
> recommending to use a GPG key and then telling you that you don't have one
> and does not have any concept of synchronization. In the area of password
> storage there are way better solutions available in the FLOSS world
>
>> Hardware support: Skanlite, Print manager
>> Utilities/system tools: KCalc, KDE Connect, Konsole, KSysguard, Kate,
>> Kamoso (if a distro wants to ship a webcam app at all)
>> Office suite: We do not recommend one at the moment
>> Pim suite: We do not recommend one at the moment.
>> Browser: We do not recommend one at the moment
>
>
> for browser I would turn the recommendation the other way: let's explicitly
> recommend to not use any of the Qt browsers.

That's just negative speech, not something we want to be responsible
for. I'm sure KDE could have a good web browser.

>> If an applicaiton does not show up in this list, this does of course
>> not mean we don't like the application or the team behind it, it just
>> means that we _currently_ don't feel confident to recommend it to
>> users.
>>
>> This is our initial proposal, now we'd like to get the input from the
>> rest of the Plasma team!
>
>
> Thanks for starting that thread, very important

Let's remember that communicating is important.
Team building is important.
Community building is important.

Aleix
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Thomas Pfeiffer

On 04.07.2016 18:37, Martin Gräßlin wrote:

Am 2016-07-04 14:43, schrieb Thomas Pfeiffer:

Hi everyone,
every now and then, distributions approach us asking which
applications they should ship by default with Plasma, or they complain
about us not providing such information.
Although the Plasma team of course does not have to provide such
information, it may still be helpful also for us because we can try to
make sure that these applications work well in Plasma.
Choosing such applications is not an easy task, but to get things
started, a group of people who were stranded in Bielefeld waiting for
their trains after a meeting sat together to come up with an initial
suggestion. Here is the result:

File manager: Dolphin
Music player: Cantata


I think Cantata is unsuited as it requires an mpd running. Given that it's out 
of scope for simple usage.


Have you set up Cantata lately? Yes, it requires mpd, but it sets one up all by 
itself if you don't have one.
You tell it where your library is and it does the rest, not more complicated 
than any other music player.

We would not have included it in this list if it required setting up mpd 
manually.


Document viewer: Okular


Here we need to be careful given that there is no release based on Qt 5 (note 
that some distros ship with it but master has a terrible and annoying warning 
in your face dialog about that) and Qt 4 is EOL. Given that viewing pdfs is 
something which has been exploited in the past and is network attackable in 
worst case, I think it's not a good choice. As long as there is no 
Qt5-maintained release I would say it needs to be evince or none.


This is a difficult issue, then. Is there any way we can help Albert with 
finishing the Qt5 port? Not
having a well-integrated PDF reader is not a good situation to be in. Of course 
the same is true
for the other areas where we don't recommend anything, but it feels like Okular 
would be the

easiest to get to a point where it could be recommended.


Software center: Discover
Communication: Konversation, KDE Telepathy (cautiously, because while
it works well at the moment, it is also looking for a maintainer)
Password storage: KWalletmanager, kwallet-pam


While KWalletmanager gives a good integration in some KDE applications it's 
nothing I would recommend as a wallet manager. It is not well integrated into 
Plasma, it is not secure, it has a terrible first run experience with 
recommending to use a GPG key and then telling you that you don't have one and 
does not have any concept of synchronization. In the area of password storage 
there are way better solutions available in the FLOSS world


I agree, KWalletmanager as it is now is _not_ a good password manager. The 
reason why we
integrated it in that list is that things like Plasma-NM only work automatically 
with KWallet, so
there is not really a way around that, and KWalletManager is the only practical 
to see or remove

passwords stored in KWallet.
The situation with KWallet is a huge problem for Plasma, which has to be solved.
KSecretService would have been the solution, but unfortunately Valentin has no 
more time to

work on it.
There are various solutions for this problem, but we have to take one, and we do 
need some

form of keyring to store things like wifi keys in an encrypted store.

I will open a separate thread for this issue, as it's too big to be discussed 
within this thread.



Hardware support: Skanlite, Print manager
Utilities/system tools: KCalc, KDE Connect, Konsole, KSysguard, Kate,
Kamoso (if a distro wants to ship a webcam app at all)
Office suite: We do not recommend one at the moment
Pim suite: We do not recommend one at the moment.
Browser: We do not recommend one at the moment


for browser I would turn the recommendation the other way: let's explicitly 
recommend to not use any of the Qt browsers.



I've heard people using e.g. QupZilla as their daily browser and not being
unhappy with it. I don't think it's at a state where I'd explicitly recommend 
it,
but it's not so bad that I'd recommend _against_ it.


If an applicaiton does not show up in this list, this does of course
not mean we don't like the application or the team behind it, it just
means that we _currently_ don't feel confident to recommend it to
users.

This is our initial proposal, now we'd like to get the input from the
rest of the Plasma team!


Thanks for starting that thread, very important


Well someone had to do it ;)
I think it's also important to make ourselves aware of the situation we're
in with regard to basic applications, because that does also contribute to
people's overall impression of "KDE" (= desktop plus apps).

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Commented On] D2090: [Kickoff] Sort Favorites in-view

2016-07-04 Thread colomar (Thomas Pfeiffer)
colomar added a comment.


  +1, good improvement!
  
  I fear that many users won't discover it, but we need a more global solution 
for that problem anyway.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D2090

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma, #plasma:_design, hein
Cc: colomar, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 131 lines] D2090: [Kickoff] Sort Favorites in-view

2016-07-04 Thread broulik (Kai Uwe Broulik)
broulik created this revision.
broulik added reviewers: Plasma, Plasma: Design, hein.
broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Similar to what Kicker and Task Manager already have. Code adapted from 
Kicker.
  Provides a much more hands-on experience when sorting favorites.

TEST PLAN
  Works. It does not scroll the view down automatically as you drop but I don't 
think Kickoff did that previously either.
  Verified that in unlockedDesktop=false Plasma (Kiosk) we cannot sort 
favorites.
  
  (The DragHelper ungrab is so it doesn't swallow clicks like we already do in 
Task Manager. Will commit separately)
  
  Video: https://www.youtube.com/watch?v=-ULgaGbnl2o

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D2090

AFFECTED FILES
  applets/kicker/plugin/draghelper.cpp
  applets/kickoff/package/contents/ui/FavoritesView.qml
  applets/kickoff/package/contents/ui/Kickoff.qml
  applets/kickoff/package/contents/ui/KickoffItem.qml

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma, #plasma:_design, hein
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128313: useraccount icons 128 px and sorted

2016-07-04 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128313/
---

(Updated July 4, 2016, 11:23 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma.


Changes
---

Submitted with commit da573e885a4fc6bea6bf9bc8a312eeb9f2c675ab by andreas kainz 
to branch master.


Repository: plasma-desktop


Description
---

all user account icons are now 128px in size in addition they were renamed to 
get a good preview (grid layout) and a nice sorting. the svg files were changed 
to svgz


Diffs
-

  kcms/useraccount/pics/Ada Lovelace.png 0fdda9f 
  kcms/useraccount/pics/Alice in Wonderland.png 35e680a 
  kcms/useraccount/pics/Blackbox.png 34ea61b 
  kcms/useraccount/pics/Dragon.png f9db690 
  kcms/useraccount/pics/Grace Hopper.png eeb51a0 
  kcms/useraccount/pics/Green.png 3594e67 
  kcms/useraccount/pics/Happy.png def7cbd 
  kcms/useraccount/pics/Kati.png 6060fb2 
  kcms/useraccount/pics/Konqui.png 1116764 
  kcms/useraccount/pics/Leonardo da Vinci.png 6eedda1 
  kcms/useraccount/pics/Listening.png ad9c595 
  kcms/useraccount/pics/Logger.png 6c843ff 
  kcms/useraccount/pics/Mahatma Gandhi.png feab431 
  kcms/useraccount/pics/Mowgli jungle book.png ab3f4b3 
  kcms/useraccount/pics/Notme.png 960c0f8 
  kcms/useraccount/pics/Parley.png 33fdb84 
  kcms/useraccount/pics/Rekonqui.png 4003e9e 
  kcms/useraccount/pics/TV.png c46f82d 
  kcms/useraccount/pics/User.png 6e4624b 
  kcms/useraccount/pics/bomb.png 9ef2232 
  kcms/useraccount/pics/sources/Ada Lovelace.svg 293bc74 
  kcms/useraccount/pics/sources/Alice in Wonderland.svg e163c6f 
  kcms/useraccount/pics/sources/Dragon.svg 2575e4c 
  kcms/useraccount/pics/sources/Grace Hopper.svg 00e241f 
  kcms/useraccount/pics/sources/Kati.svg a5a5645 
  kcms/useraccount/pics/sources/Konqui.svg 191aa4d 
  kcms/useraccount/pics/sources/Leonardo da Vinci.svg 2fdf2f2 
  kcms/useraccount/pics/sources/Logger.svg 0c75a70 
  kcms/useraccount/pics/sources/Mahatma Gandhi.svg 35cc122 
  kcms/useraccount/pics/sources/Mowgli jungle book.svg cfd38c8 
  kcms/useraccount/pics/sources/Parley.svg 7ad4359 
  kcms/useraccount/pics/sources/Rekonqui.svg 4e372d9 
  kcms/useraccount/pics/sources/User.svg 71d044d 
  kcms/useraccount/pics/sources/blackbox.svgz d6899f1 
  kcms/useraccount/pics/sources/bomb.svgz 771f6d2 
  kcms/useraccount/pics/sources/green.svgz 2d43069 
  kcms/useraccount/pics/sources/happy.svgz 7c9fc49 
  kcms/useraccount/pics/sources/listening.svgz 7e0e5ef 
  kcms/useraccount/pics/sources/notme.svgz 5b4a474 
  kcms/useraccount/pics/sources/tv.svgz d666506 

Diff: https://git.reviewboard.kde.org/r/128313/diff/


Testing
---

done.


File Attachments


new layout
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/06/28/8605b935-8113-4d1c-a7bc-65961b2e0c25__change_your_face.png
pic folder from /kcms/useraccount/
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/06/28/78aff0aa-1814-4f80-8e35-df69510c03c2__pics.zip


Thanks,

Andreas Kainz

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Commented On] D2044: WIP: [KSMServer] Load logout dialog after startup rather than on-demand

2016-07-04 Thread broulik (Kai Uwe Broulik)
broulik added a comment.


  I just upgraded my Qt version and could gracefully restart my machine with 
this patch.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D2044

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma, sitter
Cc: mart, graesslin, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated, 611 lines] D2089: Rewrite Window List plasmoid.

2016-07-04 Thread hein (Eike Hein)
hein updated this revision to Diff 4948.
hein added a comment.


  Drop RowLayout use again.
  
  Row is fine; Kicker and other UIs do it like this and it works.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D2089?vs=4947=4948

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D2089

AFFECTED FILES
  applets/window-list/contents/ui/Menu.qml
  applets/window-list/contents/ui/TaskDelegate.qml
  applets/window-list/contents/ui/main.qml
  applets/window-list/metadata.desktop

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma
Cc: broulik, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7 video

2016-07-04 Thread Łukasz Sawicki
Hi,
Here is a new video
https://youtu.be/Dmek0965ug8

Changelog:

konsole> thunderbrid
pre applications > per applications
improved kiosk voice over

Regards
Łukasz
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Commented On] D2089: Rewrite Window List plasmoid.

2016-07-04 Thread hein (Eike Hein)
hein added a comment.


  Hang on -- the RowLayout doesn't work right, I need to touch the label stuff 
again.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D2089

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma
Cc: broulik, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated, 611 lines] D2089: Rewrite Window List plasmoid.

2016-07-04 Thread hein (Eike Hein)
hein updated this revision to Diff 4947.
hein added a comment.


  - Fix Kai's issues.
  - Fix delegate right margin when scrolling.
  - When scrolling, move window pin to avoid clash with scrollbar.
  - Elide delegate labels intersecting window pin early.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D2089?vs=4946=4947

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D2089

AFFECTED FILES
  applets/window-list/contents/ui/Menu.qml
  applets/window-list/contents/ui/TaskDelegate.qml
  applets/window-list/contents/ui/main.qml
  applets/window-list/metadata.desktop

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma
Cc: broulik, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2089: Rewrite Window List plasmoid.

2016-07-04 Thread hein (Eike Hein)
hein marked 8 inline comments as done.
hein added inline comments.

INLINE COMMENTS

> hein wrote in main.qml:199
> Done.

Actually not done; I need to manage this for other reasons.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D2089

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma
Cc: broulik, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2089: Rewrite Window List plasmoid.

2016-07-04 Thread hein (Eike Hein)
hein marked 10 inline comments as done.
hein added a comment.


  I'm doing some other tweaks and then updating with the fixes.

INLINE COMMENTS

> broulik wrote in main.qml:94
> Does this ever happen? focus != activeFocus

Yes. ScrollArea is a FocusScope. The ListView never loses focus, the ScrollArea 
does.

> broulik wrote in main.qml:113
> Doesn't ListView take care of highlight placement and visibility?

Removed and replaced with highlightMoveDuration.

> broulik wrote in main.qml:121
> Unused id

Dropped.

> broulik wrote in main.qml:142
> Just make this the MouseArea, saves you one Item per delegate

Changed.

> broulik wrote in main.qml:157
> This is the default

I prefer being explicit, but dropped.

> broulik wrote in main.qml:186
> Layout.preferredWidth? I've seen wonkiness with IconItem and Layouts as it 
> has an implicitWidth/height and that has priority over width/height as far as 
> Layout is concerned

Done.

> broulik wrote in main.qml:199
> Use RowLayout instead of Row and then Layout.fillWidth?

Done.

> broulik wrote in main.qml:211
> There's a KeyNavigation attached property where you can tell it which the 
> next tab/backtab/up/down item is

I know. KeyNavigation never really works well for me, and has issues when you 
try to combine it with Keys.* as you usually have to when working with the item 
views. I prefer just stickig to Keys for consistency. It's more or less the 
same code anyways.

> broulik wrote in main.qml:218
> event.accepted twice, inside and outside the if

Thanks, copy and paste accident. Now I can actually remove the 
onUpPressed/onDownPressed again :-).

> broulik wrote in main.qml:309
> QtQuick Controls Button from which PlasmaComponents Button inherits 
> automatically handles Accessible automatically

Dropped.

> broulik wrote in main.qml:388
> Is that API even supported still? We have Plasmoid.toolTipMainText, 
> Plasmoid.toolTipSubText etc

Well, it worked ... I kept that part from the old code. I changed it now though.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D2089

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma
Cc: broulik, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Changed Subscribers] D2089: Rewrite Window List plasmoid.

2016-07-04 Thread broulik (Kai Uwe Broulik)
broulik added inline comments.

INLINE COMMENTS

> main.qml:94
> +
> +onFocusChanged: {
> +if (!focus) {

Does this ever happen? focus != activeFocus

> main.qml:113
> +highlight: PlasmaComponents.Highlight {
> +anchors.fill: windowListView.currentItem != null ? 
> windowListView.currentItem : undefined
> +

Doesn't ListView take care of highlight placement and visibility?

> main.qml:121
> +section.delegate: PlasmaComponents.Label {
> +id: sectionLabel
> +

Unused id

> main.qml:142
> +
> +delegate: Item {
> +id: item

Just make this the MouseArea, saves you one Item per delegate

> main.qml:157
> +hoverEnabled: true
> +acceptedButtons: Qt.LeftButton
> +

This is the default

> main.qml:186
> +
> +width: visible ? units.iconSizes.small : 0
> +height: width

Layout.preferredWidth? I've seen wonkiness with IconItem and Layouts as it has 
an implicitWidth/height and that has priority over width/height as far as 
Layout is concerned

> main.qml:199
> +
> +width: (parent.width - icon.width - parent.spacing)
> +

Use RowLayout instead of Row and then Layout.fillWidth?

> main.qml:211
> +
> +Keys.onTabPressed: unclutterButton.focus = true
> +Keys.onBacktabPressed: windowPin.focus = true

There's a KeyNavigation attached property where you can tell it which the next 
tab/backtab/up/down item is

> main.qml:218
> +Keys.onPressed: {
> +event.accepted = true;
> +

event.accepted twice, inside and outside the if

> main.qml:309
> +
> +Accessible.role: Accessible.Button
> +Accessible.name: text

QtQuick Controls Button from which PlasmaComponents Button inherits 
automatically handles Accessible automatically

> main.qml:388
> +Component.onCompleted: {
> +var toolTipData = new Object;
> +toolTipData["image"] = "preferences-system-window";

Is that API even supported still? We have Plasmoid.toolTipMainText, 
Plasmoid.toolTipSubText etc

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D2089

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma
Cc: broulik, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated, 624 lines] D2089: Rewrite Window List plasmoid.

2016-07-04 Thread hein (Eike Hein)
hein updated this revision to Diff 4946.
hein added a comment.


  Adjust to changed API in RB128362.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D2089?vs=4945=4946

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D2089

AFFECTED FILES
  applets/window-list/contents/ui/Menu.qml
  applets/window-list/contents/ui/TaskDelegate.qml
  applets/window-list/contents/ui/main.qml
  applets/window-list/metadata.desktop

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128362: Add unclutter/cascade window actions

2016-07-04 Thread Eike Hein

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128362/
---

(Updated July 4, 2016, 6:38 p.m.)


Review request for Plasma.


Changes
---

* Implement Kai's code improvement suggestion
* Use the KWin method names, don't invent new names


Repository: kdeclarative


Description
---

This is the remaining useful code from the Tasks engine, which is now scheduled 
for removal. It will be used by the Window List plasmoid.

As discussed on IRC.


Diffs (updated)
-

  src/qmlcontrols/kwindowsystemplugin/CMakeLists.txt ce0ea74 
  src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h a9db965 
  src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.cpp 4850011 

Diff: https://git.reviewboard.kde.org/r/128362/diff/


Testing
---


Thanks,

Eike Hein

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128362: Add unclutter/cascade window actions

2016-07-04 Thread Kai Uwe Broulik

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128362/#review97087
---



Makes sense to put it there.


src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.cpp (line 157)


This looks like it leaks.


QDBusConnection::sessionBus().asyncCall(QDBusMessage::createMethodCall(...));


- Kai Uwe Broulik


On Juli 4, 2016, 6:21 nachm., Eike Hein wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128362/
> ---
> 
> (Updated Juli 4, 2016, 6:21 nachm.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kdeclarative
> 
> 
> Description
> ---
> 
> This is the remaining useful code from the Tasks engine, which is now 
> scheduled for removal. It will be used by the Window List plasmoid.
> 
> As discussed on IRC.
> 
> 
> Diffs
> -
> 
>   src/qmlcontrols/kwindowsystemplugin/CMakeLists.txt ce0ea74 
>   src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h a9db965 
>   src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.cpp 4850011 
> 
> Diff: https://git.reviewboard.kde.org/r/128362/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Eike Hein
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 624 lines] D2089: Rewrite Window List plasmoid.

2016-07-04 Thread hein (Eike Hein)
hein created this revision.
hein added a reviewer: Plasma.
hein added a subscriber: plasma-devel.
Restricted Application added a project: Plasma.

REVISION SUMMARY
  - Ported to libtaskmanager-ng, dropping dependency on legacytaskmanager
  - Made virtual desktop sectioning in the listview actually work
- Bonus: Don't section when there is only one virtual desktop
  - Made Unclutter/Cascade window actions actually work, and fix the wonky way 
they were shown in the UI
- Code added to KWindowSystem QML plugin
  - Implemented full keyboard navigation: enter/return, arrows and tab/backtab
  - Added popup window pin
  - Added basic Accessibility support
  - Made UI sizes and margins consistent with Kicker and Folder View listviews
  
  Depends on https://git.reviewboard.kde.org/r/128362/

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D2089

AFFECTED FILES
  applets/window-list/contents/ui/Menu.qml
  applets/window-list/contents/ui/TaskDelegate.qml
  applets/window-list/contents/ui/main.qml
  applets/window-list/metadata.desktop

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 128362: Add unclutter/cascade window actions

2016-07-04 Thread Eike Hein

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128362/
---

Review request for Plasma.


Repository: kdeclarative


Description
---

This is the remaining useful code from the Tasks engine, which is now scheduled 
for removal. It will be used by the Window List plasmoid.

As discussed on IRC.


Diffs
-

  src/qmlcontrols/kwindowsystemplugin/CMakeLists.txt ce0ea74 
  src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h a9db965 
  src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.cpp 4850011 

Diff: https://git.reviewboard.kde.org/r/128362/diff/


Testing
---


Thanks,

Eike Hein

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Martin Gräßlin

Am 2016-07-04 14:43, schrieb Thomas Pfeiffer:

Hi everyone,
every now and then, distributions approach us asking which
applications they should ship by default with Plasma, or they complain
about us not providing such information.
Although the Plasma team of course does not have to provide such
information, it may still be helpful also for us because we can try to
make sure that these applications work well in Plasma.
Choosing such applications is not an easy task, but to get things
started, a group of people who were stranded in Bielefeld waiting for
their trains after a meeting sat together to come up with an initial
suggestion. Here is the result:

File manager: Dolphin
Music player: Cantata


I think Cantata is unsuited as it requires an mpd running. Given that 
it's out of scope for simple usage.



Video player: VLC
Document viewer: Okular


Here we need to be careful given that there is no release based on Qt 5 
(note that some distros ship with it but master has a terrible and 
annoying warning in your face dialog about that) and Qt 4 is EOL. Given 
that viewing pdfs is something which has been exploited in the past and 
is network attackable in worst case, I think it's not a good choice. As 
long as there is no Qt5-maintained release I would say it needs to be 
evince or none.



Software center: Discover
Communication: Konversation, KDE Telepathy (cautiously, because while
it works well at the moment, it is also looking for a maintainer)
Password storage: KWalletmanager, kwallet-pam


While KWalletmanager gives a good integration in some KDE applications 
it's nothing I would recommend as a wallet manager. It is not well 
integrated into Plasma, it is not secure, it has a terrible first run 
experience with recommending to use a GPG key and then telling you that 
you don't have one and does not have any concept of synchronization. In 
the area of password storage there are way better solutions available in 
the FLOSS world



Hardware support: Skanlite, Print manager
Utilities/system tools: KCalc, KDE Connect, Konsole, KSysguard, Kate,
Kamoso (if a distro wants to ship a webcam app at all)
Office suite: We do not recommend one at the moment
Pim suite: We do not recommend one at the moment.
Browser: We do not recommend one at the moment


for browser I would turn the recommendation the other way: let's 
explicitly recommend to not use any of the Qt browsers.




If an applicaiton does not show up in this list, this does of course
not mean we don't like the application or the team behind it, it just
means that we _currently_ don't feel confident to recommend it to
users.

This is our initial proposal, now we'd like to get the input from the
rest of the Plasma team!


Thanks for starting that thread, very important

Cheers
Martin
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Commented On] D2073: Add a KCM setting to control pulse audio module combine-sinks

2016-07-04 Thread broulik (Kai Uwe Broulik)
broulik added a comment.


  Nice! Check your coding style, though.
  
  Can we perhaps add a special sorting logic so this virtual device always 
shows at the top of the list? We have something similar to keep the 
"Notification Sounds" stream always ontop.

REPOSITORY
  rPLASMAPA Plasma Audio Volume Applet

REVISION DETAIL
  https://phabricator.kde.org/D2073

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: broulik, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Accepted] D2086: Restore asynchronous QML object incubation in krunner

2016-07-04 Thread broulik (Kai Uwe Broulik)
broulik accepted this revision.
broulik added a reviewer: broulik.
This revision is now accepted and ready to land.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D2086

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma, broulik
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 5 lines] D2086: Restore asynchronous QML object incubation in krunner

2016-07-04 Thread davidedmundson (David Edmundson)
davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Using KDeclrative::QmlObject::rootObject before it's created blocks the
  thread until it finishes so that code always "works" - but displays a
  warning and is slower.
  
  We now no longer get a warning on startup.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D2086

AFFECTED FILES
  krunner/view.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 354250] "Switch User" sometimes sends computer to Standby

2016-07-04 Thread Oliver Henshaw via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=354250

--- Comment #30 from Oliver Henshaw  ---
Created attachment 99846
  --> https://bugs.kde.org/attachment.cgi?id=99846=edit
extra logging to help understand the problem

(In reply to Fabian Köster from comment #14)
> (In reply to Kai Uwe Broulik from comment #13)
> > If you can, try https://phabricator.kde.org/D2033 as I don't switch sessions
> > that often :)
> 
> I applied your patch and tested again, but the issue still happens :-(

Kai's first patch (Diff 1, i.e. https://phabricator.kde.org/D2033?id=4799)
seems to work for me, could you try it again with the attached patch on top and
post the logs?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Jens Reuterberg
IRC is too uncommon to be "ship by default" - Cantata is not relevant enough 
and VLC will work as that if needed, or something more simplistic would be 
good (and Luca's criticism is relevant as the distro maintainers for distros 
more focused on software freedom would have massive issues).

The lightest possible image viewer would be good but to be honest there are 
very few that fit the bill: some stick to menubars on top (for no reason) some 
use a completely different logic than normal (like Photoqt) making them fiddly 
to use. Gwenview but edited to at least not stick to the menu bar logic would 
be good enough I suppose.

Also isn't Kwalletmanager dead and the replacement has yet to make an 
appearence? (is there any other walletmanager, Qtbased that would be a good 
replacement until the replacement comes along?)

Aside from that I ppersonally have no objections.

On Monday, 4 July 2016 14:43:07 CEST Thomas Pfeiffer wrote:
> Hi everyone,
> every now and then, distributions approach us asking which applications they
> should ship by default with Plasma, or they complain about us not providing
> such information.
> Although the Plasma team of course does not have to provide such
> information, it may still be helpful also for us because we can try to make
> sure that these applications work well in Plasma.
> Choosing such applications is not an easy task, but to get things started, a
> group of people who were stranded in Bielefeld waiting for their trains
> after a meeting sat together to come up with an initial suggestion. Here is
> the result:
> 
> File manager: Dolphin
> Music player: Cantata
> Video player: VLC
> Document viewer: Okular
> Software center: Discover
> Communication: Konversation, KDE Telepathy (cautiously, because while it
> works well at the moment, it is also looking for a maintainer)
> Password storage: KWalletmanager, kwallet-pam
> Hardware support: Skanlite, Print manager
> Utilities/system tools: KCalc, KDE Connect, Konsole, KSysguard, Kate, Kamoso
> (if a distro wants to ship a webcam app at all)
> Office suite: We do not recommend one at the moment
> Pim suite: We do not recommend one at the moment.
> Browser: We do not recommend one at the moment
> 
> If an applicaiton does not show up in this list, this does of course not
> mean we don't like the application or the team behind it, it just means
> that we _currently_ don't feel confident to recommend it to users.
> 
> This is our initial proposal, now we'd like to get the input from the rest
> of the Plasma team!
> 
> Thanks,
> Thomas
> ___
> Plasma-devel mailing list
> Plasma-devel@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel


___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Thomas Pfeiffer

On 04.07.2016 14:59, Luca Beltrame wrote:


In data lunedì 4 luglio 2016 14:54:25 CEST, Luca Beltrame ha scritto:

Music player: Cantata

And something I forgot: it doesn't integrate too well with the workspace. At
least for my cheap keyboard, multimedia keys aren't handled unless the
application has focus, and its tray icon behaves not too well.

Good point: Actually, we think that multimedia key support should
always be handled via MPRIS, not via the applications themselves, and
that does work fine in Cantata.
Looks like we should recommend distributions to activate the MPRIS Plasmoid
by default, assign the multimedia keys to it, activate MPRIS in Cantata
and deactivate its tray icon by default. That setup works perfectly fine and
should make all MPRIS-supporting apps behave the same.

Also for some reason the MPRIS plasmoid doesn't show cover info, but that may
be a local MPD issue.

It must be a local issue, I can see cover art both in the Plasmoid and the
task manager tooltip.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Thomas Pfeiffer

On 04.07.2016 14:54, Luca Beltrame wrote:

In data lunedì 4 luglio 2016 14:43:07 CEST, Thomas Pfeiffer ha scritto:


Music player: Cantata

Its reliance on MPD makes it problematic for certain distributions, because
you can't rip out the bad bits from MPD, IIRC (while you can from VLC and
ffmpeg).

By bad, I mean patent-encumbered.

Fair enough, but those distributions can either
a) Look for a server which is compatible with the MPD protocol but
does not contain patent-encumbered technology
b) Just skip Cantata and ship only VLC.

This may be off-topic, so apologies in advance.


It's not off-topic. This is a relevant bit of information which prepares
us for distros bringing that up. Thank you!

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Thomas Pfeiffer

On 04.07.2016 14:48, Marco Martin wrote:

On Monday 04 July 2016, Thomas Pfeiffer wrote:

Office suite: We do not recommend one at the moment
Pim suite: We do not recommend one at the moment.
Browser: We do not recommend one at the moment

i'm fine with the list (would add gwenview tough) should just be clear we are
not actually the maintainer of the applications on that list


Oh yes, definitely Gwenview! That simply got lost (I blame Aleix who
sent me his notes from the meeting but seems to have forgotten
to write down Gwenview :P).

And yes, we should mention that most of these are not maintained by
the Plasma team, and some are even not KDE apps at all.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2084: [libkwineffects] Add signals windowShown and windowHidden to EffectsHandler

2016-07-04 Thread Martin Gräßlin
graesslin added a dependent revision: D2085: [effects] Adjust SlidingPopups to 
new windowShown and windowHidden signals.

REPOSITORY
  rKWIN KWin

REVISION DETAIL
  https://phabricator.kde.org/D2084

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2085: [effects] Adjust SlidingPopups to new windowShown and windowHidden signals

2016-07-04 Thread Martin Gräßlin
graesslin added a dependency: D2084: [libkwineffects] Add signals windowShown 
and windowHidden to EffectsHandler.

REPOSITORY
  rKWIN KWin

REVISION DETAIL
  https://phabricator.kde.org/D2085

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 44 lines] D2085: [effects] Adjust SlidingPopups to new windowShown and windowHidden signals

2016-07-04 Thread Martin Gräßlin
graesslin created this revision.
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added subscribers: kwin, plasma-devel.
Restricted Application added projects: Plasma on Wayland, KWin.

REVISION SUMMARY
  With this change SlidingPopups is able to animate Wayland clients
  properly, though windowHidden does not yet work for OpenGL based
  windows (buffer seems to get lost somewhere).

REPOSITORY
  rKWIN KWin

BRANCH
  sliding-popups-window-shown-hidden

REVISION DETAIL
  https://phabricator.kde.org/D2085

AFFECTED FILES
  effects/slidingpopups/slidingpopups.cpp
  effects/slidingpopups/slidingpopups.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2083: Use hiddenInternal() for a ShellClient which got unmapped

2016-07-04 Thread Martin Gräßlin
graesslin added a dependent revision: D2084: [libkwineffects] Add signals 
windowShown and windowHidden to EffectsHandler.

REPOSITORY
  rKWIN KWin

REVISION DETAIL
  https://phabricator.kde.org/D2083

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2084: [libkwineffects] Add signals windowShown and windowHidden to EffectsHandler

2016-07-04 Thread Martin Gräßlin
graesslin added a dependency: D2083: Use hiddenInternal() for a ShellClient 
which got unmapped.

REPOSITORY
  rKWIN KWin

REVISION DETAIL
  https://phabricator.kde.org/D2084

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 49 lines] D2084: [libkwineffects] Add signals windowShown and windowHidden to EffectsHandler

2016-07-04 Thread Martin Gräßlin
graesslin created this revision.
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added subscribers: kwin, plasma-devel.
Restricted Application added projects: Plasma on Wayland, KWin.

REVISION SUMMARY
  This allows effects to animate when a window is shown again and when
  a window gets hidden but not yet closed/destroyed. This situation
  happens on X11 for e.g. auto hiding panels and on Wayland for pretty
  much any window which properly unmaps (windowHidden) prior to destroy.

REPOSITORY
  rKWIN KWin

BRANCH
  window-shown-hidden-effectshandler

REVISION DETAIL
  https://phabricator.kde.org/D2084

AFFECTED FILES
  autotests/integration/shell_client_test.cpp
  effects.cpp
  libkwineffects/kwineffects.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2059: Merge signal connections for AbstractClient in Workspace

2016-07-04 Thread Martin Gräßlin
graesslin added a dependent revision: D2083: Use hiddenInternal() for a 
ShellClient which got unmapped.

REPOSITORY
  rKWIN KWin

REVISION DETAIL
  https://phabricator.kde.org/D2059

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 33 lines] D2083: Use hiddenInternal() for a ShellClient which got unmapped

2016-07-04 Thread Martin Gräßlin
graesslin created this revision.
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added subscribers: kwin, plasma-devel.
Restricted Application added projects: Plasma on Wayland, KWin.

REVISION SUMMARY
  So far when a ShellClient got unmapped ready_for_painting was set to
  false. That is the ShellClient was treated in the same way as a not
  yet shown window. It was completely excluded from painting, a close
  animation impossible.
  
  This change makes use of the functionality available in
  Client::hiddenInternal(). The window is considered as hidden, thus
  still excluded from e.g. getting input events, but could be rendered
  any time as we still have a previous window pixmap (if referenced).
  This allows to have it considered in the rendering pass, but effects
  still cannot make use of it as that state is not yet exposed to the
  effects.

REPOSITORY
  rKWIN KWin

BRANCH
  shell-client-window-hidden

REVISION DETAIL
  https://phabricator.kde.org/D2083

AFFECTED FILES
  abstract_client.h
  autotests/integration/shell_client_test.cpp
  client.h
  composite.cpp
  input.cpp
  scene.cpp
  shell_client.cpp
  shell_client.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2083: Use hiddenInternal() for a ShellClient which got unmapped

2016-07-04 Thread Martin Gräßlin
graesslin added a dependency: D2059: Merge signal connections for 
AbstractClient in Workspace.

REPOSITORY
  rKWIN KWin

REVISION DETAIL
  https://phabricator.kde.org/D2083

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7.0 tars

2016-07-04 Thread Harald Sitter
On Mon, Jul 4, 2016 at 2:29 PM, David Edmundson
 wrote:
>
>
> On Mon, Jul 4, 2016 at 1:20 PM, Sebastian Kügler  wrote:
>>
>> On Thursday, June 30, 2016 11:06:46 PM CEST Harald Sitter wrote:
>> > On Thu, Jun 30, 2016 at 4:45 PM, David Edmundson
>> >
>> >  wrote:
>> > > *sigh* seems so. Yet plasma-workspace is from the right branch and
>> > > it's
>> > > done by an automated script(!)
>> >
>> > Haven't we fixed that for beta already?
>>
> It was the cached kde_projects.xml that you fixed for the beta
> I hadn't pulled your changes as I needed some of my other local mods to get
> round some hardcoded paths. So partly my fault - though I still think it's
> very broken that we get version information from two different sources
> within the same script.

Yes. I mean no. I am confused.
If the projects.xml says that the stable branch of libkscreen is
Plasma/5.6 we can't simply override that on the release side of
things, l10n would still be generated from the wrong branch so you'd
misalign translations and source. That said, one could do a
consistency enforcement by failing if one of the things in a 'set'
(e.g. kde/workspace) is not in line with the rest.
BUT the plasma bash hell on top of releaseme bypasses most smarts of
releaseme todo with projects processing. In this particular case the
fact that releaseme can release a set (e.g kde/workspace) and handle
it as a 'set' is bypassed by the bash scripts wanting to decide what
to release and then calling tarme in a loop for each project.

So, assuming that is actually a smart way to do it (which it isn't
since it replicates information from projects.xml and allows for
additional human error and makes tarball creation slower), the bash
script would have to check consistency of the output coming out of
tarme (namely the release_data) and make sure that everything is using
the same branch.

HS
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7.0 tars

2016-07-04 Thread Sebastian Kügler
On Monday, July 4, 2016 1:59:41 PM CEST David Edmundson wrote:
> I meant "if we accidentally released libkscreen 5.6" as "libkscreen 5.7
> beta", please tell me.
> 
> We didn't.

Ok, good. Thanks for checking!
-- 
sebas

http://www.kde.org | http://vizZzion.org
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7.0 tars

2016-07-04 Thread David Edmundson
On Mon, Jul 4, 2016 at 1:45 PM, Sebastian Kügler  wrote:

> On Monday, July 4, 2016 1:29:55 PM CEST David Edmundson wrote:
> > ... and if not, I need to know, because that means that
> >
> > - nobody tested the changes in libkscreen's master in the beta
> > - the test reports that I did get from the beta are meaningless (which
> could
> > be a good thing ;))
> >
> > Could you check?
> >
> > I don't know what you mean, but the tarballs that are up are fine.
>
> I meant "if we accidentally released libkscreen 5.6" as "libkscreen 5.7
> beta",
> please tell me.
>

We didn't.

David



> --
> sebas
>
> http://www.kde.org | http://vizZzion.org
>
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Luca Beltrame
In data lunedì 4 luglio 2016 14:54:25 CEST, Luca Beltrame ha scritto:
> > Music player: Cantata

And something I forgot: it doesn't integrate too well with the workspace. At 
least for my cheap keyboard, multimedia keys aren't handled unless the 
application has focus, and its tray icon behaves not too well.

Also for some reason the MPRIS plasmoid doesn't show cover info, but that may 
be a local MPD issue.

-- 
Luca Beltrame - KDE Forums team
KDE Science supporter
GPG key ID: A29D259B

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Luca Beltrame
In data lunedì 4 luglio 2016 14:43:07 CEST, Thomas Pfeiffer ha scritto:

> Music player: Cantata

Its reliance on MPD makes it problematic for certain distributions, because 
you can't rip out the bad bits from MPD, IIRC (while you can from VLC and 
ffmpeg). 

By bad, I mean patent-encumbered.

This may be off-topic, so apologies in advance.

-- 
Luca Beltrame - KDE Forums team
KDE Science supporter
GPG key ID: A29D259B

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Marco Martin
On Monday 04 July 2016, Thomas Pfeiffer wrote:
> Office suite: We do not recommend one at the moment
> Pim suite: We do not recommend one at the moment.
> Browser: We do not recommend one at the moment

i'm fine with the list (would add gwenview tough) should just be clear we are 
not actually the maintainer of the applications on that list


-- 
Marco Martin
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Sebastian Kügler
On Monday, July 4, 2016 2:43:07 PM CEST Thomas Pfeiffer wrote:
> every now and then, distributions approach us asking which applications
> they  should ship by default with Plasma, or they complain about us not
> providing such information.
> Although the Plasma team of course does not have to provide such
> information, it  may still be helpful also for us because we can try to
> make sure that these applications work well in Plasma.
> Choosing such applications is not an easy task, but to get things started,
> a  group of people who were stranded in Bielefeld waiting for their trains
> after a meeting sat together to come up with an initial suggestion. Here is
> the result:
> 
> File manager: Dolphin
> Music player: Cantata
> Video player: VLC
> Document viewer: Okular
> Software center: Discover
> Communication: Konversation, KDE Telepathy (cautiously, because while it
> works  well at the moment, it is also looking for a maintainer)
> Password storage: KWalletmanager, kwallet-pam
> Hardware support: Skanlite, Print manager
> Utilities/system tools: KCalc, KDE Connect, Konsole, KSysguard, Kate, Kamoso
> (if  a distro wants to ship a webcam app at all)
> Office suite: We do not recommend one at the moment
> Pim suite: We do not recommend one at the moment.
> Browser: We do not recommend one at the moment
> 
> If an applicaiton does not show up in this list, this does of course not
> mean we  don't like the application or the team behind it, it just means
> that we _currently_ don't feel confident to recommend it to users.
> 
> This is our initial proposal, now we'd like to get the input from the rest
> of  the Plasma team!

No image viewer?
-- 
sebas

http://www.kde.org | http://vizZzion.org
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7.0 tars

2016-07-04 Thread Sebastian Kügler
On Monday, July 4, 2016 1:29:55 PM CEST David Edmundson wrote:
> ... and if not, I need to know, because that means that
> 
> - nobody tested the changes in libkscreen's master in the beta
> - the test reports that I did get from the beta are meaningless (which could
> be a good thing ;))
> 
> Could you check?
> 
> I don't know what you mean, but the tarballs that are up are fine.

I meant "if we accidentally released libkscreen 5.6" as "libkscreen 5.7 beta", 
please tell me.
-- 
sebas

http://www.kde.org | http://vizZzion.org
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Which applications does the Plasma team recommend to use with Plasma?

2016-07-04 Thread Thomas Pfeiffer

Hi everyone,
every now and then, distributions approach us asking which applications they 
should ship by default with Plasma, or they complain about us not providing such 
information.
Although the Plasma team of course does not have to provide such information, it 
may still be helpful also for us because we can try to make sure that these 
applications work well in Plasma.
Choosing such applications is not an easy task, but to get things started, a 
group of people who were stranded in Bielefeld waiting for their trains after a 
meeting sat together to come up with an initial suggestion. Here is the result:


File manager: Dolphin
Music player: Cantata
Video player: VLC
Document viewer: Okular
Software center: Discover
Communication: Konversation, KDE Telepathy (cautiously, because while it works 
well at the moment, it is also looking for a maintainer)

Password storage: KWalletmanager, kwallet-pam
Hardware support: Skanlite, Print manager
Utilities/system tools: KCalc, KDE Connect, Konsole, KSysguard, Kate, Kamoso (if 
a distro wants to ship a webcam app at all)

Office suite: We do not recommend one at the moment
Pim suite: We do not recommend one at the moment.
Browser: We do not recommend one at the moment

If an applicaiton does not show up in this list, this does of course not mean we 
don't like the application or the team behind it, it just means that we 
_currently_ don't feel confident to recommend it to users.


This is our initial proposal, now we'd like to get the input from the rest of 
the Plasma team!


Thanks,
Thomas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7.0 tars

2016-07-04 Thread David Edmundson
On Mon, Jul 4, 2016 at 1:20 PM, Sebastian Kügler  wrote:

> On Thursday, June 30, 2016 11:06:46 PM CEST Harald Sitter wrote:
> > On Thu, Jun 30, 2016 at 4:45 PM, David Edmundson
> >
> >  wrote:
> > > *sigh* seems so. Yet plasma-workspace is from the right branch and it's
> > > done by an automated script(!)
> >
> > Haven't we fixed that for beta already?
>
> It was the cached kde_projects.xml that you fixed for the beta
I hadn't pulled your changes as I needed some of my other local mods to get
round some hardcoded paths. So partly my fault - though I still think it's
very broken that we get version information from two different sources
within the same script.


> ... and if not, I need to know, because that means that
>
> - nobody tested the changes in libkscreen's master in the beta
> - the test reports that I did get from the beta are meaningless (which
> could
>   be a good thing ;))
>
> Could you check?
>

I don't know what you mean, but the tarballs that are up are fine.

David
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7.0 tars

2016-07-04 Thread Sebastian Kügler
On Thursday, June 30, 2016 11:06:46 PM CEST Harald Sitter wrote:
> On Thu, Jun 30, 2016 at 4:45 PM, David Edmundson
> 
>  wrote:
> > *sigh* seems so. Yet plasma-workspace is from the right branch and it's
> > done by an automated script(!)
> 
> Haven't we fixed that for beta already?

... and if not, I need to know, because that means that

- nobody tested the changes in libkscreen's master in the beta
- the test reports that I did get from the beta are meaningless (which could 
  be a good thing ;))

Could you check?
-- 
sebas

http://www.kde.org | http://vizZzion.org
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Closed] D2080: do not 'default' handle switch-cases

2016-07-04 Thread sitter (Harald Sitter)
This revision was automatically updated to reflect the committed changes.
Closed by commit rKSCREENb484e6d6bf27: do not 'default' handle switch-cases 
(authored by sitter).

REPOSITORY
  rKSCREEN KScreen

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D2080?vs=4927=4935

REVISION DETAIL
  https://phabricator.kde.org/D2080

AFFECTED FILES
  console/console.cpp
  kded/generator.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sitter, sebas
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Closed] D2081: handle DVI

2016-07-04 Thread sitter (Harald Sitter)
This revision was automatically updated to reflect the committed changes.
Closed by commit rKSCREENab629a30c398: handle DVI (authored by sitter).

REPOSITORY
  rKSCREEN KScreen

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D2081?vs=4930=4936

REVISION DETAIL
  https://phabricator.kde.org/D2081

AFFECTED FILES
  console/console.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sitter, sebas
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Closed] D1698: A component shouldn't manage it's own geometry

2016-07-04 Thread davidedmundson (David Edmundson)
This revision was automatically updated to reflect the committed changes.
Closed by commit rKIRIGAMI2a988bdfcf0f: A component shouldn't manage it's own 
geometry (authored by davidedmundson).

CHANGED PRIOR TO COMMIT
  https://phabricator.kde.org/D1698?vs=4028=4934#toc

REPOSITORY
  rKIRIGAMI Kirigami

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D1698?vs=4028=4934

REVISION DETAIL
  https://phabricator.kde.org/D1698

AFFECTED FILES
  src/controls/templates/private/PageRow.qml

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma, mart
Cc: plasma-devel
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Closed] D1697: Don't set the implicitWidth the parent's actual width

2016-07-04 Thread davidedmundson (David Edmundson)
This revision was automatically updated to reflect the committed changes.
Closed by commit rKIRIGAMI7240b66f6d8b: Don't set the implicitWidth the 
parent's actual width (authored by davidedmundson).

CHANGED PRIOR TO COMMIT
  https://phabricator.kde.org/D1697?vs=4027=4933#toc

REPOSITORY
  rKIRIGAMI Kirigami

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D1697?vs=4027=4933

REVISION DETAIL
  https://phabricator.kde.org/D1697

AFFECTED FILES
  src/controls/templates/AbstractListItem.qml

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma, mart
Cc: plasma-devel, #plasma
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated, 242 lines] D1989: Introduce QQuickItem to nest kwin_wayland

2016-07-04 Thread bdhruve (Bhavisha Dhruve)
bdhruve updated this revision to Diff 4932.
bdhruve added a comment.


  Fixed issues mentioned by @graesslin.

REPOSITORY
  rKWIN KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D1989?vs=4886=4932

BRANCH
  kwinqml

REVISION DETAIL
  https://phabricator.kde.org/D1989

AFFECTED FILES
  config-kwin.h.cmake
  plugins/CMakeLists.txt
  plugins/qml/CMakeLists.txt
  plugins/qml/kwinqml/CMakeLists.txt
  plugins/qml/kwinqml/kwinplugin.cpp
  plugins/qml/kwinqml/kwinplugin.h
  plugins/qml/kwinqml/kwinqml.cpp
  plugins/qml/kwinqml/kwinqml.h
  plugins/qml/kwinqml/qmldir

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: bdhruve, graesslin, #plasma_on_wayland, bshah
Cc: bshah, graesslin, plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Commented On] D1989: Introduce QQuickItem to nest kwin_wayland

2016-07-04 Thread bdhruve (Bhavisha Dhruve)
bdhruve added a comment.


  In https://phabricator.kde.org/D1989#38023, @graesslin wrote:
  
  > I would like to see a test case which verifies that it starts the 
kwin_wayland instance and terminates it again.
  
  
  I am not sure on how to add testcase for this? 
  Here is code I use for testing locally:
  
  import QtQuick 2.0
  import org.kde.kwin.app 1.0
  Text {
  
text: "Hello world"
KWinApp {
 id: kwin
 //socketName:"kwin-emulator-wayland-0"
 anchors.fill:parent
 Component.onCompleted: kwin.start()
}
  
  }

REPOSITORY
  rKWIN KWin

REVISION DETAIL
  https://phabricator.kde.org/D1989

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: bdhruve, bshah, #plasma_on_wayland, graesslin
Cc: bshah, graesslin, plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Accepted] D1699: Limit BasicListItem width and elide text

2016-07-04 Thread mart (Marco Martin)
mart accepted this revision.
mart added a reviewer: mart.
This revision is now accepted and ready to land.

REPOSITORY
  rKIRIGAMI Kirigami

BRANCH
  patch3

REVISION DETAIL
  https://phabricator.kde.org/D1699

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma, mart
Cc: plasma-devel
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Accepted] D1697: Don't set the implicitWidth the parent's actual width

2016-07-04 Thread mart (Marco Martin)
mart accepted this revision.
mart added a reviewer: mart.
This revision is now accepted and ready to land.

REPOSITORY
  rKIRIGAMI Kirigami

BRANCH
  patch1

REVISION DETAIL
  https://phabricator.kde.org/D1697

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma, mart
Cc: plasma-devel, #plasma
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Accepted] D1698: A component shouldn't manage it's own geometry

2016-07-04 Thread mart (Marco Martin)
mart accepted this revision.
mart added a reviewer: mart.
This revision is now accepted and ready to land.

REPOSITORY
  rKIRIGAMI Kirigami

BRANCH
  patch2

REVISION DETAIL
  https://phabricator.kde.org/D1698

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma, mart
Cc: plasma-devel
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Accepted] D2081: handle DVI

2016-07-04 Thread Sebastian Kügler
sebas accepted this revision.
sebas added a comment.
This revision is now accepted and ready to land.


  Thanks!

REPOSITORY
  rKSCREEN KScreen

BRANCH
  Plasma/5.7-handle-dvi

REVISION DETAIL
  https://phabricator.kde.org/D2081

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sitter, sebas
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Accepted] D2080: do not 'default' handle switch-cases

2016-07-04 Thread Sebastian Kügler
sebas accepted this revision.
sebas added a comment.
This revision is now accepted and ready to land.


  Nice improvement!

REPOSITORY
  rKSCREEN KScreen

BRANCH
  Plasma/5.7

REVISION DETAIL
  https://phabricator.kde.org/D2080

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sitter, sebas
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Minutes Monday Plasma Hangout

2016-07-04 Thread Sebastian Kügler
Minutes Plasma 'hangout', 4-7-2016, 12:00 CET

Present: mgraesslin, bshah, Sho, kbroulik, d_ed, notmart, Riddell, sebas

mgraesslin:
* Improved internal Test library in KWin
* New test case which verifies rendering is correct in QPainter compositor
* rendering fixes in QPainter compositor * Fixes for failing cursor init in 
drm platform, falls back to software rendering
* support for tooltip window type on Wayland, pending review: https://
git.reviewboard.kde.org/r/128300/ * XClipboard Sync merged
* investigated plasma-workspace tests which time out on build.kde.org
* started support for windowShown/windowHidden to be properly handled by 
effect system to make sliding popups work on Wayland. Show already working
* handle situation of no XDG_RUNTIME_DIR more gracefully

bshah:
- Two image updates, latest on thursday
- Last one is much more stable due libtaskmanager-ng based task switcher
- Got to know why we need patch in qtbase to get opengl working, - It used to 
fail with : "OpenGL ES 3.0 entry points not found"
- proper fix : https://github.com/libhybris/libhybris/pull/316
- Some freedreno/drm experiments
- Will look into getting Qt 5.7 in mobile this week

Sho:
- Fixed a tricky regression in TM layout code in time for final tag thanks to 
debug output from Marco
- Trying to reproduce bug report for Eclipse launcher apparently misbehaving 
(Java ...)
- Decided to rewrite Window List plasmoid from scratch, old GSoC code was too 
bad to keep (about half the code unused/wrong ...)
-- No longer using the tasks data engine & legacylibtaskmanager; data engine 
will be removed as previously discussed
-- Will fix one of the most-reported Plasma crashers (the Window List plasmoid 
is the #1 vector for the itemsChanged crash)
-- Submitting for review later today
- Also worked a bit more on Pager, git branch soon

kbroulik:
* Migrated Color Picker from KDeclarative DragArea to Qt's "Drag" attached 
property as the latter does not eat way mouse events from the MouseArea, ie. 
you can now drag colors from color picker to your favorite drawing app's color 
palette without jeopardizing the context menu
* Added fixes for right-to-left languages for Plasma, ie. now the popups 
(kickoff and what not) use LayoutMirroring, panel no longer pushes applets out 
the view, etc. Will do further fixes in that area. Unfortunately for Qt to 
auto-detect RTL you need qttranslations
* Using the new KIO::highlightInFileManager everywhere we have a "open 
containing folder" action (eg. FolderView, KFind, ..)

d_ed:
* Plasma 5.7 tagged and ready for release: release due tomorrow
* Added special pulse audio setting specially for Kai
* Found a speed boost in plasma framework with setSize over setWidth/Height
* Fixed a thing with system tray not keeping the auto hiding panel visible
* Made a WIP to make panel save size in grid units
* Spent ages on kwin test framework for activities
* Still have the following: https://phabricator.kde.org/D1697 https://
phabricator.kde.org/D1698 https://phabricator.kde.org/D1699

notmart:
* ported plasma mobile task manager to new libtaskmanager, merged and in image
* ghns button for look and feel kcm (to merge when server ready)
* bugfix in knewstuff (fail when all categories are missing, not just one)
* merged kirigami branch, pretty much ready for 1.0
* continued the separate desktop layout per look and feel (desktop scripting 
will probably have to bind functions to add applets with a specific id in 
order to work)
* started a tool in plasma-sdk, lookandfeelexplorer, to create look and feel 
packages that are based on the current desktop setup

Riddell:
* poked d_ed about release details
* Qt 5.7 entered Neon

sebas:
* bit of hacking on windowmetadata: thumbnails and qtquickitems (wip)
* kscreen bug triage, seems very little fallout from changes in this cycle
* apparently, a plugin loading regression in the kscreen kcm -- will 
investigate (BUG:365040)
* Neon dev-ed now running on my main system, makes tracking kscreen stuff 
easier on more machines
* ordered a new graphics card that can hopefully drive my 4K display at native 
res
* does anyone want / need a Radeon 6670 passively cooled? (will take it to 
Akademy)
* will poke apol to document test instruments in applet so we actually use it 
more



-- 
sebas

http://www.kde.org | http://vizZzion.org
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Commented On] D1986: WIP: Save PanelView settings in terms of grid units

2016-07-04 Thread davidedmundson (David Edmundson)
davidedmundson added a comment.


  > uh, also no more plans of supporting QT_SCREEN_SCALE_FACTORS instead? (in 
this case the pixel based sizes should be just fine as they would be scaled?)
  
  We /could/ but using a hybrid of two scaling systems seems like it'll get 
complex

INLINE COMMENTS

> mart wrote in panelview.cpp:1096
> why adding a new setter and getter that work in pixels? if something like 
> that should be added, i wuld like just working with grid units.
> 
> perhaps having the old thickness/setThickness etc that still accept sizes in 
> pixels for compatibility and  have new setUnitsThickness or whatever that 
> accepts units?

I can, but that just moves the conversion into being in multiple places instead 
of just 1.

None of the other users of the thickness (the drag handles or working out the 
workspace screen area) want grid units.

> mart wrote in panelview.cpp:1098
> in the scripting the m height is binded, not the width

you're right.

  int gridUnit = 
QFontMetrics(QGuiApplication::font()).boundingRect(QStringLiteral("M")).height();

Will fix.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D1986

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: mart, plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Jenkins-kde-ci: oxygen-icons5 qt4 latest-qt4 » Linux,gcc - Build # 1 - Successful!

2016-07-04 Thread no-reply

GENERAL INFO

BUILD SUCCESS
Build URL: 
https://build.kde.org/job/oxygen-icons5%20qt4%20latest-qt4/PLATFORM=Linux,compiler=gcc/1/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Mon, 04 Jul 2016 10:13:46 +
Build duration: 23 sec

CHANGE SET
No changes


JUNIT RESULTS

Name: (root) Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 
test(s)

COBERTURA RESULTS

Cobertura Coverage Report
  

By packages
  ___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Jenkins-kde-ci: oxygen-icons5 qt4 stable-qt4 » Linux,gcc - Build # 1 - Successful!

2016-07-04 Thread no-reply

GENERAL INFO

BUILD SUCCESS
Build URL: 
https://build.kde.org/job/oxygen-icons5%20qt4%20stable-qt4/PLATFORM=Linux,compiler=gcc/1/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Mon, 04 Jul 2016 10:13:44 +
Build duration: 24 sec

CHANGE SET
No changes


JUNIT RESULTS

Name: (root) Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 
test(s)

COBERTURA RESULTS

Cobertura Coverage Report
  

By packages
  ___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128316: Make use of QQuickItem::setSize instead of width and height indpedently.

2016-07-04 Thread David Edmundson

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128316/
---

(Updated July 4, 2016, 10:06 a.m.)


Status
--

This change has been marked as submitted.


Review request for KDE Frameworks and Plasma.


Changes
---

Submitted with commit fb3e8d5fc5841cd176e88827016cefb3eac30ef0 by David 
Edmundson to branch master.


Repository: plasma-framework


Description
---

Even though QQuickItem only has a width and height accessor there is a
usable public setSize method.

This gets rid of a lot of potential pointless re-evaluation as internal geometry
is updated before widthChanged is emitted.


Diffs
-

  src/plasmaquick/appletquickitem.cpp a9d044b1b50eace5d20441700eba3733c14b1ffd 
  src/plasmaquick/configview.cpp 5c2920bdc97643a353d47d6698a56f5d898b82e7 
  src/scriptengines/qml/plasmoid/wallpaperinterface.cpp 
adacbe19c3f306cf442850d45fed2933e48e6b4b 

Diff: https://git.reviewboard.kde.org/r/128316/diff/


Testing
---

Restarted Plasma, everything seems normal


Thanks,

David Edmundson

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 2 lines] D2081: handle DVI

2016-07-04 Thread sitter (Harald Sitter)
sitter created this revision.
sitter added a reviewer: sebas.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REPOSITORY
  rKSCREEN KScreen

BRANCH
  Plasma/5.7-handle-dvi

REVISION DETAIL
  https://phabricator.kde.org/D2081

AFFECTED FILES
  console/console.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sitter, sebas
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 9 lines] D2080: do not 'default' handle switch-cases

2016-07-04 Thread sitter (Harald Sitter)
sitter created this revision.
sitter added a reviewer: sebas.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  using the default keyword hides compiler warnings. this causes enum
  expansion to require a dev to remember to expand all switch-cases that use
  the enum to make sure they are still correct, since that never ever happens
  don't use default. also, why do what a computer can do for us...
  
  drop default keyword usage. replace it with explicitly break cases, which
  is explicitly overriding the compiler warning, and taking default actions
  *after* the switch-case block.
  this allows the compiler to warn about unhandled enum values. such as
  
  > console.cpp:139:12: warning: enumeration value ‘DVI’ not handled in switch 
[-Wswitch]

REPOSITORY
  rKSCREEN KScreen

BRANCH
  Plasma/5.7

REVISION DETAIL
  https://phabricator.kde.org/D2080

AFFECTED FILES
  console/console.cpp
  kded/generator.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sitter, sebas
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Long term support release for Plasma?

2016-07-04 Thread Matthias Klumpp
Hey :)

Just a quick notice: All the judgement on this issue given by Debian
and Kubuntu people apply to Tanglu as well (and we completely agree
with Martin Steigerwalds post).

Cheers and greetings from Debconf!
Matthias
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Long term support release for Plasma?

2016-07-04 Thread Martin Graesslin
On Wednesday, June 29, 2016 8:35:26 PM CEST Bernhard Rosenkraenzer wrote:
> On 2016-06-27 14:28, Martin Graesslin wrote:
> > Hi distributions,
> > 
> > in Plasma we are considering to add a long term support release. For
> > this idea
> > we want to get some feedback from your side to know how we should set
> > this up.
> 
>  From an OpenMandriva perspective:
> > We would like to know from you:
> > * is that something which is useful to you?
> 
> We're not too interested in another 5.7.x release when 5.8 is out,
> usually another .x doesn't break things too badly.
> 
> However:
> > * how often should we do an LTS release?
> 
> For us it would be useful to make an end-of-line release LTS. 5.(last
> before 6 is released), then 6.(last before 7) etc. being LTS would help
> us with people who are scared of more significant UI changes. We still
> have a few people who long for the "good old days" of KDE 3.x.
> We probably won't treat LTS releases that aren't end-of-line differently
> from "normal" releases.

That's something completely different to what we had in mind and I don't think 
that's possible. E.g. currently with 5.x we had 4.11 a long term support 
release for quite some time. But right now it's no longer supported and cannot 
be supported because Qt 4 is already EOL. Even if users want it, I think it's 
a disservice for all users to provide them unmaintained software. If Qt 
doesn't provide support any more, we cannot provide support for software 
depending on it either.

I don't expect that this will be different once Qt 6 comes out. So I don't 
expect it to be possible to provide support for Plasma 5 for the life time of 
Plasma 6.

Not to mention of all the problems which start to exist once you upgrade the 
system without touching everything. Recently I was contacted by an NVIDIA dev 
about a problem their latest beta driver exposes in KWin 4.11. A problem which 
would require a large restructuring of the source code which exists in the 5.x 
branch. It's something you don't want in a LTS release. But it shows the big 
problem: you cannot move the stack underneath without touching everything. 
Things like adjustments to newer systemd (hello things moving around from udev 
to somewhere else), adjustments to newer compilers (hello gcc6), adjustments 
for obscure things like XServer no longer running as root (caused problems in 
Qt 4). These are all examples for showing that you cannot just hibernate part 
of the stack.

Cheers
Martin

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 234 - Still Unstable!

2016-07-04 Thread no-reply

GENERAL INFO

BUILD UNSTABLE
Build URL: 
https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/234/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Mon, 04 Jul 2016 08:20:57 +
Build duration: 18 min

CHANGE SET
No changes


JUNIT RESULTS

Name: (root) Failed: 2 test(s), Passed: 8 test(s), Skipped: 0 test(s), Total: 
10 test(s)Failed: TestSuite.org.kde.plasma.analogclock-testFailed: 
TestSuite.org.kde.plasma.kickoff-test

COBERTURA RESULTS

Cobertura Coverage Report
  PACKAGES 11/11 (100%)FILES 50/67 (75%)CLASSES 50/67 (75%)LINE 1973/5286 
(37%)CONDITIONAL 1382/5432 (25%)

By packages
  
drkonqi.parser
FILES 6/10 (60%)CLASSES 6/10 (60%)LINE 303/423 (72%)CONDITIONAL 
478/616 (78%)
drkonqi.tests.backtraceparsertest
FILES 4/4 (100%)CLASSES 4/4 (100%)LINE 74/74 (100%)CONDITIONAL 
33/50 (66%)
kioslave.desktop
FILES 2/3 (67%)CLASSES 2/3 (67%)LINE 113/168 (67%)CONDITIONAL 
37/92 (40%)
kioslave.desktop.tests
FILES 1/1 (100%)CLASSES 1/1 (100%)LINE 66/66 (100%)CONDITIONAL 
26/50 (52%)
klipper
FILES 12/13 (92%)CLASSES 12/13 (92%)LINE 256/384 
(67%)CONDITIONAL 109/210 (52%)
klipper.autotests
FILES 4/4 (100%)CLASSES 4/4 (100%)LINE 630/693 (91%)CONDITIONAL 
377/820 (46%)
libtaskmanager
FILES 5/16 (31%)CLASSES 5/16 (31%)LINE 139/3011 (5%)CONDITIONAL 
88/3159 (3%)
libtaskmanager.autotests
FILES 2/2 (100%)CLASSES 2/2 (100%)LINE 150/150 
(100%)CONDITIONAL 85/170 (50%)
runners.bookmarks
FILES 8/8 (100%)CLASSES 8/8 (100%)LINE 89/159 (56%)CONDITIONAL 
34/96 (35%)
runners.bookmarks.browsers
FILES 4/4 (100%)CLASSES 4/4 (100%)LINE 88/93 (95%)CONDITIONAL 
84/107 (79%)
runners.bookmarks.tests
FILES 2/2 (100%)CLASSES 2/2 (100%)LINE 65/65 (100%)CONDITIONAL 
31/62 (50%)___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128300: Set Tooltip window type on ToolTipDialog through KWindowSystem API

2016-07-04 Thread Marco Martin

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128300/#review97078
---


Ship it!




Ship It!

- Marco Martin


On June 28, 2016, 11:43 a.m., Martin Gräßlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128300/
> ---
> 
> (Updated June 28, 2016, 11:43 a.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> ---
> 
> In order to have support for tooltip window type in Wayland, the
> window type NET::Tooltip is also set through KWindowSystem API once
> the platform surface is created.
> 
> 
> Diffs
> -
> 
>   src/declarativeimports/core/tooltipdialog.cpp 
> 18aa520bf15e1a03eb6a4816e49ab35e2350f8da 
> 
> Diff: https://git.reviewboard.kde.org/r/128300/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128311: standardize kcm layout - autostart

2016-07-04 Thread Marco Martin

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128311/#review97077
---


Ship it!




Ship It!

- Marco Martin


On July 2, 2016, 8:34 a.m., Andreas Kainz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128311/
> ---
> 
> (Updated July 2, 2016, 8:34 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> rearrange buttons according to other kcm's
> 
> 
> Diffs
> -
> 
>   kcms/autostart/autostartconfig.ui 6dfdb8b 
> 
> Diff: https://git.reviewboard.kde.org/r/128311/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> before
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/07/02/67d0047a-124e-4c7a-b14f-b0656b67019c__before.png
> after
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/07/02/90dcdd71-e769-4116-adb3-1adcfe45fcbd__after.png
> 
> 
> Thanks,
> 
> Andreas Kainz
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 128313: useraccount icons 128 px and sorted

2016-07-04 Thread Marco Martin

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128313/#review97076
---


Ship it!




Ship It!

- Marco Martin


On June 28, 2016, 11:13 p.m., Andreas Kainz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128313/
> ---
> 
> (Updated June 28, 2016, 11:13 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> all user account icons are now 128px in size in addition they were renamed to 
> get a good preview (grid layout) and a nice sorting. the svg files were 
> changed to svgz
> 
> 
> Diffs
> -
> 
>   kcms/useraccount/pics/Ada Lovelace.png 0fdda9f 
>   kcms/useraccount/pics/Alice in Wonderland.png 35e680a 
>   kcms/useraccount/pics/Blackbox.png 34ea61b 
>   kcms/useraccount/pics/Dragon.png f9db690 
>   kcms/useraccount/pics/Grace Hopper.png eeb51a0 
>   kcms/useraccount/pics/Green.png 3594e67 
>   kcms/useraccount/pics/Happy.png def7cbd 
>   kcms/useraccount/pics/Kati.png 6060fb2 
>   kcms/useraccount/pics/Konqui.png 1116764 
>   kcms/useraccount/pics/Leonardo da Vinci.png 6eedda1 
>   kcms/useraccount/pics/Listening.png ad9c595 
>   kcms/useraccount/pics/Logger.png 6c843ff 
>   kcms/useraccount/pics/Mahatma Gandhi.png feab431 
>   kcms/useraccount/pics/Mowgli jungle book.png ab3f4b3 
>   kcms/useraccount/pics/Notme.png 960c0f8 
>   kcms/useraccount/pics/Parley.png 33fdb84 
>   kcms/useraccount/pics/Rekonqui.png 4003e9e 
>   kcms/useraccount/pics/TV.png c46f82d 
>   kcms/useraccount/pics/User.png 6e4624b 
>   kcms/useraccount/pics/bomb.png 9ef2232 
>   kcms/useraccount/pics/sources/Ada Lovelace.svg 293bc74 
>   kcms/useraccount/pics/sources/Alice in Wonderland.svg e163c6f 
>   kcms/useraccount/pics/sources/Dragon.svg 2575e4c 
>   kcms/useraccount/pics/sources/Grace Hopper.svg 00e241f 
>   kcms/useraccount/pics/sources/Kati.svg a5a5645 
>   kcms/useraccount/pics/sources/Konqui.svg 191aa4d 
>   kcms/useraccount/pics/sources/Leonardo da Vinci.svg 2fdf2f2 
>   kcms/useraccount/pics/sources/Logger.svg 0c75a70 
>   kcms/useraccount/pics/sources/Mahatma Gandhi.svg 35cc122 
>   kcms/useraccount/pics/sources/Mowgli jungle book.svg cfd38c8 
>   kcms/useraccount/pics/sources/Parley.svg 7ad4359 
>   kcms/useraccount/pics/sources/Rekonqui.svg 4e372d9 
>   kcms/useraccount/pics/sources/User.svg 71d044d 
>   kcms/useraccount/pics/sources/blackbox.svgz d6899f1 
>   kcms/useraccount/pics/sources/bomb.svgz 771f6d2 
>   kcms/useraccount/pics/sources/green.svgz 2d43069 
>   kcms/useraccount/pics/sources/happy.svgz 7c9fc49 
>   kcms/useraccount/pics/sources/listening.svgz 7e0e5ef 
>   kcms/useraccount/pics/sources/notme.svgz 5b4a474 
>   kcms/useraccount/pics/sources/tv.svgz d666506 
> 
> Diff: https://git.reviewboard.kde.org/r/128313/diff/
> 
> 
> Testing
> ---
> 
> done.
> 
> 
> File Attachments
> 
> 
> new layout
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/06/28/8605b935-8113-4d1c-a7bc-65961b2e0c25__change_your_face.png
> pic folder from /kcms/useraccount/
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/06/28/78aff0aa-1814-4f80-8e35-df69510c03c2__pics.zip
> 
> 
> Thanks,
> 
> Andreas Kainz
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Commented On] D2019: [Task Manager] Add media controls in context menu

2016-07-04 Thread Alex T
*colomar added a comment.

  Ah damn you're right, Plasma once again does more than I know ;)
  Back to this review request, though: Shouldn't the application
itself handle media controls as jump lists?
  Cantata, for example, already does just that. Would it then show
playback controls twice (which we should really avoid)?
*



I'm using Plasma 5.7 beta + Frameworks 5.23 + Qt 5.7 on Arch.
Cantata has static media controls on taskbar menu popup, but they don't
work.
Also MPRIS applet with cantata doesn't work well at all.
I can help you debug the problem if you tell me how.
Cheers, 5.7 is THE release i was waiting for.

Alex
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.7.0 tars

2016-07-04 Thread Eric Hameleers

On Thu, 30 Jun 2016, David Edmundson wrote:


Fixed - and checked a bit more thorougly this time.
Thanks again.

David


I have two compile errors in plasma-desktop which were not there when 
I built the 5.6.95 tarballs:


...
[ 93%] Building CXX object 
applets/trash/plugin/CMakeFiles/trashplugin.dir/trash

plugin.cpp.o
[ 93%] Building CXX object 
applets/pager/CMakeFiles/pagerplugin.dir/plugin/pager

.cpp.o
[ 93%] Building CXX object 
containments/desktop/plugins/desktop/CMakeFiles/desktopplugin.dir/systemsettings.cpp.o

/tmp/kde-build/plasma/plasma-desktop-5.7.0/applets/pager/plugin/pager.cpp:52:18:
 fatal error: task.h: No such file or directory
compilation terminated.
applets/pager/CMakeFiles/pagerplugin.dir/build.make:86: recipe for 
target 'applets/pager/CMakeFiles/pagerplugin.dir/plugin/pager.cpp.o' failed
make[2]: *** 
[applets/pager/CMakeFiles/pagerplugin.dir/plugin/pager.cpp.o] Error 1
CMakeFiles/Makefile2:7980: recipe for target 
'applets/pager/CMakeFiles/pagerplugin.dir/all' failed

make[1]: *** [applets/pager/CMakeFiles/pagerplugin.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs
[ 93%] Building CXX object 
containments/desktop/plugins/desktop/CMakeFiles/desktopplugin.dir/desktopplugin_automoc.cpp.o
[ 93%] Building CXX object 
applets/trash/plugin/CMakeFiles/trashplugin.dir/trashplugin_automoc.cpp.o

...


and later:

...
[ 88%] Building CXX object 
applets/taskmanager/CMakeFiles/taskmanagerplugin.dir/

plugin/backend.cpp.o
In file included from 
/tmp/kde-build/plasma/plasma-desktop-5.7.0/applets/taskmanager/plugin/backend.cpp:22:0:

/tmp/kde-build/plasma/plasma-desktop-5.7.0/applets/taskmanager/plugin/backend.h:
26:26: fatal error: groupmanager.h: No such file or directory
compilation terminated.
applets/taskmanager/CMakeFiles/taskmanagerplugin.dir/build.make:62: 
recipe for target 
'applets/taskmanager/CMakeFiles/taskmanagerplugin.dir/plugin/backend.cpp.o

' failed
make[2]: *** 
[applets/taskmanager/CMakeFiles/taskmanagerplugin.dir/plugin/backend.cpp.o] 
Error 1
CMakeFiles/Makefile2:7874: recipe for target 
'applets/taskmanager/CMakeFiles/taskmanagerplugin.dir/all' failed
make[1]: *** 
[applets/taskmanager/CMakeFiles/taskmanagerplugin.dir/all] Error 2

Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

What's up?

Cheers, Eric

--
Eric Hameleers 
Home: http://alien.slackbook.org/blog/
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Long term support release for Plasma?

2016-07-04 Thread Philip Muskovac

Hi,

Am 27.06.2016 um 14:28 schrieb Martin Graesslin:

Hi distributions,

in Plasma we are considering to add a long term support release. For this idea
we want to get some feedback from your side to know how we should set this up.

Here is an example (I mean it!) for how this could look like:
* 5.8 first LTS release based on Qt 5.6 (which is also an LTS) for X11
* every fourth release will be another LTS release, so with the 3 month cycle
there is one LTS per year - 5.8, 5.12, 5.16
* Support is for five release cycles, so with 5.13 the 5.8 LTS goes EOL, with
5.17 the 5.12 release goes EOL
* bug fix release continue in the Fibonacci schedule till the EOL release

We would like to know from you:
* is that something which is useful to you?
So after initially being a bit surprised, I would probably appreciate 
this. Now let me put my Kubuntu hat on.


Kubuntu is on a 6 month release schedule with an LTS every 2 years in 
April. Our current LTS release ships Plasma 5.5 (because 5.6 was far 
late in our cycle to get done), and we're still backporting fixes in 
various places for it. Being able to ship an LTS with a Plasma release 
that receives upstream support until the .1 LTS update 3 months after 
release would be awesome for us and our users.

* how often should we do an LTS release?
as I said, we're on a 2 year schedule, so that's all we really need. As 
other distributions are shifted, one per year sounds reasonable IMO.

* how often should we do bug fix releases for an LTS?
I think the fibonacci updates haven been doing their job just fine? That 
would give ~8 updates over a year. As 13 and 21 weeks are a bit far 
apart between the last 2 releases, maybe switch to a 2 month schedule 
after .6? That would give 11 updates in total I think?. (I don't think 
more updates make sense, and you also need to find developers that are 
willing to fix something in the pre-pre-pre-current release)

* how long should a LTS be supported?
I would be happy with your proposal, but it would be nice if security 
issues could be evaluated for the previous LTS as well. Doesn't need a 
release, a note in the advisory would be sufficient.


Related to that:
* what to do with frameworks?
* Would you freeze the frameworks version or continue to backport newer
framework versions to your distribution?
* Would you want an LTS branch for frameworks as well?
* What would you expect that to look like?
For frameworks I'm clueless, really. To be able to use a frameworks 
release in Kubuntu as an "update" it would have to...
- not cause any regressions [Not in plasma, not in any other software 
that uses it] (Hi behavior change when closing applications)
- not have any new or updated dependencies ("new" as in: we would need 
to add/update a 3rd party package)
- not have any component or pieces of a component removed (counts as 
"regression" [not frameworks, but hi kdepim for once removing unused 
public libs in a "bugfix" update])
- not have any components added (ok, that could probably be handled if 
really necessary)


That's why we and other distributions were asking for bugfix releases 
for frameworks 2 years ago. Did something change in the meantime that I 
missed?


Looking forward to your input on this rather important topic.

Cheers
Martin


Here you have it and thanks for the idea ;)

Philip
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Customizing KDE

2016-07-04 Thread Marek Marczykowski-Górecki
Hi,

I'd like to setup some defaults for all users (at package level),
including some customizations:

1. Setup default theme
2. Setup default wallpaper
3. Set custom menu icon (application launcher?)
4. Set default menu style to "Application Menu"
5. Remove section "Places" from "Computer" tab, or even remove
"Computer" tab completely in Application Launcher
6. Remove "New Session" feature (from all the places: menu, screen
locker, etc)
7. Place some applications as "Favorites" by default
8. Remove "device notifier" and some other applets


Any idea how to do (any of) that? Preferably without patching existing
programs (but if no other option - this will also do).

I've tried using scripts:
https://userbase.kde.org/KDE_System_Administration/PlasmaTwoDesktopScripting
but failed. For example I see no way to remove
entries/applets/plasmoids. Also debugging is quite painful, because I
have no idea where to looks for logs (like what entry was loaded, what
was ignored at all etc), I can only guess looking at the result. And
frequently wondering why my script wasn't launched at all... 

It's all about KDE5.

Thanks in advance.

PS I'm not subscribed to the list, so please cc me on response.

PPS Initially post to k...@mail.kde.org, but haven't got any response
there.

-- 
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?


signature.asc
Description: PGP signature
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Long term support release for Plasma?

2016-07-04 Thread Bernhard Rosenkraenzer

On 2016-06-27 14:28, Martin Graesslin wrote:

Hi distributions,

in Plasma we are considering to add a long term support release. For 
this idea
we want to get some feedback from your side to know how we should set 
this up.


From an OpenMandriva perspective:


We would like to know from you:
* is that something which is useful to you?


We're not too interested in another 5.7.x release when 5.8 is out, 
usually another .x doesn't break things too badly.

However:


* how often should we do an LTS release?


For us it would be useful to make an end-of-line release LTS. 5.(last 
before 6 is released), then 6.(last before 7) etc. being LTS would help 
us with people who are scared of more significant UI changes. We still 
have a few people who long for the "good old days" of KDE 3.x.
We probably won't treat LTS releases that aren't end-of-line differently 
from "normal" releases.



* how often should we do bug fix releases for an LTS?


Distros on fixed release schedules will disagree, but I think there's 
not much of a point in scheduling bug fix releases - I'd opt for 
"release when an important bug is fixed".



* how long should a LTS be supported?


If following the end-of-line idea, ideally until the next big release is 
out (so support 4.x until 6.0 is stable, 5.x until 7.0 is stable, ...)



Related to that:
* what to do with frameworks?
* Would you freeze the frameworks version or continue to backport newer
framework versions to your distribution?
* Would you want an LTS branch for frameworks as well?
* What would you expect that to look like?


I think as far as we're concerned, that's done anyway - there will 
probably be a 5.x branch when switching to Qt6 at some point.


ttyl
bero
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 233 - Still Unstable!

2016-07-04 Thread no-reply

GENERAL INFO

BUILD UNSTABLE
Build URL: 
https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/233/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Mon, 04 Jul 2016 08:05:39 +
Build duration: 13 min

CHANGE SET
No changes


JUNIT RESULTS

Name: (root) Failed: 2 test(s), Passed: 8 test(s), Skipped: 0 test(s), Total: 
10 test(s)Failed: TestSuite.org.kde.plasma.analogclock-testFailed: 
TestSuite.org.kde.plasma.kickoff-test

COBERTURA RESULTS

Cobertura Coverage Report
  PACKAGES 11/11 (100%)FILES 50/67 (75%)CLASSES 50/67 (75%)LINE 1973/5286 
(37%)CONDITIONAL 1382/5432 (25%)

By packages
  
drkonqi.parser
FILES 6/10 (60%)CLASSES 6/10 (60%)LINE 303/423 (72%)CONDITIONAL 
478/616 (78%)
drkonqi.tests.backtraceparsertest
FILES 4/4 (100%)CLASSES 4/4 (100%)LINE 74/74 (100%)CONDITIONAL 
33/50 (66%)
kioslave.desktop
FILES 2/3 (67%)CLASSES 2/3 (67%)LINE 113/168 (67%)CONDITIONAL 
37/92 (40%)
kioslave.desktop.tests
FILES 1/1 (100%)CLASSES 1/1 (100%)LINE 66/66 (100%)CONDITIONAL 
26/50 (52%)
klipper
FILES 12/13 (92%)CLASSES 12/13 (92%)LINE 256/384 
(67%)CONDITIONAL 109/210 (52%)
klipper.autotests
FILES 4/4 (100%)CLASSES 4/4 (100%)LINE 630/693 (91%)CONDITIONAL 
377/820 (46%)
libtaskmanager
FILES 5/16 (31%)CLASSES 5/16 (31%)LINE 139/3011 (5%)CONDITIONAL 
88/3159 (3%)
libtaskmanager.autotests
FILES 2/2 (100%)CLASSES 2/2 (100%)LINE 150/150 
(100%)CONDITIONAL 85/170 (50%)
runners.bookmarks
FILES 8/8 (100%)CLASSES 8/8 (100%)LINE 89/159 (56%)CONDITIONAL 
34/96 (35%)
runners.bookmarks.browsers
FILES 4/4 (100%)CLASSES 4/4 (100%)LINE 88/93 (95%)CONDITIONAL 
84/107 (79%)
runners.bookmarks.tests
FILES 2/2 (100%)CLASSES 2/2 (100%)LINE 65/65 (100%)CONDITIONAL 
31/62 (50%)___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2078: Handle situation of no XDG_RUNTIME_DIR gracefully

2016-07-04 Thread Martin Gräßlin
graesslin added a dependency: D2077: [server] Handle no XDG_RUNTIME_DIR 
situation more gracefully.

REPOSITORY
  rKWIN KWin

REVISION DETAIL
  https://phabricator.kde.org/D2078

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Updated] D2077: [server] Handle no XDG_RUNTIME_DIR situation more gracefully

2016-07-04 Thread Martin Gräßlin
graesslin added a dependent revision: D2078: Handle situation of no 
XDG_RUNTIME_DIR gracefully.

REPOSITORY
  rKWAYLAND KWayland

REVISION DETAIL
  https://phabricator.kde.org/D2077

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #plasma_on_wayland
Cc: plasma-devel, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Request, 125 lines] D2078: Handle situation of no XDG_RUNTIME_DIR gracefully

2016-07-04 Thread Martin Gräßlin
graesslin created this revision.
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added subscribers: kwin, plasma-devel.
Restricted Application added projects: Plasma on Wayland, KWin.

REVISION SUMMARY
  If KWin fails to start the Wayland server due to XDG_RUNTIME_DIR not
  being set, kwin_wayland should terminate with an error condition but
  not crash.
  
  This change makes sure that KWin detects that the Wayland server does
  not work and terminates the startup early and ensures that it doesn't
  crash while going down.
  
  An error message is shown that we could not create the Wayland server.

TEST PLAN
  Test case added which verifies that WaylandServer fails to
  init. Manual testing that kwin_wayland exits with error 1.

REPOSITORY
  rKWIN KWin

BRANCH
  no-xdg-runtime-dir

REVISION DETAIL
  https://phabricator.kde.org/D2078

AFFECTED FILES
  autotests/integration/CMakeLists.txt
  autotests/integration/debug_console_test.cpp
  autotests/integration/decoration_input_test.cpp
  autotests/integration/dont_crash_aurorae_destroy_deco.cpp
  autotests/integration/dont_crash_cancel_animation.cpp
  autotests/integration/dont_crash_empty_deco.cpp
  autotests/integration/dont_crash_glxgears.cpp
  autotests/integration/dont_crash_no_border.cpp
  autotests/integration/input_stacking_order.cpp
  autotests/integration/internal_window.cpp
  autotests/integration/lockscreen.cpp
  autotests/integration/maximize_test.cpp
  autotests/integration/move_resize_window_test.cpp
  autotests/integration/no_xdg_runtime_dir_test.cpp
  autotests/integration/plasma_surface_test.cpp
  autotests/integration/plasmawindow_test.cpp
  autotests/integration/platformcursor.cpp
  autotests/integration/pointer_input.cpp
  autotests/integration/quick_tiling_test.cpp
  autotests/integration/scene_qpainter_test.cpp
  autotests/integration/shade_test.cpp
  autotests/integration/shell_client_test.cpp
  autotests/integration/start_test.cpp
  autotests/integration/struts_test.cpp
  autotests/integration/touch_input_test.cpp
  autotests/integration/transient_no_input_test.cpp
  autotests/integration/transient_placement.cpp
  autotests/integration/xclipboardsync_test.cpp
  main_wayland.cpp
  wayland_server.cpp
  wayland_server.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Jenkins-kde-ci: oxygen-icons5 master latest-qt4 » Linux,gcc - Build # 1 - Failure!

2016-07-04 Thread no-reply

GENERAL INFO

BUILD FAILURE
Build URL: 
https://build.kde.org/job/oxygen-icons5%20master%20latest-qt4/PLATFORM=Linux,compiler=gcc/1/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Mon, 04 Jul 2016 07:25:47 +
Build duration: 53 sec

CHANGE SET
No changes
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


  1   2   >