Re: Review Request 117041: kdeplasma-addons/applets/pastebin: add two new options

2014-03-27 Thread Aleix Pol Gonzalez
> On March 25, 2014, 12:41 a.m., Aleix Pol Gonzalez wrote: > > The frameworks version is ready, but the GUI is a complete rewrite as in > > Plasma 2, C++ plasmoids are not supported. > > > > Personally, I don't think these options are needed: > > 1. I've never had a problem with accidentally sh

Re: Review Request 117041: kdeplasma-addons/applets/pastebin: add two new options

2014-03-27 Thread Gregor Mi
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117041/ --- (Updated March 27, 2014, 11:54 p.m.) Review request for Plasma, Aleix Pol

Re: Review Request 117041: kdeplasma-addons/applets/pastebin: add two new options

2014-03-27 Thread Gregor Mi
> On March 25, 2014, 12:41 a.m., Aleix Pol Gonzalez wrote: > > The frameworks version is ready, but the GUI is a complete rewrite as in > > Plasma 2, C++ plasmoids are not supported. > > > > Personally, I don't think these options are needed: > > 1. I've never had a problem with accidentally sh

Re: Review Request 117041: kdeplasma-addons/applets/pastebin: add two new options

2014-03-27 Thread Gregor Mi
> On March 25, 2014, 12:34 p.m., Dominik Haumann wrote: > > applets/pastebin/pastebin.cpp, line 763 > > > > > > You could use the dontAskAgainName here to add a checkbox to the dialog: > > KMessageBox::questi

Re: Review Request 117041: kdeplasma-addons/applets/pastebin: add two new options

2014-03-27 Thread Gregor Mi
> On March 25, 2014, 12:34 p.m., Dominik Haumann wrote: > > applets/pastebin/pastebin.cpp, line 763 > > > > > > You could use the dontAskAgainName here to add a checkbox to the dialog: > > KMessageBox::questi

Re: Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread David Edmundson
> On March 27, 2014, 5:57 p.m., Marco Martin wrote: > > "break" as in.. ? > > David Edmundson wrote: > Add kickoff to the desktop. Resize it so it's compact then hover over it. > Broken. > > Marco Martin wrote: > uhm, works for me? http://wstaw.org/m/2014/03/27/plasma-desktopLW1833.png

Re: Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread Marco Martin
> On March 27, 2014, 5:57 p.m., Marco Martin wrote: > > "break" as in.. ? > > David Edmundson wrote: > Add kickoff to the desktop. Resize it so it's compact then hover over it. > Broken. > > Marco Martin wrote: > uhm, works for me? http://wstaw.org/m/2014/03/27/plasma-desktopLW1833.png

Re: Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread David Edmundson
> On March 27, 2014, 5:57 p.m., Marco Martin wrote: > > "break" as in.. ? > > David Edmundson wrote: > Add kickoff to the desktop. Resize it so it's compact then hover over it. > Broken. > > Marco Martin wrote: > uhm, works for me? http://wstaw.org/m/2014/03/27/plasma-desktopLW1833.png

Re: Changes in plasma desktop scripting

2014-03-27 Thread Marco Martin
On Thursday 27 March 2014 21:39:12 Marco Martin wrote: > Hi all, > > we had still a problem in the plasma desktop scripting api, because it comes > from a time when one containment == one activity, therefore right now > adding new activities from the script is pretty much broken. > > Since now is

Re: Review Request 116907: Implemented "Player" MPRIS spec adaptor

2014-03-27 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116907/#review54375 --- This review has been submitted with commit 9d7fb9c1efa1228c3a

Changes in plasma desktop scripting

2014-03-27 Thread Marco Martin
Hi all, we had still a problem in the plasma desktop scripting api, because it comes from a time when one containment == one activity, therefore right now adding new activities from the script is pretty much broken. Since now is the only moment we can screw the api, today I made the following

Re: Review Request 116907: Implemented "Player" MPRIS spec adaptor

2014-03-27 Thread Ashish Madeti
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116907/ --- (Updated March 27, 2014, 8:33 p.m.) Status -- This change has been m

Re: Review Request 116907: Implemented "Player" MPRIS spec adaptor

2014-03-27 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116907/#review54371 --- This review has been submitted with commit 2b9d28e02660906a41

