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

2016-01-06 Thread Heiko Tietze
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/#review90685 --- The blurred shadow looks wrong. As reference I would use the H

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

2016-01-06 Thread Andreas ka
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126629/#review90686 --- File Attachment: how it looks after the update see preview se

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Thomas Pfeiffer wrote: > Therefore what I'd like to see is that when our components are used within > Android, they draw the buttons in the top bar (where they are - and that is > a research-supported fact - hard to reach, but consistent with other > Android applicatio

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Dirk Hohndel wrote: > Side note: I am not a huge friend of the side swipes being used for the > drawers. I find swipes a great way to do navigation. There are a bunch of > email applications where swiping left and right you can move to the > previous / next item in you

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Thomas Pfeiffer wrote: > > Support a "first start" mode for Plasma Mobile Components based apps that > > demonstrates to the user how things get used. So it shows a see through > > finger that taps and holds (the pulsing circles around the touch area) > > the FAB and m

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

2016-01-06 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

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Tuesday 05 January 2016, Dirk Hohndel wrote: > Hi there, > > I just subscribed so please bear with me if I'm going over things that > have been discussed. Welcome aboard Dirk! > The Subsurface team would love to see an option to cleanly disable the > action button in the MobileComponents. >

Re: Review Request 126637: IconItem: Add animated property

2016-01-06 Thread Marco Martin
> 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 126647: [Task Manager] Provide media controls in tooltips

2016-01-06 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/#review90689 --- I like the idea a lot, the detail i don't like much is the gra

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

2016-01-06 Thread Marco Martin
> On Jan. 6, 2016, 9:08 a.m., Heiko Tietze wrote: > > The blurred shadow looks wrong. As reference I would use the HIG about > > images https://techbase.kde.org/Projects/Usability/HIG/Layout/Image "Use a > > semi-transparent background behind the (was: caption text) controls to > > preserve co

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

2016-01-06 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/34/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 06 Jan 2016 11:34:16 + Build duration: 56 sec CHANGE SET Revision d8985dde54bd05c352d190c7d

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

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

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

2016-01-06 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/#review90691 --- +1 i think it's pure sexy now :D - Marco Martin On Jan. 6,

Re: Review Request 126637: IconItem: Add animated property

2016-01-06 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126637/#review90692 --- I'm not arguing against the addition of this property, my patc

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

2016-01-06 Thread Andreas ka
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126629/ --- (Updated Jan. 6, 2016, 12:25 p.m.) Status -- This change has been ma

Re: Review Request 126637: IconItem: Add animated property

2016-01-06 Thread Marco Martin
> On Jan. 6, 2016, 12:14 p.m., Kai Uwe Broulik wrote: > > I'm not arguing against the addition of this property, my patch should > > significantly reduce the need for it and prevent abusing it, however. > > > > Most cases where you don't want to fade the pixmap is when the view it's in > > was

Review Request 126652: [Units] Return at least 1ms for durations

2016-01-06 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126652/ --- Review request for Plasma. Bugs: 357352 http://bugs.kde.org/show_bug.

Re: Review Request 126652: [Units] Return at least 1ms for durations

2016-01-06 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126652/#review90694 --- On the other hand, given we don't have any UI config for this

Re: Review Request 126652: [Units] Return at least 1ms for durations

2016-01-06 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126652/ --- (Updated Jan. 6, 2016, 12:42 nachm.) Review request for Plasma. Bugs: 3

Re: Review Request 126652: [Units] Return at least 1ms for durations

2016-01-06 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126652/#review90695 --- Ship it! Ha!, interesting.. i guess/hope with 1 ms it will st

Review Request 126653: Revert "Don't fade animate icon in plasma tooltips"

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

Re: Review Request 126637: IconItem: Add animated property

2016-01-06 Thread David Rosca
> On Jan. 6, 2016, 12:14 p.m., Kai Uwe Broulik wrote: > > I'm not arguing against the addition of this property, my patch should > > significantly reduce the need for it and prevent abusing it, however. > > > > Most cases where you don't want to fade the pixmap is when the view it's in > > was

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Thomas Pfeiffer wrote: > Another thing I want to see changed in our components, though is the > actual dragging interaction with the FAB. Currently one has to drag it > pretty far to open a drawer, which defeats the whole "Not having to > stretch your thumb to a corne

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

2016-01-06 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126621/ --- (Updated Jan. 6, 2016, 2:13 nachm.) Review request for Plasma, KDE Usabil

