shared maintenance of APIs for accessing online multimedia content

2017-12-30 Thread Matthieu Gallien
Hello,

As part of the development of the elisa music player, I would like to add the 
ability to access online content. I know that the Plasma MediaCenter project 
has written such code for youtube (at least).

Is there any interest to try to form a common team to maintain such code in a 
form that would benefit everybody (Plasma MediaCenter, Babe, Elisa, ...) ?

I am convinced that would be the most efficient way to maintain such code 
instead of having everybody come with its own special code.

Best regards

--
Matthieu Gallien


D9487: [effects/slide] Handle moving clients

2017-12-30 Thread Vlad Zagorodniy
zzag added a comment.
Restricted Application edited projects, added Plasma; removed KWin.


  Hmm, weird.. I've just applied for a developer account. If I get commit 
rights, I will land them.
  
  Happy holidays! :)

REPOSITORY
  R108 KWin

BRANCH
  master

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

To: zzag, #kwin, #plasma, graesslin
Cc: graesslin, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart


D9070: KDE platform plugin: don't force default stylename on user-specified fonts

2017-12-30 Thread René J . V . Bertin
rjvbb added a comment.


  >   So this this actually resolve 
https://bugs.kde.org/show_bug.cgi?id=378523, or just lay the groundwork for 
resolving it?
  
  No, it's more a change similar to proposed fix for that bug (don't set a 
stylename in KFontRequester).
  This patch implements the principle that you shouldn't set a style name if 
you cannot guarantee that the side-effects of that change are without 
consequence. The platform theme plugin loads the desktop fonts and if it 
imposes a style name on them no application can set them to bold or italic 
anymore.
  
  So you could say that this patch is necessary for the KFontRequester fix to 
work as intended (as far as the desktop fonts are concerned).

REPOSITORY
  R135 Integration for Qt applications in Plasma

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

To: rjvbb, #frameworks, davidedmundson, graesslin, cfeck, dfaure
Cc: anthonyfieroni, ngraham, cfeck, fvogt, plasma-devel, ZrenBot, progwolff, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D9070: KDE platform plugin: don't force default stylename on user-specified fonts

2017-12-30 Thread Nathaniel Graham
ngraham added a comment.


  So this this actually resolve https://bugs.kde.org/show_bug.cgi?id=378523, or 
just lay the groundwork for resolving it?

REPOSITORY
  R135 Integration for Qt applications in Plasma

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

To: rjvbb, #frameworks, davidedmundson, graesslin, cfeck, dfaure
Cc: anthonyfieroni, ngraham, cfeck, fvogt, plasma-devel, ZrenBot, progwolff, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D9070: KDE platform plugin: don't force default stylename on user-specified fonts

2017-12-30 Thread Anthony Fieroni
anthonyfieroni added a reviewer: dfaure.
anthonyfieroni added a comment.


  I'm not sure it's Qt bug, it change behavior and now it doing in *right* way. 
So i'm patch to be merged +1.

REPOSITORY
  R135 Integration for Qt applications in Plasma

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

To: rjvbb, #frameworks, davidedmundson, graesslin, cfeck, dfaure
Cc: anthonyfieroni, ngraham, cfeck, fvogt, plasma-devel, ZrenBot, progwolff, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D7747: Added an extra fuzzytime setting - Hobbit Time

2017-12-30 Thread Luigi Toscano
ltoscano added a comment.


  Also, according those links:
  
  - 
http://askmiddlearth.tumblr.com/post/41765286488/the-seven-daily-hobbit-meals
  - 
https://scifi.stackexchange.com/questions/105209/what-are-hobbits-daily-eating-schedules
  
  dinner and supper are only differentiated in the movies (another hint for the 
translators)

REPOSITORY
  R114 Plasma Addons

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

To: jayturner, ngraham
Cc: ltoscano, jensreuterberg, broulik, ngraham, davidedmundson, plasma-devel, 
ZrenBot, progwolff, lesliezhai, ali-mohamed, abetts, sebas, apol, mart


D7747: Added an extra fuzzytime setting - Hobbit Time

2017-12-30 Thread Luigi Toscano
ltoscano added a comment.


  Late to the party, but as those strings are connected to The Lord of the 
Rings books, it would be nice to add a proper context for those messages, so 
that translators can go and look in their copy of LOTR/Hobbit/etc

REPOSITORY
  R114 Plasma Addons

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

To: jayturner, ngraham
Cc: ltoscano, jensreuterberg, broulik, ngraham, davidedmundson, plasma-devel, 
ZrenBot, progwolff, lesliezhai, ali-mohamed, abetts, sebas, apol, mart


D9542: Improve display of technical app info

2017-12-30 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes.
Closed by commit R134:bc2e71d3a877: Improve display of technical app info 
(authored by ngraham).

REPOSITORY
  R134 Discover Software Store

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D9542?vs=24470=24489

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

AFFECTED FILES
  discover/qml/ApplicationPage.qml

To: ngraham, apol, #vdg, #discover_software_store
Cc: colomar, januz, alexeymin, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D9542: Improve display of technical app info

2017-12-30 Thread Nathaniel Graham
ngraham added a comment.


  We already had everything in the main content area before we had a real 
toolbar on top, at which point the icon and app name were moved there. I'll 
land this, and then we can continue iterating as necessary.

REPOSITORY
  R134 Discover Software Store

BRANCH
  master

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

To: ngraham, apol, #vdg, #discover_software_store
Cc: colomar, januz, alexeymin, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D9542: Improve display of technical app info

2017-12-30 Thread Aleix Pol Gonzalez
apol accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R134 Discover Software Store

BRANCH
  master

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

To: ngraham, apol, #vdg, #discover_software_store
Cc: colomar, januz, alexeymin, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D9542: Improve display of technical app info

2017-12-30 Thread Thomas Pfeiffer
colomar added a comment.


  This layout makes more sense to me as well overall. The only thing that I 
find a bit strange is that the summary now looks a bit like a headline, 
especially given that the application name is in the toolbar instead of the 
content area. I'm wondering if we should move the name and icon into the 
content area.

REPOSITORY
  R134 Discover Software Store

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

To: ngraham, apol, #vdg, #discover_software_store
Cc: colomar, januz, alexeymin, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D9487: [effects/slide] Handle moving clients

2017-12-30 Thread Martin Flöser
graesslin added a comment.
Restricted Application edited projects, added KWin; removed Plasma.


  I'm not able to apply it with arc patch. It complains about a missing base 
commit and tries to download from github which I don't use.

REPOSITORY
  R108 KWin

BRANCH
  master

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

To: zzag, #kwin, #plasma, graesslin
Cc: graesslin, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart