[Differential] [Closed] D3542: Remove outdated script for handling translations

2016-11-29 Thread ltoscano (Luigi Toscano)
This revision was automatically updated to reflect the committed changes. Closed by commit R119:3cf44a9101e0: Remove outdated script for handling translations (authored by ltoscano). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3542?vs=8614=8616

[Differential] [Commented On] D3538: Drop resize animation when adding pages

2016-11-29 Thread jensreuterberg (Jens Reuterberg)
jensreuterberg added a comment. Lovelly! <3 The animations remaining feel way cleaner REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D3538 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: apol, #kirigami, mart Cc:

[Differential] [Accepted] D3542: Remove outdated script for handling translations

2016-11-29 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D3542 EMAIL PREFERENCES

[Differential] [Request, 91 lines] D3542: Remove outdated script for handling translations

2016-11-29 Thread ltoscano (Luigi Toscano)
ltoscano created this revision. ltoscano added a reviewer: Plasma. ltoscano added a subscriber: plasma-devel. Restricted Application added a project: Plasma. REVISION SUMMARY The files were relevant when the modules lived in its own repository. Translations are now included in the

[Differential] [Request, 107 lines] D3541: Introduce a first test for PageRow

2016-11-29 Thread apol (Aleix Pol Gonzalez)
apol created this revision. apol added reviewers: Kirigami, mart. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Passes Makes sure that basic push and pop usage works. Makes sure that pressing the go back button

Re: [Technical Question] - How to trigger the applets actions menu from a second window of a Panel? (e.g. Now Dock 0.5)

