Re: Review Request 126275: Make the "Plasma (Wayland)" xsession file actually say "Plasma (Wayland)"

2015-12-07 Thread Martin Gräßlin

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


-2 for another reason: the display system should not be a user facing part. 
That's a technical detail which will go away in a year or two. Do you want to 
rename it back to Plasma then? If at all we should change the X11 session to 
have the name adjusted.

- Martin Gräßlin


On Dec. 8, 2015, 6 a.m., Martin Klapetek wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126275/
> ---
> 
> (Updated Dec. 8, 2015, 6 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> ---
> 
> The commit/review says it's displayed as "Plasma (Wayland)"
> but the name is just "Plasma", making it confusing in login
> manager.
> 
> 
> Diffs
> -
> 
>   plasmawayland.desktop.cmake c5d5757 
> 
> Diff: https://git.reviewboard.kde.org/r/126275/diff/
> 
> 
> Testing
> ---
> 
> Sddm now clearly shows which "Plasma" is Waylandplasma.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

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


Re: RFC: split platformtheme plugin from frameworkintegration and move to kde/workspace

2015-12-07 Thread Martin Graesslin
On Monday, December 7, 2015 3:54:31 PM CET Mark Gaiser wrote:
> While at it. Why does frameworkintegration force [1] specific fonts upon
> the user?
> 
> It's fine that apparently some folks prefer Oxygen fonts over all else, but
> i thoroughly dislike it.
> I always end up blacklisting it in my pacman manager (pacman, archlinux)
> since 99% of the time when i have font issues, it's because of that package
> being installed.

We cannot find a font which makes everybody happy. This is impossible, so 
please let's not derail the discussion.
> 
> Regarding your proposal. When running KDE apps on something other then
> Plasma, you would also want to have the frameworksintegration plugin to be
> loaded, right?

No. As I outlined, it would not get loaded as the required env variables are 
set by startkde. I doubt that GNOME is announcing the KDE_SESSION_VERSION env 
variable.

> Specially the platformtheme folder with the vastly improved dialogs over
> stock Qt. Remember, those improved dialogs have the power of KIO behind it
> instead of the default Qt support (only the local filesystem)

On other desktop environments it should pick the platform's native dialog. 
E.g. on GNOME we want to give users the GNOME's file dialog to have an 
integrated experience. Please don't start about GNOME's dialog being not that 
good as ours. That's not the point. We want GTK applications to pick our file 
dialog in Plasma and we want our application's to pick GNOME's file dialog in 
GNOME. Our subjective feeling of superior user experience is pointless if the 
user is used to the platform's file dialog.

> 
> I really see value in having that usable in - say - openbox or any other
> non plasma environment where people would want to open KDE applications
> that make use of framework libraries (like KIO).

which is still possible. They need to install the package and set the env 
variables. That's the same as right now: they need to install the package and 
set the env variables. It's not an automagically works anyway.

> 
> Isn't the only plasma specific part the KStyle folder?

No.

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: 5.5 video

2015-12-07 Thread Lucas S
Hi,

- Added simple cross fade effect between the end of the sequences and the titles

Link
https://youtu.be/WZWmtAohp7s

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


Re: RFC: split platformtheme plugin from frameworkintegration and

2015-12-07 Thread Jonathan Riddell
On Mon, Dec 07, 2015 at 03:54:31PM +0100, Mark Gaiser wrote:
> While at it. Why does frameworkintegration force [1] specific fonts upon the
> user?

For the same reason it sets an icon theme, to give Plasma desktops a consistent 
look and feel.  It switches to Noto in 5.5.

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


Re: RFC: split platformtheme plugin from frameworkintegration and move to kde/workspace

2015-12-07 Thread Mark Gaiser
On Mon, Dec 7, 2015 at 1:09 PM, Martin Graesslin  wrote:

> Hi all,
>
> based on the discussion in [1] I propose that we split out platformtheme
> plugin from frameworkintegration into a dedicated repository and move it to
> kde/workspace to be released together with Plasma 5.6.
>
> The main reasoning is that this plugin only gets loaded with env variables
> set
> in startkde anyway, which makes it a Plasma (Desktop) specific plugin. It's
> about integrating Qt application into Plasma and by that pretty useless for
> anything outside Plasma. Moving it to Plasma has in my opinion some
> advantages
> as we can better synchronize the release cycle and also allow us to depend
> on
> libraries provided by workspace [2].
>
> I'd like to get some feedback on the proposal and if this gets positive
> feedback, I'll look into performing the split.
>
> Best Regards
> Martin Gräßlin
>
> [1] https://mail.kde.org/pipermail/kde-frameworks-devel/2015-November/
> 029022.html
> [2]  e.g. I need a dependency to KWayland, which is currently not yet
> possible
> due to KWayland not yet being moved to frameworks.
> ___
> Kde-frameworks-devel mailing list
> kde-frameworks-de...@kde.org
> https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
>
>
While at it. Why does frameworkintegration force [1] specific fonts upon
the user?

It's fine that apparently some folks prefer Oxygen fonts over all else, but
i thoroughly dislike it.
I always end up blacklisting it in my pacman manager (pacman, archlinux)
since 99% of the time when i have font issues, it's because of that package
being installed.

Imho, it's ok if Plasma "prefers" one font over another (which can be
Oxygen) but it should definitely not force it upon the user.


Regarding your proposal. When running KDE apps on something other then
Plasma, you would also want to have the frameworksintegration plugin to be
loaded, right?
Specially the platformtheme folder with the vastly improved dialogs over
stock Qt. Remember, those improved dialogs have the power of KIO behind it
instead of the default Qt support (only the local filesystem)

I really see value in having that usable in - say - openbox or any other
non plasma environment where people would want to open KDE applications
that make use of framework libraries (like KIO).

Isn't the only plasma specific part the KStyle folder?


[1]
https://quickgit.kde.org/?p=frameworkintegration.git=blob=75dc9b92578577c28288500795198027b4c725ad=fcb3f531ae7161f4ef0768a86db8ce98571f3118=CMakeLists.txt#l72
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 126272: Remove unused dependencies in polkit-kde-agent-1

2015-12-07 Thread Michael Palimaka

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

(Updated Dec. 7, 2015, 5:07 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma.


Changes
---

Submitted with commit 7665ca02a536faf6c2bb643b60c9060d54896889 by Michael 
Palimaka to branch master.


Repository: polkit-kde-agent-1


Description
---

KF5Config doesn't look like it was ever really used, and the last use of 
KF5Notifications seems to be in 595b96ed99d44f8a718e8e3f8c9834852bc7f6d7.


Diffs
-

  AuthDialog.cpp 7c84009f15b1ca778711ce1aa53d28161952cfbb 
  CMakeLists.txt 88037d1cdc0e9a26e184f82de30a8c0dc094dc32 

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


Testing
---

Inspected source, builds.


Thanks,

Michael Palimaka

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


Re: Review Request 126272: Remove unused dependencies in polkit-kde-agent-1

2015-12-07 Thread David Edmundson

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

Ship it!


Ship It!

- David Edmundson


On Dec. 7, 2015, 4:40 p.m., Michael Palimaka wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126272/
> ---
> 
> (Updated Dec. 7, 2015, 4:40 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: polkit-kde-agent-1
> 
> 
> Description
> ---
> 
> KF5Config doesn't look like it was ever really used, and the last use of 
> KF5Notifications seems to be in 595b96ed99d44f8a718e8e3f8c9834852bc7f6d7.
> 
> 
> Diffs
> -
> 
>   AuthDialog.cpp 7c84009f15b1ca778711ce1aa53d28161952cfbb 
>   CMakeLists.txt 88037d1cdc0e9a26e184f82de30a8c0dc094dc32 
> 
> Diff: https://git.reviewboard.kde.org/r/126272/diff/
> 
> 
> Testing
> ---
> 
> Inspected source, builds.
> 
> 
> Thanks,
> 
> Michael Palimaka
> 
>

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


[Powerdevil] [Bug 337674] kded5 is eating CPU

2015-12-07 Thread Martin Klapetek via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=337674

--- Comment #73 from Martin Klapetek  ---
Please follow the steps from Comment #8.

-- 
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


Review Request 126272: Remove unused dependencies in polkit-kde-agent-1

2015-12-07 Thread Michael Palimaka

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

Review request for Plasma.


Repository: polkit-kde-agent-1


Description
---

KF5Config doesn't look like it was ever really used, and the last use of 
KF5Notifications seems to be in 595b96ed99d44f8a718e8e3f8c9834852bc7f6d7.


Diffs
-

  AuthDialog.cpp 7c84009f15b1ca778711ce1aa53d28161952cfbb 
  CMakeLists.txt 88037d1cdc0e9a26e184f82de30a8c0dc094dc32 

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


Testing
---

Inspected source, builds.


Thanks,

Michael Palimaka

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


5.5 video

2015-12-07 Thread Lucas S
Hi
Thanks for all comments. I think that I fixed most (all?) remaining issues.

- Improved  visibility of the text by adding black transparent
background and by moving to the beginning of the sequences
- Fixed a typo
- Added 2 more activities so filtering looks more natural now. (With
more than 5 activities there is a scrollbar visible, which looks kinda
out of place)
- Removed a window with activities settings
- Improved muon part (cropped "freedom")
- Added simple fade in/fade out effect for the first and last title
(but still think that 3d dissolving animation looked the best ;))

Link:
https://youtu.be/DsSyPpHS6HI

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


Status Notifier specification

2015-12-07 Thread Alberts Muktupāvels
Hi!

I think that it is time to update this specification. Here is list with
changes that I propose.

1. Specification.
=

1.1. Currently we have three components - Watcher, Host and Item. We should
rename specification from 'Status Notifier Item' to 'Status Notifier'.

1.2. Current implementations use org.kde.* for dbus names. While this
specification is not used by all desktop environments it is not used only
by kde either. We should start using org.freedesktop.* for dbus names.

1.3. We should start to use versioned dbus interface names.

2. Watcher.
===

2.1. Make method, property and signal names shorter by removing
'StatusNotifier' part from names.

2.2. RegisterHost method, change arguments to:
- bus_name (s) - the bus name of Status Notifier Host or empty to use
sender;
- object_path (o) - the object path of Status Notifier Host.

2.3. RegisterItem method, change arguments to:
- bus_name (s) - the bus name of Status Notifier Item or empty to use
sender;
- object_path (o) - the object path of Status Notifier Item.

2.4. RegisteredItems property, change type from 'as' to 'a(so)' - bus name
and object path for each registered item.

2.5. ProtocolVersion property, remove it.

2.6. Rename 'StatusNotifierHostRegistered' signal to 'HostsChanged'. Should
be emitted when new host is registered or existing disappears.

2.7. ItemRegistered signal, change arguments to:
- bus_name (s) - the bus name of Status Notifier Item that was registered;
- object_path (o) - the object path of Status Notifier Item that was
registered;

2.8. ItemUnregistered signal, change arguments to:
- bus_name (s) - the bus name of Status Notifier Item that was unregistered;
- object_path (o) - the object path of Status Notifier Item that was
unregistered;

Watcher should use:
- org.freedesktop.StatusNotifier1.Watcher as bus name and interface name;
- /org/freedesktop/StatusNotifier1/Watcher as object path.

Improvements over existing interface:
- removed unused property;
- ability to create/register multiple items/hosts from one connection
(currently it is not possible because there can not be two objects with
same path);
- hosts and items are not required to create separate bus name.

Updated dbus interface is added as attachment.

3. Host.


Host should use:
- org.freedesktop.StatusNotifier1.Host as interface name.

Optionally (not required, but similar to current specification):
- org.freedesktop.StatusNotifier1.Host-{pid}-{id} as bus name where {pid}
is process id and {id} is unique identifier starting with 1 (was
'org.kde.StatusNotifierHost-{pid}');
- /org/freedesktop/StatusNotifier1/Host{id} as object path where {id} is
unique identifier starting with 1 (was '/StatusNotifierHost').

4. Item.


4.1. Drop all New* signals and use PropertiesChanged to get new/changed
data.

4.2. New property - IconThemePath (type=s). An additional path to add to
the theme search path to find the icons. It looks like this is used by
current implementations.

4.3. New property - ItemIsMenu (type=b). I have seen this in interface
files, does anyone knows if this is really used? If not then it should not
be included in updated spec.

4.4. New property - Menu (type=o). Object path to dbus menu
(com.canonical.dbusmenu interface). This is used at least by ubuntu.

Maybe type could be changed from o to s? So if there is no menu it could be
set to empty string instead of non-existent path?

4.5. Specification should mention which properties are required and which
ones are optional.

Item should use:
- org.freedesktop.StatusNotifier1.Item as interface name.

Optionally (not required, but similar to current specification):
- org.freedesktop.StatusNotifier1.Item-{pid}-{id} as bus name where {pid}
is process id and {id} is unique identifier starting with 1 (was
'org.kde.StatusNotifierItem-{pid}-{id}');
- /org/freedesktop/StatusNotifier1/Item{id} as object path where {id} is
unique identifier starting with 1 (was '/StatusNotifierItem').

Updated dbus interface is added as attachment.

-- 
Alberts Muktupāvels

http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd;>

  

  


http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd;>

  


  
  



  
  









  
  



  
  


  


http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd;>

  












  
  







































  

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


Re: 5.5 video

2015-12-07 Thread Mario Fux
Am Monday 07 December 2015, 06.56:53 schrieb Lucas S:
> Hi

Morning Lucas

> Thanks for all comments. I think that I fixed most (all?) remaining issues.
> 
> - Improved  visibility of the text by adding black transparent
> background and by moving to the beginning of the sequences
> - Fixed a typo
> - Added 2 more activities so filtering looks more natural now. (With
> more than 5 activities there is a scrollbar visible, which looks kinda
> out of place)
> - Removed a window with activities settings
> - Improved muon part (cropped "freedom")
> - Added simple fade in/fade out effect for the first and last title
> (but still think that 3d dissolving animation looked the best ;))
> 
> Link:
> https://youtu.be/DsSyPpHS6HI

