Re: some feedback from a mobile application development team

2016-01-05 Thread Dirk Hohndel
Hi Thomas On Wed, Jan 06, 2016 at 02:56:09AM +0100, Thomas Pfeiffer wrote: You clearly have interesting working hours... just like Sebastian > There is a difference between edge swipe and regular horizontal swipes. All > stock Android apps that have a lef

Re: some feedback from a mobile application development team

2016-01-05 Thread Thomas Pfeiffer
On Dienstag, 5. Januar 2016 16:58:10 CET Dirk Hohndel wrote: > Hi Thomas, > > > On Jan 5, 2016, at 3:22 PM, Thomas Pfeiffer > > wrote: as the person in charge of the KDE Mobile HIG (and therefore also > > responsible for the usability of the Plasma Components), I'm very happy > > to get feedback

Re: some feedback from a mobile application development team

2016-01-05 Thread Dirk Hohndel
Hi Thomas, > On Jan 5, 2016, at 3:22 PM, Thomas Pfeiffer wrote: > as the person in charge of the KDE Mobile HIG (and therefore also responsible > for the usability of the Plasma Components), I'm very happy to get feedback > from users of an actual working application using our components! Awes

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/ --- (Updated Jan. 5, 2016, 11:40 nachm.) Review request for Plasma, KDE Usabi

Re: some feedback from a mobile application development team

2016-01-05 Thread Thomas Pfeiffer
On Dienstag, 5. Januar 2016 08:16:38 CET Dirk Hohndel wrote: > Hi there, Hi Dirk, as the person in charge of the KDE Mobile HIG (and therefore also responsible for the usability of the Plasma Components), I'm very happy to get feedback from users of an actual working application using our compon

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/ --- (Updated Jan. 5, 2016, 11:21 nachm.) Review request for Plasma, KDE Usabi

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/ --- (Updated Jan. 5, 2016, 11:15 nachm.) Review request for Plasma, KDE Usabi

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-05 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/#review90674 --- I agree with the album art suggestion - or we could build on y

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-05 Thread Thomas Pfeiffer
> On Jan. 5, 2016, 9:28 p.m., Martin Klapetek wrote: > > One idea I was thinking about for this case - why not replace > > the audio player window thumbnail with the album cover art? > > > > The window of the player is not really useful in the thumbnail; > > it's mostly just a window with some l

Re: robots.txt in quickgit.kde.org

2016-01-05 Thread Ben Cooksley
On Wed, Jan 6, 2016 at 3:17 AM, Kevin Funk wrote: > On Wednesday, December 30, 2015 12:57:23 PM Ben Cooksley wrote: >> On Tue, Dec 29, 2015 at 11:16 PM, Kevin Funk wrote: >> > On Tuesday, December 29, 2015 10:39:01 PM Ben Cooksley wrote: >> >> On Tue, Dec 29, 2015 at 7:59 AM, Lydia Pintscher wro

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-05 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/#review90672 --- One idea I was thinking about for this case - why not replace

Jenkins-kde-ci: plasma-workspace Plasma-5.5 stable-kf5-qt5 » Linux,gcc - Build # 33 - Still Failing!

2016-01-05 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-workspace%20Plasma-5.5%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/33/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 05 Jan 2016 19:50:14 + Build duration: 1 min 55 sec CHANGE SET Revision 80ccb1c88d41752fb41

Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/ --- Review request for Plasma, KDE Usability and Eike Hein. Bugs: 352126

Re: Review Request 126621: [Task Manager] Add support for Unity Launcher API and Application Jobs

2016-01-05 Thread Eike Hein
> On Jan. 4, 2016, 11:01 p.m., Eike Hein wrote: > > Ship It! > > Kai Uwe Broulik wrote: > I need a proper frame graphic, "hover" isn't particularly good but I > don't want to mess around with the tasks.svgz due to Review 126373 > > Kai Uwe Broulik wrote: > What am I supposed to do in t

Re: Review Request 126621: [Task Manager] Add support for Unity Launcher API and Application Jobs

2016-01-05 Thread Kai Uwe Broulik
> On Jan. 4, 2016, 11:01 nachm., Eike Hein wrote: > > Ship It! > > Kai Uwe Broulik wrote: > I need a proper frame graphic, "hover" isn't particularly good but I > don't want to mess around with the tasks.svgz due to Review 126373 What am I supposed to do in this case: the desktop file is