Re: Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread Marco Martin
> On March 27, 2014, 5:57 p.m., Marco Martin wrote: > > "break" as in.. ? > > David Edmundson wrote: > Add kickoff to the desktop. Resize it so it's compact then hover over it. > Broken. > > Marco Martin wrote: > uhm, works for me? http://wstaw.org/m/2014/03/27/plasma-desktopLW1833.png

Re: Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread Marco Martin
> On March 27, 2014, 5:57 p.m., Marco Martin wrote: > > "break" as in.. ? > > David Edmundson wrote: > Add kickoff to the desktop. Resize it so it's compact then hover over it. > Broken. > > Marco Martin wrote: > uhm, works for me? http://wstaw.org/m/2014/03/27/plasma-desktopLW1833.png

Re: Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread Marco Martin
> On March 27, 2014, 5:57 p.m., Marco Martin wrote: > > "break" as in.. ? > > David Edmundson wrote: > Add kickoff to the desktop. Resize it so it's compact then hover over it. > Broken. uhm, works for me? http://wstaw.org/m/2014/03/27/plasma-desktopLW1833.png - Marco -

Re: Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread David Edmundson
> On March 27, 2014, 5:57 p.m., Marco Martin wrote: > > "break" as in.. ? Add kickoff to the desktop. Resize it so it's compact then hover over it. Broken. - David --- This is an automatically generated e-mail. To reply, visit: https:/

Re: Review Request 117058: plasma-nm CMake fixes

2014-03-27 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117058/ --- (Updated March 27, 2014, 6:59 p.m.) Status -- This change has been m

Re: Review Request 117058: plasma-nm CMake fixes

2014-03-27 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117058/#review54359 --- This review has been submitted with commit 1984f5d1fa63eeb8d2

Re: Review Request 117058: plasma-nm CMake fixes

2014-03-27 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117058/#review54360 --- This review has been submitted with commit 647158be634408413a

Re: Plasma 2014.6 Alpha 1

2014-03-27 Thread Sebastian Kügler
On Thursday, March 27, 2014 17:55:41 Jonathan Riddell wrote: > Alpha 1 is due out on Tuesday but due to the splitting of kde-runtme > and kde-workspace I'm going to try and get it tagged and tarred > tomorrow. If you have any must have patched to get in time to do it > toot sweet. > > I'll also t

Re: Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117121/#review54355 --- "break" as in.. ? - Marco Martin On March 27, 2014, 5:34 p.

Plasma 2014.6 Alpha 1

2014-03-27 Thread Jonathan Riddell
Alpha 1 is due out on Tuesday but due to the splitting of kde-runtme and kde-workspace I'm going to try and get it tagged and tarred tomorrow. If you have any must have patched to get in time to do it toot sweet. I'll also tar up oxygen-fonts as Plasma now uses it by default. Then I'll send it

Re: Review Request 117101: This patch fixes font size behaviour when pmc is resized.

2014-03-27 Thread Shantanu Tushar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117101/#review54353 --- Doesn't really look on my box http://i.imgur.com/nJMYi9E.jpg A

Review Request 117121: Add property to determine if popups should be inside or outside the parentItem

2014-03-27 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117121/ --- Review request for Plasma. Repository: plasma-framework Description ---

Re: Review Request 116874: Patch for the next and previous button functionality in an inactive playlist.

2014-03-27 Thread Shantanu Tushar
> On March 27, 2014, 1:05 p.m., Sebastian Kügler wrote: > > mediaelements/playlist/Playlist.qml, line 159 > > > > > > indenting is wrong now > > R.Harish Navnit wrote: > I have added a new diff file after I

Re: Review Request 116874: Patch for the next and previous button functionality in an inactive playlist.

2014-03-27 Thread R.Harish Navnit
> On March 27, 2014, 1:05 p.m., Sebastian Kügler wrote: > > mediaelements/playlist/Playlist.qml, line 159 > > > > > > indenting is wrong now I have added a new diff file after I pulled the latest source and buil

Re: Review Request 117058: plasma-nm CMake fixes

2014-03-27 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117058/ --- (Updated March 27, 2014, 4:02 p.m.) Review request for Plasma. Changes

Re: Review Request 116874: Patch for the next and previous button functionality in an inactive playlist.

2014-03-27 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116874/#review54293 --- If a button doesn't do anything, it should be indicated by dis

Re: The happy bugs thread