Re: some feedback from a mobile application development team

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 11:58:30AM +0100, Marco Martin wrote: > > I will talk to Sebastian about synching our code base with upstream again > > and hope that I can help contribute to the upstream project. > > One thing I would love to see, is the components downloaded and built at > setup > time

[Powerdevil] [Bug 357621] New: Screen cannot be turned on after it has been set to "energy saving" (goes off)

2016-01-06 Thread Simone Gaiarin via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=357621 Bug ID: 357621 Summary: Screen cannot be turned on after it has been set to "energy saving" (goes off) Product: Powerdevil Version: 5.5.3 Platform: Other OS: Linux

Re: some feedback from a mobile application development team

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 11:29:16AM +0100, Marco Martin wrote: > > > > Therefore what I'd like to see is that when our components are used > > > within Android, they draw the buttons in the top bar (where they are - > > > and that is a research-supported fact - hard to reach, but consistent > > > w

Re: some feedback from a mobile application development team

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 11:14:18AM +0100, Marco Martin wrote: > On Wednesday 06 January 2016, Thomas Pfeiffer wrote: > > Therefore what I'd like to see is that when our components are used within > > Android, they draw the buttons in the top bar (where they are - and that is > > a research-supporte

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Dirk Hohndel wrote: > Building for Android is painful (as Sebastian will confirm). Adding > something where an application after being started downloads other > components... not going to happen. At least not for Subsurface. I want to > be able to offer the user an .ap

[Powerdevil] [Bug 357621] Screen cannot be turned on after it has been set to "energy saving" (goes off)

2016-01-06 Thread Kai Uwe Broulik via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=357621 Kai Uwe Broulik changed: What|Removed |Added CC||k...@privat.broulik.de --- Comment #1 from Ka

Re: Review Request 126637: IconItem: Add animated property

2016-01-06 Thread Eike Hein
> On Jan. 6, 2016, 12:14 p.m., Kai Uwe Broulik wrote: > > I'm not arguing against the addition of this property, my patch should > > significantly reduce the need for it and prevent abusing it, however. > > > > Most cases where you don't want to fade the pixmap is when the view it's in > > was

Re: Review Request 126637: IconItem: Add animated property

2016-01-06 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126637/#review90700 --- Ship it! ok, so let's go for it, both this and kai's one - M

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Dirk Hohndel wrote: > This depends where you are planning to focus on when it comes to > relevance. If you believe that plasma-mobile will be the next Android and > that the next billion devices will run plasma-mobile, then I see your > point. If you want Qt/QML/plasma

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Dirk Hohndel wrote: > > Well, before we removed the FAB I was quite easily able to get into > situations where the FAB would disappear and would not show back up, > regardless what I tried. The top bar doesn't disappear, so those buttons > are always there. Yet one mo

Re: Review Request 126637: IconItem: Add animated property

2016-01-06 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126637/#review90701 --- Ship it! Ship It! - Kai Uwe Broulik On Jan. 6, 2016, 11 vo

Review Request 126654: rework ColorScope, hopefully fix black on black battery on phone

2016-01-06 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126654/ --- Review request for Plasma and Bhushan Shah. Repository: plasma-framework

Re: some feedback from a mobile application development team

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 04:39:33PM +0100, Marco Martin wrote: > On Wednesday 06 January 2016, Dirk Hohndel wrote: > > Building for Android is painful (as Sebastian will confirm). Adding > > something where an application after being started downloads other > > components... not going to happen. At

Re: some feedback from a mobile application development team

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 05:41:27PM +0100, Marco Martin wrote: > On Wednesday 06 January 2016, Dirk Hohndel wrote: > > > > Well, before we removed the FAB I was quite easily able to get into > > situations where the FAB would disappear and would not show back up, > > regardless what I tried. The to

Re: some feedback from a mobile application development team

2016-01-06 Thread Kai Uwe Broulik
Hi, The issue with the virtual keyboard shifting the contents on Android is an issue I also face with pure Qt apps and as of now there is no way to prevent or detect this behavior, at least not with Qt APIs. I think Qt 5.6 will provide means to detect the keyboard geometry in QInputMethod with

Re: some feedback from a mobile application development team

2016-01-06 Thread Dirk Hohndel
> On Jan 6, 2016, at 10:55, Kai Uwe Broulik wrote: > > Hi, > > The issue with the virtual keyboard shifting the contents on Android is an > issue I also face with pure Qt apps and as of now there is no way to prevent > or detect this behavior, at least not with Qt APIs. > > I think Qt 5.6 w

Re: some feedback from a mobile application development team