2016-11-29 Thread Michail Vourlakos
Plasma::Applet *applet = 0; foreach (QObject *appletObject, m_appletInterfaces) { if (AppletInterface *ai = qobject_cast*>(appletObject)) { if (ai->isVisible() && ai->contains(ai->mapFromItem(this, event->posF( { applet = ai->applet();

[Differential] [Commented On] D3538: Drop resize animation when adding pages

2016-11-29 Thread colomar (Thomas Pfeiffer)
colomar added a comment. +1, definitely looks better to me (and Jens will love any killed animation, anyway ;) ) REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D3538 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: apol,

[Differential] [Commented On] D3539: [Notifications] Show interactive thumbnails on notifications

2016-11-29 Thread colomar (Thomas Pfeiffer)
colomar added a comment. Really nice! The single thumbnail might be a bit too large, as having large notification windows pop up could be irritating. Maybe half the size would be big enough? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3539 EMAIL

[Technical Question] - How to trigger the applets actions menu from a second window of a Panel? (e.g. Now Dock 0.5)

2016-11-29 Thread Michail Vourlakos
Hello everyone, A couple of days ago, in a discussion with plasma developers it was decided that Now Dock should better support window masking (for animations etc.) in a separate window than the main panel one. What is doing in the following 0.5 version is a technique based on a plasma

[Differential] [Updated] D3539: [Notifications] Show interactive thumbnails on notifications

2016-11-29 Thread broulik (Kai Uwe Broulik)
broulik updated the test plan for this revision. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3539 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, #plasma:_design Cc: plasma-devel, lesliezhai,

[Differential] [Request, 328 lines] D3539: [Notifications] Show interactive thumbnails on notifications

2016-11-29 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, Plasma: Design. broulik set the repository for this revision to R120 Plasma Workspace. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY When a notification

Review Request 129586: [KNotification] Allow annotating notifications with URLs

2016-11-29 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129586/ --- Review request for KDE Frameworks and Plasma. Repository: knotifications

[Differential] [Updated, 16 lines] D3501: Scale surface damage events

2016-11-29 Thread davidedmundson (David Edmundson)
davidedmundson updated this revision to Diff 8607. davidedmundson retitled this revision from "Scale surface damage events Damage events come in global compositor space, not relative to the surface. For a glTex2D we need the events relatvie to the surface, so they need scaling up" to "Scale

[Differential] [Updated, 2 lines] D3499: drm cursor scaling

2016-11-29 Thread davidedmundson (David Edmundson)
davidedmundson updated this revision to Diff 8605. davidedmundson added a comment. Restricted Application edited projects, added Plasma; removed KWin. Update line REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3499?vs=8507=8605 REVISION DETAIL

[Differential] [Request, 18 lines] D3538: Drop resize animation when adding pages

2016-11-29 Thread apol (Aleix Pol Gonzalez)
apol created this revision. apol added reviewers: Kirigami, mart. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY It's a bit glitchy and looks a bit odd TEST PLAN Current: https://youtu.be/pUjmqis9E-w NoAnimations:

Jenkins-kde-ci: plasma-desktop Plasma-5.8 stable-kf5-qt5 » Linux,gcc - Build # 68 - Fixed!

2016-11-29 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.8%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/68/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 29 Nov 2016 14:56:07 + Build duration: 6 min 7 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: plasma-desktop Plasma-5.8 stable-kf5-qt5 » Linux,gcc - Build # 68 - Fixed!

2016-11-29 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.8%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/68/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 29 Nov 2016 14:56:07 + Build duration: 6 min 7 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: plasma-desktop Plasma-5.8 stable-kf5-qt5 » Linux,gcc - Build # 67 - Failure!

2016-11-29 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.8%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/67/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 29 Nov 2016 14:42:33 + Build duration: 8 min 53 sec CHANGE SET Revision

[Differential] [Closed] D3533: Select correct index without timer

2016-11-29 Thread mart (Marco Martin)
This revision was automatically updated to reflect the committed changes. Closed by commit R119:d9d52a37c6ba: Select correct index without timer (authored by mart). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3533?vs=8575=8602 REVISION DETAIL

[Differential] [Closed] D3532: Select correct index without timer

2016-11-29 Thread mart (Marco Martin)
This revision was automatically updated to reflect the committed changes. Closed by commit R120:5434e4bc0141: Select correct index without timer (authored by mart). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3532?vs=8574=8601 REVISION DETAIL

[Differential] [Accepted] D3533: Select correct index without timer

2016-11-29 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop BRANCH phab/selectindex REVISION DETAIL https://phabricator.kde.org/D3533 EMAIL PREFERENCES

[Differential] [Updated, 20 lines] D3496: Scale input on windowed backend so input co-ordinate system matches host

2016-11-29 Thread davidedmundson (David Edmundson)
davidedmundson updated this revision to Diff 8600. davidedmundson added a comment. Restricted Application edited projects, added Plasma; removed KWin. Updated line REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3496?vs=8504=8600 REVISION DETAIL

[Differential] [Accepted] D3532: Select correct index without timer

2016-11-29 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. This revision is now accepted and ready to land. REPOSITORY R120 Plasma Workspace BRANCH phab/wallpselectedindex REVISION DETAIL https://phabricator.kde.org/D3532 EMAIL PREFERENCES

[Differential] [Accepted] D3425: If we get a notification with no body, treat the subject line as the body

2016-11-29 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D3425 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/

[Differential] [Closed] D3508: Implement borderlessMaximizedWindows for Wayland windows

2016-11-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit R108:f32e6550311f: Implement borderlessMaximizedWindows for Wayland windows (authored by graesslin). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D3508?vs=8522=8598#toc REPOSITORY R108 KWin

[Differential] [Closed] D3507: Properly implement maximize of ShellClient

2016-11-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit R108:9934f5b57537: Properly implement maximize of ShellClient (authored by graesslin). REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3507?vs=8521=8597 REVISION DETAIL

[Differential] [Accepted] D3508: Implement borderlessMaximizedWindows for Wayland windows

2016-11-29 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY R108 KWin BRANCH shell-client-borderless-maximized-windows REVISION DETAIL https://phabricator.kde.org/D3508 EMAIL PREFERENCES

[Differential] [Commented On] D3530: Import plasma-workspace kioslaves

2016-11-29 Thread mart (Marco Martin)
mart added a comment. In https://phabricator.kde.org/D3530#65867, @dfaure wrote: > But desktop:/ and applications:/ make no sense in other workspaces (I'm surprised we even still have applications:/, it's kind of a toy, isn't it?). > > (ok applications:/ might make sense in gnome,

[Differential] [Commented On] D3531: Drop kioslave folder, will be in KIO 5.30

2016-11-29 Thread elvisangelaccio (Elvis Angelaccio)
elvisangelaccio added inline comments. INLINE COMMENTS > bshah wrote in CMakeLists.txt:8 > You can't do this, framework version is bumped by release script > automatically, so when you push this there is no 5.30.0 version of KIO. Which > will make it fail, you need to bump version once

[Differential] [Changed Subscribers] D3531: Drop kioslave folder, will be in KIO 5.30

2016-11-29 Thread bshah (Bhushan Shah)
bshah added inline comments. INLINE COMMENTS > CMakeLists.txt:8 > set(QT_MIN_VERSION "5.6.1") > -set(KF5_MIN_VERSION "5.26.0") > +set(KF5_MIN_VERSION "5.30.0") > set(INSTALL_SDDM_THEME TRUE) You can't do this, framework version is bumped by release script automatically, so when you push this

[Differential] [Requested Changes To] D3472: API for window tabbing

2016-11-29 Thread Martin Gräßlin
graesslin requested changes to this revision. graesslin added a comment. This revision now requires changes to proceed. Overall looks quite good. Please watch the coding style a little bit. That created a few comments. There are two conceptional things I don't get. One is why you have

[Differential] [Updated] D3531: Drop kioslave folder, will be in KIO 5.30

2016-11-29 Thread elvisangelaccio (Elvis Angelaccio)
elvisangelaccio updated the summary for this revision. REPOSITORY R120 Plasma Workspace BRANCH drop-slaves REVISION DETAIL https://phabricator.kde.org/D3531 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: elvisangelaccio, davidedmundson, #plasma Cc:

[Differential] [Updated] D3530: Import plasma-workspace kioslaves

2016-11-29 Thread dfaure (David Faure)
dfaure added a comment. remote:/ sounds very workspace-independent indeed, it sounds useful to have in kio. But desktop:/ and applications:/ make no sense in other workspaces (I'm surprised we even still have applications:/, it's kind of a toy, isn't it?). (ok applications:/ might