Re: Review Request 126637: IconItem: Add animated property

2016-01-05 Thread David Rosca
> On Jan. 5, 2016, 1:56 p.m., David Edmundson wrote: > > +1 from me > > > > Though AFIAK Kai was doing something related here too? > > David Rosca wrote: > Yes, but that is only to disable the animation when showing the IconItem > after being hidden. > This completely disables the anim

Re: Review Request 126637: IconItem: Add animated property

2016-01-05 Thread Eike Hein
> On Jan. 5, 2016, 1:56 p.m., David Edmundson wrote: > > +1 from me > > > > Though AFIAK Kai was doing something related here too? > > David Rosca wrote: > Yes, but that is only to disable the animation when showing the IconItem > after being hidden. > This completely disables the anim

Re: Review Request 126642: [Theme] Take Plasma Framework version into account in theme cache

2016-01-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126642/#review90664 --- I don't really understand the problem this is solving. This wo

Review Request 126642: [Theme] Take Plasma Framework version into account in theme cache

2016-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126642/ --- Review request for Plasma. Repository: plasma-framework Description ---

Re: Review Request 126641: [Units] scale desktop icon sizes

2016-01-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126641/#review90663 --- src/declarativeimports/core/units.cpp (line 119)

Re: Review Request 126641: [Units] scale desktop icon sizes

2016-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126641/ --- (Updated Jan. 5, 2016, 5:41 p.m.) Status -- This change has been mar

Re: Review Request 126641: [Units] scale desktop icon sizes

2016-01-05 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126641/#review90662 --- Ship it! Hm, that was the idea all along, and I recall having

Review Request 126641: [Units] scale desktop icon sizes

2016-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126641/ --- Review request for Plasma. Repository: plasma-framework Description ---

Re: Review Request 126636: TaskManager: Enable vertical scroll in tooltip

2016-01-05 Thread Kai Uwe Broulik
> On Jan. 5, 2016, 2:02 nachm., David Edmundson wrote: > > applets/taskmanager/package/contents/ui/ToolTipDelegate.qml, line 62 > > > > > > I'm a bit confused. > > > > We explicitly disable interactive (s

Re: Review Request 126636: TaskManager: Enable vertical scroll in tooltip

2016-01-05 Thread David Rosca
> On Jan. 5, 2016, 2:02 p.m., David Edmundson wrote: > > applets/taskmanager/package/contents/ui/ToolTipDelegate.qml, line 62 > > > > > > I'm a bit confused. > > > > We explicitly disable interactive (scr

Re: Review Request 126636: TaskManager: Enable vertical scroll in tooltip

2016-01-05 Thread Marco Martin
> On Jan. 5, 2016, 2:02 p.m., David Edmundson wrote: > > applets/taskmanager/package/contents/ui/ToolTipDelegate.qml, line 62 > > > > > > I'm a bit confused. > > > > We explicitly disable interactive (scr

some feedback from a mobile application development team

2016-01-05 Thread Dirk Hohndel
Hi there, I just subscribed so please bear with me if I'm going over things that have been discussed. Some background for context: Subsurface is a reasonably popular open source dive log program aimed primarily at scuba divers. It's somewhat known in open source circles as it was started by Linu

Jenkins-kde-ci: plasma-workspace Plasma-5.5 stable-kf5-qt5 » Linux,gcc - Build # 32 - Still Failing!

2016-01-05 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-workspace%20Plasma-5.5%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/32/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 05 Jan 2016 16:03:01 + Build duration: 1 min 52 sec CHANGE SET Revision cccf297b25f8d03903d

Re: Review Request 126632: add a "Complementary" color scheme to kcolorscheme

2016-01-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126632/#review90655 --- +1 makes sense to me. - Aleix Pol Gonzalez On Jan. 5, 2016,

Review Request 126640: [WindowThumbnail] Discard pixmap on map events.

2016-01-05 Thread Mihail Ivchenko
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126640/ --- Review request for Plasma. Repository: plasma-framework Description ---

Re: Review Request 126636: TaskManager: Enable vertical scroll in tooltip

2016-01-05 Thread David Rosca
> On Jan. 5, 2016, 2:02 p.m., David Edmundson wrote: > > applets/taskmanager/package/contents/ui/ToolTipDelegate.qml, line 62 > > > > > > I'm a bit confused. > > > > We explicitly disable interactive (scr

Re: Plasma 5.6 schedule