2016-01-06 Thread Kai Uwe Broulik
Have a look at Qt.inputMethod.visible and Qt.inputMethod.keyboardRectangle - with that you should be able to transform the FAB so it stays visible on screen. I'm not sure if you can influence the behavior that Android shifts the view upwards if you tab an input field that would otherwise be cove

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Kai Uwe Broulik wrote: > Hi, > > The issue with the virtual keyboard shifting the contents on Android is an > issue I also face with pure Qt apps and as of now there is no way to > prevent or detect this behavior, at least not with Qt APIs. > > I think Qt 5.6 will pr

Re: some feedback from a mobile application development team

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Marco Martin wrote: > On Wednesday 06 January 2016, Kai Uwe Broulik wrote: > > Hi, > > > > The issue with the virtual keyboard shifting the contents on Android is > > an issue I also face with pure Qt apps and as of now there is no way to > > prevent or detect this be

AW: some feedback from a mobile application development team

2016-01-06 Thread Kai Uwe Broulik
> ‎qinputmethod does have keyboard geometry even in 5.5, on linux it works  fine... so that doesn't work on android? ‎Yes, Google in their infinite wisdom believe that an app should never ever need to probe for the keyboard geometry, only in 5.6 (or even 5.7, can't find the codereview right now)

Re: some feedback from a mobile application development team

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 08:03:16PM +0100, Marco Martin wrote: > On Wednesday 06 January 2016, Marco Martin wrote: > > On Wednesday 06 January 2016, Kai Uwe Broulik wrote: > > > Hi, > > > > > > The issue with the virtual keyboard shifting the contents on Android is > > > an issue I also face with p

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

2016-01-06 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126635/ --- (Updated Jan. 6, 2016, 7:15 p.m.) Status -- This change has been mar

modifying the drawer menus

2016-01-06 Thread Dirk Hohndel
I'm sure this email will mostly just expose my ignorance - but I'm OK with that :-) -- I asked on IRC but I think no one was around who knew. I want to be able to change the content of the context menu, depending on, err, context. Seems like a reasonable thing to want to do. So I'm sure there

Re: Review Request 126637: IconItem: Add animated property

2016-01-06 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126637/ --- (Updated Jan. 6, 2016, 7:38 p.m.) Status -- This change has been mar

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

2016-01-06 Thread Alessandro Longo
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126621/#review90707 --- Compliments! Your work is great as always. From visual design

Plasma 5.5.3 is out

2016-01-06 Thread Jonathan Riddell
Plasma 5.5.3 is now released https://www.kde.org/announcements/plasma-5.5.3.php ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: modifying the drawer menus

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Dirk Hohndel wrote: > I'm sure this email will mostly just expose my ignorance - but I'm OK with > that :-) -- I asked on IRC but I think no one was around who knew. > > I want to be able to change the content of the context menu, depending on, > err, context. See

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

2016-01-06 Thread Kai Uwe Broulik
> On Jan. 6, 2016, 7:57 nachm., Alessandro Longo wrote: > > Compliments! Your work is great as always. From visual design POV I think > > how it currently looks it's very fine; for the implementation: > > if I understand well the circle and the progress indicators will be two > > SVGs to be put

Re: modifying the drawer menus

2016-01-06 Thread Tomaz Canabrava
On Wed, Jan 6, 2016 at 6:29 PM, Marco Martin wrote: > On Wednesday 06 January 2016, Dirk Hohndel wrote: > > I'm sure this email will mostly just expose my ignorance - but I'm OK > with > > that :-) -- I asked on IRC but I think no one was around who knew. > > > > I want to be able to change t

Re: modifying the drawer menus

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Tomaz Canabrava wrote: > We need a menu that will change its contents based on whats being displayed > on the main part of the application. > > Something like: > > contextDrawer: MobileComponents.ContextDrawer { > id: contextDrawer > actions: mainDisplay.

Re: modifying the drawer menus

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Tomaz Canabrava wrote: > We need a menu that will change its contents based on whats being displayed > on the main part of the application. > > Something like: > > contextDrawer: MobileComponents.ContextDrawer { > id: contextDrawer > actions: mainDisplay.

Re: modifying the drawer menus

2016-01-06 Thread Tomaz Canabrava
On Wed, Jan 6, 2016 at 6:39 PM, Marco Martin wrote: > On Wednesday 06 January 2016, Tomaz Canabrava wrote: > > We need a menu that will change its contents based on whats being > displayed > > on the main part of the application. > > > > Something like: > > > > contextDrawer: MobileComponents.Con

Re: modifying the drawer menus

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Tomaz Canabrava wrote: > > > > hmm, not sure it answers much yet.. who is mainDisplay? > > Its one of the Pages in an AplicationWindow, like the initialPage. > the "current" one should give the Actions for the contextDrawer. so, if it depends from pages, you just de

Re: some feedback from a mobile application development team

2016-01-06 Thread Thomas Pfeiffer
On Dienstag, 5. Januar 2016 18:15:38 CET Dirk Hohndel wrote: > Wow. I did not realize that. Just tried it and indeed, you are right. > No conflict, no issue. Cool. Great! :-) > I learned something new today :-) Yeah, it seems Android could use some tutorials, too ;) > > > What do you think abo