Looks nice. Thanks a lot.
One last nitpick from my side:
- The appearance of the overlays texts and the some scene cuts at roughly the 
same time should be more synchronous.

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


Review Request 126271: [server] Add support for pointer input transformation

2015-12-07 Thread Martin Gräßlin

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

Review request for Plasma and Sebastian Kügler.


Repository: kwayland


Description
---

So far we only supported mapping global to surface-local coordinates
using a 2D-offset. With this change it's possible to register a
QMatrix4x4 to describe the transformation for going from global to
surface-local coordinates in a full 3D space.

The existing 2D-offset is transformed to use the new matrix based
variant describing a translation.


Diffs
-

  src/server/seat_interface.h f6e347243aad6c0330c2e769292fc519795cbee0 
  autotests/client/test_wayland_seat.cpp 
2d3f15683240bf40b91968bbeece18d0ceec8d8e 
  src/server/pointer_interface.cpp 67f6175e731b6b650eca983cf86e959134e48b21 
  src/server/seat_interface.cpp 031c1c702eb96b99bb9c8b104839744fd5aec9af 
  src/server/seat_interface_p.h 7f679528f923b868ac7c8d54d1a04926bb10df72 

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


Testing
---

existing seat test still passes and new added test cases, too.


Thanks,

Martin Gräßlin

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