2016-01-05 Thread Jonathan Riddell
No comments on this so let's go with it Jonathan On Sat, Dec 26, 2015 at 04:07:46PM +, Jonathan Riddell wrote: > Here's a proposed schedule for Plasma 5.6 as discussed at the kickoff meeting > > https://techbase.kde.org/Schedules/Plasma_5 > https://calendar.google.com/calendar/embed?src=03

Re: Review Request 126636: TaskManager: Enable vertical scroll in tooltip

2016-01-05 Thread David Rosca
> On Jan. 5, 2016, 2:02 p.m., David Edmundson wrote: > > how do you get a situation where you need vertical scrolling? I want to scroll with mouse wheel (vertical) through all windows. Without this patch, I must scroll horizontally, which is slow on my mouse and also not every mouse even can s

Re: Review Request 126637: IconItem: Add animated property

2016-01-05 Thread David Rosca
> On Jan. 5, 2016, 1:56 p.m., David Edmundson wrote: > > +1 from me > > > > Though AFIAK Kai was doing something related here too? Yes, but that is only to disable the animation when showing the IconItem after being hidden. This completely disables the animation, which I think would be useful

Re: Review Request 126636: TaskManager: Enable vertical scroll in tooltip

2016-01-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126636/#review90648 --- how do you get a situation where you need vertical scrolling?

Re: Review Request 126637: IconItem: Add animated property

2016-01-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126637/#review90647 --- +1 from me Though AFIAK Kai was doing something related here

Re: Review Request 126635: TaskManager: Fix tooltip overflowing screen size

2016-01-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126635/#review90646 --- Ship it! - David Edmundson On Jan. 5, 2016, 12:52 p.m., Dav

Re: Review Request 126629: update the sddm kcm first step

2016-01-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126629/#review90645 --- Ship it! Ship It! - David Edmundson On Jan. 5, 2016, 2:34

Re: Review Request 126629: update the sddm kcm first step

2016-01-05 Thread andreas kainz
> On Jan. 5, 2016, 12:45 nachm., Aleix Pol Gonzalez wrote: > > What do you mean by "align right"? You want it to take all the horizontal > > space? the mail adddress and Version should be right align to the preview - andreas --- This i

Review Request 126637: IconItem: Add animated property

2016-01-05 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126637/ --- Review request for Plasma. Repository: plasma-framework Description ---

Review Request 126636: TaskManager: Enable vertical scroll in tooltip

2016-01-05 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126636/ --- Review request for Plasma. Repository: plasma-desktop Description -

Review Request 126635: TaskManager: Fix tooltip overflowing screen size

2016-01-05 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126635/ --- Review request for Plasma. Repository: plasma-desktop Description -

Re: Review Request 126629: update the sddm kcm first step

2016-01-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126629/#review90642 --- What do you mean by "align right"? You want it to take all the

Re: Review Request 126634: SystemTray: Fix height of lines in table in Entries config

2016-01-05 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126634/ --- (Updated Jan. 5, 2016, 12:34 p.m.) Status -- This change has been ma

Re: Review Request 126634: SystemTray: Fix height of lines in table in Entries config

2016-01-05 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126634/#review90641 --- Ship it! Ship It! - Sebastian Kügler On Jan. 5, 2016, 12:2

Review Request 126634: SystemTray: Fix height of lines in table in Entries config

2016-01-05 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126634/ --- Review request for Plasma. Repository: plasma-workspace Description ---

Re: Review Request 126630: Do not unconditionally enable logging

2016-01-05 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126630/#review90638 --- Please also disable it in the same way for the kwayland backen

[Differential] [Request, 195 lines] D742: Wayland integration: support for server-side decorations

2016-01-05 Thread Martin Gräßlin
graesslin created this revision. graesslin added reviewers: mart, sebas. graesslin added a subscriber: plasma-devel. graesslin set the repository for this revision to rPLASMAINTEGRATION Integration for Qt applications in Plasma. REVISION SUMMARY If the application runs on Wayland, check whether

Re: Review Request 126630: Do not unconditionally enable logging

2016-01-05 Thread Peter Wu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126630/ --- (Updated Jan. 5, 2016, 1:08 p.m.) Review request for Plasma, Solid, Danie

[Differential] [Abandoned] D741: Adding .arcconfig

2016-01-05 Thread Martin Gräßlin
graesslin abandoned this revision. REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D741 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, sebas, mart Cc: plasma-devel __