Re: modifying the drawer menus

2016-01-06 Thread Dirk Hohndel
> On Jan 6, 2016, at 12:47, Marco Martin wrote: > > On Wednesday 06 January 2016, Tomaz Canabrava wrote: >>> >>> hmm, not sure it answers much yet.. who is mainDisplay? >> >> Its one of the Pages in an AplicationWindow, like the initialPage. >> the "current" one should give the Actions for th

Re: Draft split for qpt plugin from frameworkintegration

2016-01-06 Thread Albert Astals Cid
El Friday 18 December 2015, a les 10:03:39, Martin Graesslin va escriure: > On Thursday, December 17, 2015 5:48:47 PM CET Martin Graesslin wrote: > > On Thursday, December 17, 2015 4:32:48 PM CET Aleix Pol wrote: > > > On Wed, Dec 16, 2015 at 4:12 PM, Martin Graesslin > > > > wrote: > > > > Hi al

Re: modifying the drawer menus

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Dirk Hohndel wrote: > > On Jan 6, 2016, at 12:47, Marco Martin wrote: > > > > On Wednesday 06 January 2016, Tomaz Canabrava wrote: > >>> hmm, not sure it answers much yet.. who is mainDisplay? > >> > >> Its one of the Pages in an AplicationWindow, like the initialPa

Re: Review Request 126630: Do not unconditionally enable logging

2016-01-06 Thread Peter Wu
> On Jan. 5, 2016, 1:10 p.m., Sebastian Kügler wrote: > > Please also disable it in the same way for the kwayland backend. Already done, is there anything I should change before it can get merged? - Peter --- This is an automatically ge

Re: modifying the drawer menus

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 10:32:59PM +0100, Marco Martin wrote: > > > so, if it depends from pages, you just define a different > > > contextualactions list for every page definition... > > > > Currently we have one page that has two different states. You view a dive > > or you edit a dive. Dependin

Re: modifying the drawer menus

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 01:49:28PM -0800, Dirk Hohndel wrote: > On Wed, Jan 06, 2016 at 10:32:59PM +0100, Marco Martin wrote: > > > > so, if it depends from pages, you just define a different > > > > contextualactions list for every page definition... > > > > > > Currently we have one page that ha

Re: Review Request 126630: Do not unconditionally enable logging

2016-01-06 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126630/#review90715 --- Ship it! Ship It! - Daniel Vrátil On Jan. 5, 2016, 1:08 p.

Re: modifying the drawer menus

2016-01-06 Thread Kai Uwe Broulik
Don't we handle the visible and enabled property of Action? That should work for enabling / disabling certain drawer entries if in edit mode.‎ Irregardless of that usecase I would really expect that to work. ‎ ___ Plasma-devel mailing list Plasma-devel

Re: modifying the drawer menus

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Kai Uwe Broulik wrote: > Don't we handle the visible and enabled property of Action? That should > work for enabling / disabling certain drawer entries if in edit mode.‎ > Irregardless of that usecase I would really expect that to work. i just pushed the support of en

Re: Review Request 126630: Do not unconditionally enable logging

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

Re: modifying the drawer menus

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 11:31:58PM +0100, Kai Uwe Broulik wrote: > Don't we handle the visible and enabled property of Action? That should work > for enabling / disabling certain drawer entries if in edit mode.‎ > Irregardless of that usecase I would really expect that to work. There is no "visi

Re: modifying the drawer menus

2016-01-06 Thread Kai Uwe Broulik
‎> i just pushed the support of enabled. Awesome. However, considering the future of QtQuick Controls 1 and the fact that QQC 2 don't have an "Action" we should probably provide our own simple QObject for this. Once we sort out the deployment, that is. There will most likely be additional prope

Re: modifying the drawer menus

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Kai Uwe Broulik wrote: > ‎> i just pushed the support of enabled. > > Awesome. However, considering the future of QtQuick Controls 1 and the fact > that QQC 2 don't have an "Action" we should probably provide our own > simple QObject for this. Once we sort out the dep