Re: 5.5 video

2015-12-07 Thread Lucas S
Hi

>The appearance of the overlays texts and the some scene cuts at roughly the
>same time should be more synchronous.

This is intentional, but I will try to play with some fade out effect
to make cuts more subtle.

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


Re: 5.5 video

2015-12-07 Thread Olivier Churlaud

Hi,

Great job!

I have some feedback, maybe for the next version.. Don't take it bad, I 
really value your work: it's just another view point I wanted to give.


After having seen the gnome video, I think that music doesn't contribute 
to seem "pro".


I don't know how they achieve this quality (it really looks like a pro 
advertisement), but it seems like the direction to follow, isn't it?


A company should be able to show the presentation video to it's 
stockholders and say: "we move to this". I don't know if the current 
video can be used like this.


Again, it's only my very personal opinion, and I wouldn't be able to do 
half of what you did, so I totally respect what you did.


Cheers
Olivier

Date: Mon, 7 Dec 2015 06:56:53 +0100
From: Lucas S 
To: plasma-devel@kde.org
Subject: 5.5 video
Message-ID:

Content-Type: text/plain; charset=UTF-8

Hi
Thanks for all comments. I think that I fixed most (all?) remaining issues.

- Improved  visibility of the text by adding black transparent
background and by moving to the beginning of the sequences
- Fixed a typo
- Added 2 more activities so filtering looks more natural now. (With
more than 5 activities there is a scrollbar visible, which looks kinda
out of place)
- Removed a window with activities settings
- Improved muon part (cropped "freedom")
- Added simple fade in/fade out effect for the first and last title
(but still think that 3d dissolving animation looked the best ;))