Re: Review Request 126630: Do not unconditionally enable logging

2016-01-05 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126630/#review90637 --- backends/fake/fake.cpp

Re: Review Request 126373: change the taskbar color from blue to gray

2016-01-05 Thread andreas kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126373/ --- (Updated Jan. 5, 2016, noon) Status -- This change has been marked a

[Differential] [Request, 199 lines] D741: Adding .arcconfig

2016-01-05 Thread Martin Gräßlin
graesslin created this revision. graesslin added reviewers: sebas, mart. graesslin added a subscriber: plasma-devel. REVISION SUMMARY Adding .arcconfig Wayland integration: support for server-side decorations If the application runs on Wayland, check whether the ServerSideDecoration

Re: Review Request 126373: change the taskbar color from blue to gray

2016-01-05 Thread Andreas ka
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126373/#review90636 --- Ship it! Ship It! - Andreas ka On Dez. 16, 2015, 7:23 nach

Review Request 126630: Do not unconditionally enable logging

2016-01-05 Thread Peter Wu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126630/ --- Review request for Plasma, Solid, Daniel Vrátil, and Sebastian Kügler. Re

Re: Review Request 126632: add a "Complementary" color scheme to kcolorscheme

2016-01-05 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126632/ --- (Updated Jan. 5, 2016, 11:16 a.m.) Review request for KDE Frameworks and

Review Request 126632: add a "Complementary" color scheme to kcolorscheme

2016-01-05 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126632/ --- Review request for KDE Frameworks and Plasma. Repository: kconfigwidgets

[Differential] [Closed] D740: Wayland integration: support for server-side decorations

2016-01-05 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAINTEGRATION5ba5cdf2326c: Adding .arcconfig (authored by graesslin). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D740?vs=1744&id=1746#toc REPOSITORY rPLASMAINTEGRATION Integration for Qt

Re: Request for delaying Plasma 5.5.2 by a day

2016-01-05 Thread Jonathan Riddell
Tomorrow should be fine, just ping me when you're happy with it Jonathan On 5 January 2016 at 05:04, Martin Klapetek wrote: > Hey all, > > I was hoping the [1] would get at least one more > tester over the Christmas but I got none. I consider > that bug that crept in 5.5 to be quite serious (so

Re: Review Request 126373: change the taskbar color from blue to gray

2016-01-05 Thread Marco Martin
> On Dec. 20, 2015, 8:45 p.m., Eike Hein wrote: > > General PSA: I will soon push a fixed-up version of tasks.svgz to the repo > > that cleans up all the random margins and inconsistent corner styles in it > > that currently mess up alignments inside and outside of task buttons as > > well as

[Differential] [Updated] D740: Wayland integration: support for server-side decorations

2016-01-05 Thread Martin Gräßlin
graesslin retitled this revision from "Adding .arcconfig" to "Wayland integration: support for server-side decorations". REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D740 EMAIL PREFERENCES https://phabricator.kde.org/s

[Differential] [Commented On] D740: Adding .arcconfig

2016-01-05 Thread Martin Gräßlin
graesslin added inline comments. INLINE COMMENTS .arcconfig:1 This file is already in master. Should not have been included. Trying to learn phab ;-) REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D740 EMAIL PREFERENCES

[Differential] [Request, 199 lines] D740: Adding .arcconfig

2016-01-05 Thread Martin Gräßlin
graesslin created this revision. graesslin added reviewers: sebas, mart. graesslin added a subscriber: plasma-devel. REVISION SUMMARY Wayland integration: support for server-side decorations If the application runs on Wayland, check whether the ServerSideDecoration protocol is supported.

[Differential] [Accepted] D739: Disable ptrace

2016-01-05 Thread bshah (Bhushan Shah)
bshah accepted this revision. bshah added a comment. This revision is now accepted and ready to land. Looks good to me. REPOSITORY rPOLKITKDEAGENT Policykit (Polkit) KDE Agent BRANCH disable-ptrace REVISION DETAIL https://phabricator.kde.org/D739 EMAIL PREFERENCES https://phabricator.

[Differential] [Updated] D739: Disable ptrace

2016-01-05 Thread Martin Gräßlin
graesslin added reviewers: sitter, lukas, bshah. graesslin added a subscriber: plasma-devel. REPOSITORY rPOLKITKDEAGENT Policykit (Polkit) KDE Agent REVISION DETAIL https://phabricator.kde.org/D739 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graessli