2014-03-27 Thread Marco Martin
On Thursday 27 March 2014 16:24:14 R.Harish Navnit wrote: > On Thu, Mar 27, 2014 at 3:38 PM, Marco Martin wrote: > > i was wondering this moring why the cunt was so smaller than yesterday ;) > > I'm sorry but how's that even related ? :P > A typo perhaps ? > > Anyways, awesome work indeed ! oh,

Re: Review Request 116874: Patch for the next and previous button functionality in an inactive playlist.

2014-03-27 Thread R.Harish Navnit
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116874/ --- (Updated March 27, 2014, 11:22 a.m.) Review request for Plasma, Shantanu

Re: Review Request 116907: Implemented "Player" MPRIS spec adaptor

2014-03-27 Thread Alex Merry
> On March 26, 2014, 6:09 p.m., Alex Merry wrote: > > All looks sane. Is this Qt4 or Qt5? Because you've got a KUrl in the > > added code... > > Bhushan Shah wrote: > Qt4 OK, that's fine then. - Alex --- This is an automatically

Re: The happy bugs thread

2014-03-27 Thread Bhushan Shah
On Thu, Mar 27, 2014 at 3:28 PM, David Edmundson wrote: > which shows just how many bugs we've fixed. never mind, I am going to report many more! ;P -- Bhushan Shah http://bhush9.github.io IRC Nick : bshah on Freenode ___ Plasma-devel mailing list Pl

Re: The happy bugs thread

2014-03-27 Thread R.Harish Navnit
On Thu, Mar 27, 2014 at 3:38 PM, Marco Martin wrote: > > i was wondering this moring why the cunt was so smaller than yesterday ;) > I'm sorry but how's that even related ? :P A typo perhaps ? Anyways, awesome work indeed ! Cheers. R.Harish Navnit The Enigma

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

2014-03-27 Thread Bhushan Shah
https://bugs.kde.org/show_bug.cgi?id=328593 Bhushan Shah changed: What|Removed |Added Status|UNCONFIRMED |CONFIRMED Ever confirmed|0

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

2014-03-27 Thread Bhushan Shah
https://bugs.kde.org/show_bug.cgi?id=328593 Bhushan Shah changed: What|Removed |Added CC||bhus...@gmail.com Depends on|

Re: The happy bugs thread

2014-03-27 Thread Marco Martin
On Thursday 27 March 2014 10:58:47 David Edmundson wrote: > Yesterday we started a bug triage session. BShah then joined in and > went completely nuts. Results are looking awesome: > > https://bugs.kde.org/reports.cgi?product=plasma-shell&output=show_chart&data > sets=CONFIRMED&datasets=ASSIGNED&d

The happy bugs thread

2014-03-27 Thread David Edmundson
Yesterday we started a bug triage session. BShah then joined in and went completely nuts. Results are looking awesome: https://bugs.kde.org/reports.cgi?product=plasma-shell&output=show_chart&datasets=CONFIRMED&datasets=ASSIGNED&datasets=REOPENED&datasets=UNCONFIRMED In general 'unconfirmed' bugs

Re: kde-l10n-* pre-releases rolled from wrong l10n branch

2014-03-27 Thread Harald Sitter
On Thu, Mar 27, 2014 at 10:46 AM, Albert Astals Cid wrote: > That is known and will be fixed for the final release of 4.13 > > Cheers, > Albert > > P.S: I thought noone would realize, good work :D lol :P ___ Plasma-devel mailing list Plasma-devel@kde.

Re: kde-l10n-* pre-releases rolled from wrong l10n branch

2014-03-27 Thread Albert Astals Cid
That is known and will be fixed for the final release of 4.13 Cheers,   Albert P.S: I thought noone would realize, good work :D - Mensaje original - De: Harald Sitter Para: KDE release coordination ; "kde-i18n-...@kde.org" ; plasma-devel@kde.org CC: Enviado: Jueves 27 de marzo de 2

kde-l10n-* pre-releases rolled from wrong l10n branch

2014-03-27 Thread Harald Sitter
Aloha It was brought to my attention that both czech and slovenian language tars apparently do not result in a fully translated plasmas for 4.12.95 (e.g. [1]). Looking at the cs tar I see no launcher po but a kickoff po: ~/tmp/kde-l10n-cs-4.12.95/messages/kde-workspace  ls *kickoff* plasma_apple