Link:
https://youtu.be/DsSyPpHS6HI

Cheers
Lucas



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


Minutes Monday Plasma Hangout

2015-12-07 Thread Sebastian Kügler
Present: bshah, Riddell, kbroulik, notmart, mgraesslin, sebas
Date: December 7 2015


bshah:
- trying to get camera working with new libhybris, no success so far
- Please register for: https://sprints.kde.org/sprint/291

Riddell:
- Plasma 5.5 tomorrow

kbroulik:
- improvements to notes widget
- implemented jump lists for icon widget (will post review)

notmart:
- some fixes to breeze theme
- improved navigation in mobilecomponents
- continued work on simple systemtray prototype
- related: https://git.reviewboard.kde.org/r/126244/

mgraesslin:
- continued porting to new shader API ( https://git.reviewboard.kde.org/r/
126215/ )
- observed security problem in kidletime: https://git.reviewboard.kde.org/r/
126220/
- started on server-side window decorations, related RRs:
https://git.reviewboard.kde.org/r/126271/
- move the KDE platform plugin (QPA) to kde/workspace?

sebas:
- smallish fixes to mobilecomponents
- worked quite a bit on the developer guide https://community.kde.org/Plasma/
DeveloperGuide
- screen management: to clean up libkscreen kwayland

-- 
sebas

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


RFC: split platformtheme plugin from frameworkintegration and move to kde/workspace

2015-12-07 Thread Martin Graesslin
Hi all,

based on the discussion in [1] I propose that we split out platformtheme 
plugin from frameworkintegration into a dedicated repository and move it to 
kde/workspace to be released together with Plasma 5.6.

The main reasoning is that this plugin only gets loaded with env variables set 
in startkde anyway, which makes it a Plasma (Desktop) specific plugin. It's 
about integrating Qt application into Plasma and by that pretty useless for 
anything outside Plasma. Moving it to Plasma has in my opinion some advantages 
as we can better synchronize the release cycle and also allow us to depend on 
libraries provided by workspace [2].

I'd like to get some feedback on the proposal and if this gets positive 
feedback, I'll look into performing the split.

Best Regards
Martin Gräßlin

[1] https://mail.kde.org/pipermail/kde-frameworks-devel/2015-November/
029022.html
[2]  e.g. I need a dependency to KWayland, which is currently not yet possible 
due to KWayland not yet being moved to frameworks.

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 126268: Move shutdown scripts into ksmserver cleanup

2015-12-07 Thread Matthias Klumpp

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

Ship it!


Good thing!
I wasted a few hours yesterday in finding out why this works, just to come to 
the conclusion that it can't possibly work.
Thank you for the explanation, this does make more sense to me now :)

- Matthias Klumpp


On Dez. 7, 2015, 12:26 vorm., David Edmundson wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126268/
> ---
> 
> (Updated Dez. 7, 2015, 12:26 vorm.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> ---
> 
> Shutdown scripts are done by startkde after ksmserver quits. Which never
> happens because we've told systemd to shutdown.
> 
> Old systems worked because they used to communicate with the display
> manager which then closed us before shutting down, this is no longer the
> case.
> 
> This moves handling the shutdown scripts into ksmserver (which already
> handles startup scripts) and runs them before asking logind to shutdown.
> 
> BUG: 356190
> 
> 
> Diffs
> -
> 
>   ksmserver/server.h 1a2f0810c53d508d7d721fb4be1c25f0585c0602 
>   ksmserver/server.cpp 9477e542399ad65f993e581fc4212883f282b70e 
>   startkde/startkde.cmake 41a8975cce1fb2a4e7a034e697ce6e2cc59d5b1e 
>   startkde/startplasma.cmake 8360a636d3f68c957a15158484360a611cfe3ff8 
> 
> Diff: https://git.reviewboard.kde.org/r/126268/diff/
> 
> 
> Testing
> ---
> 
> had a shutdown script to touch a file in my home, worked for shutdown and 
> logout.
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

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


Re: Urgent: Google Code-In tasks needed

2015-12-07 Thread Sebastian Kügler
On Saturday, December 05, 2015 03:34:36 PM Mario Fux wrote:
> What about asking students to make each a video to explain the setup and
> use  of a single Plasma feature. Simple tasks. For people that have never
> seen Plasma or don't know about all the features. I mean really the
> "simple" stuff: - Digital clock: setup (add plasmoid somewhere), features
> and usage - User Switcher: setup, features and usage
> - Panels: setup, features and usage
> - ...
> 
> We'd need new tasks by tomorrow. Anybody willing to mentor and add them?
> ping  valorie, nightrose or bshah on IRC.
> 
> Let's show the power of Plasma via short videos. They could be redone (new 
> releases) every year at GCI?