Re: modifying the drawer menus

2016-01-06 Thread Kai Uwe Broulik
Let's move QAction to QtGui \o/‎ :(  > it belongs in Qt, even if they don't want it.. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: modifying the drawer menus

2016-01-06 Thread Marco Martin
On Wednesday 06 January 2016, Dirk Hohndel wrote: > If I write this as > > property list viewActions: [ > Action { }, Action { } > ] > property list editActions: [ > Action { }, Action { } > ] > > it doesn't fail on syntax, but doesn't work. It always shows the > viewActio

Re: modifying the drawer menus

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 11:57:24PM +0100, Marco Martin wrote: > On Wednesday 06 January 2016, Dirk Hohndel wrote: > > If I write this as > > > > property list viewActions: [ > > Action { }, Action { } > > ] > > property list editActions: [ > > Action { }, Action { } > > ] > > > >

Re: modifying the drawer menus

2016-01-06 Thread Dirk Hohndel
On Wed, Jan 06, 2016 at 03:08:02PM -0800, Dirk Hohndel wrote: > On Wed, Jan 06, 2016 at 11:57:24PM +0100, Marco Martin wrote: > > On Wednesday 06 January 2016, Dirk Hohndel wrote: > > > If I write this as > > > > > > property list viewActions: [ > > > Action { }, Action { } > > > ] > > > prope

Re: Moving kimpanel to plasma-desktop

2016-01-06 Thread Eike Hein
The code has now been imported with full history (going back to SVN even, further than the old git repo) into plasma-desktop, currently as a toplevel kimpanel/ directory not yet added to CMake. It's getting late, so I'll take care of moving it into its final position and making it build tomorrow.

Review Request 126658: Notifications again not works correctly

2016-01-06 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126658/ --- Review request for Plasma, David Edmundson, Marco Martin, and Martin Klapet

Re: Review Request 126658: Notifications again not works correctly

2016-01-06 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126658/#review90725 --- No. I'm fine with disabling the first animation. Post a separ

Re: Review Request 126658: Notifications again not works correctly

2016-01-06 Thread Anthony Fieroni
> On Ян. 7, 2016, 5:36 преди обяд, Martin Klapetek wrote: > > No. > > > > I'm fine with disabling the first animation. Post a separate patch > > for that and that only. > > > > Any configuration issues that are left need to be identified > > and fixed, not by reverting and moving things back ag

Re: Review Request 126658: Notifications again not works correctly [revert parch 126408]

2016-01-06 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126658/ --- (Updated Ян. 7, 2016, 5:41 преди обяд) Review request for Plasma, David E

Re: Review Request 126658: Notifications again not works correctly [revert parch 126408]

2016-01-06 Thread Martin Klapetek
> On Jan. 7, 2016, 4:36 a.m., Martin Klapetek wrote: > > No. > > > > I'm fine with disabling the first animation. Post a separate patch > > for that and that only. > > > > Any configuration issues that are left need to be identified > > and fixed, not by reverting and moving things back again.

Re: Review Request 126658: Notifications again not works correctly [revert parch 126408]

2016-01-06 Thread Anthony Fieroni
> On Ян. 7, 2016, 5:36 преди обяд, Martin Klapetek wrote: > > No. > > > > I'm fine with disabling the first animation. Post a separate patch > > for that and that only. > > > > Any configuration issues that are left need to be identified > > and fixed, not by reverting and moving things back ag

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

2016-01-06 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/35/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 07 Jan 2016 04:00:51 + Build duration: 3 min 20 sec CHANGE SET Revision 32ace708bdaa5bf0494

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

2016-01-06 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/36/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 07 Jan 2016 05:47:38 + Build duration: 1 min 53 sec CHANGE SET Revision 03cf4b2ba8810323e63

Re: Review Request 126658: Notifications again not works correctly [revert parch 126408]

2016-01-06 Thread Martin Klapetek
> On Jan. 7, 2016, 4:36 a.m., Martin Klapetek wrote: > > No. > > > > I'm fine with disabling the first animation. Post a separate patch > > for that and that only. > > > > Any configuration issues that are left need to be identified > > and fixed, not by reverting and moving things back again.

two more mobile-components oddities

2016-01-06 Thread Dirk Hohndel
So if I have one of the drawers open and then tap on the back button on the device, I think that should close that drawer instead of going back one level in the app and leaving the drawer open... the NEXT tap on the back button should now move back one level. Next, imagine a screen where you can i