That usually takes a huge amount of mentoring and review.

Sorry to say it, but I simply don't have time for this.
-- 
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: 5.5 video

2015-12-07 Thread Jonathan Riddell

Lovely, draft announcement updated

Jonathan


On Mon, Dec 07, 2015 at 06:56:53AM +0100, Lucas S wrote:
> Hi
> Thanks for all comments. I think that I fixed most (all?) remaining issues.
> 
> - Improved  visibility of the text by adding black transparent
> background and by moving to the beginning of the sequences
> - Fixed a typo
> - Added 2 more activities so filtering looks more natural now. (With
> more than 5 activities there is a scrollbar visible, which looks kinda
> out of place)
> - Removed a window with activities settings
> - Improved muon part (cropped "freedom")
> - Added simple fade in/fade out effect for the first and last title
> (but still think that 3d dissolving animation looked the best ;))
> 
> Link:
> https://youtu.be/DsSyPpHS6HI
> 
> Cheers
> Lucas
> ___
> 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: 5.5 video

2015-12-07 Thread Mario Fux
Am Monday 07 December 2015, 16.30:26 schrieb Lucas S:
> Hi,

Morning

> - Added simple cross fade effect between the end of the sequences and the
> titles
> 
> Link
> https://youtu.be/WZWmtAohp7s

Found another typo:
- 1.42: "managment" => "management"

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


Re: Review Request 126268: Move shutdown scripts into ksmserver cleanup

2015-12-07 Thread Alex Richardson

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



ksmserver/server.cpp (line 1105)


continue?


- Alex Richardson


On Dec. 7, 2015, 12:26 a.m., David Edmundson wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126268/
> ---
> 
> (Updated Dec. 7, 2015, 12:26 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> ---
> 
> Shutdown scripts are done by startkde after ksmserver quits. Which never
> happens because we've told systemd to shutdown.
> 
> Old systems worked because they used to communicate with the display
> manager which then closed us before shutting down, this is no longer the
> case.
> 
> This moves handling the shutdown scripts into ksmserver (which already
> handles startup scripts) and runs them before asking logind to shutdown.
> 
> BUG: 356190
> 
> 
> Diffs
> -
> 
>   ksmserver/server.h 1a2f0810c53d508d7d721fb4be1c25f0585c0602 
>   ksmserver/server.cpp 9477e542399ad65f993e581fc4212883f282b70e 
>   startkde/startkde.cmake 41a8975cce1fb2a4e7a034e697ce6e2cc59d5b1e 
>   startkde/startplasma.cmake 8360a636d3f68c957a15158484360a611cfe3ff8 
> 
> Diff: https://git.reviewboard.kde.org/r/126268/diff/
> 
> 
> Testing
> ---
> 
> had a shutdown script to touch a file in my home, worked for shutdown and 
> logout.
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

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


Re: Review Request 126275: Make the "Plasma (Wayland)" xsession file actually say "Plasma (Wayland)"

2015-12-07 Thread Bhushan Shah

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


-2

Where your session file is installed? if it is installed in wayland-sessions 
sddm will automatically append " (wayland)". Seems you are putting session file 
in Xsessions dir?

- Bhushan Shah


On Dec. 8, 2015, 10:30 a.m., Martin Klapetek wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126275/
> ---
> 
> (Updated Dec. 8, 2015, 10:30 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> ---
> 
> The commit/review says it's displayed as "Plasma (Wayland)"
> but the name is just "Plasma", making it confusing in login
> manager.
> 
> 
> Diffs
> -
> 
>   plasmawayland.desktop.cmake c5d5757 
> 
> Diff: https://git.reviewboard.kde.org/r/126275/diff/
> 
> 
> Testing
> ---
> 
> Sddm now clearly shows which "Plasma" is Waylandplasma.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

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