Re: Kirigami Page.title

2016-04-20 Thread Dirk Hohndel
> On Apr 20, 2016, at 7:51 PM, Martin Klapetek > wrote: > > Is it possible to either hide the blue titlebar when empty > or add things like buttons and icons in there? > > In SpaceBar, the active conversation has this layout at > the top of the Page: > > [Presence

Re: Review Request 127586: [calendar] Add a mark to days containing events

2016-04-20 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127586/ --- (Updated April 21, 2016, 3:41 a.m.) Status -- This change has been

Kirigami Page.title

2016-04-20 Thread Martin Klapetek
Is it possible to either hide the blue titlebar when empty or add things like buttons and icons in there? In SpaceBar, the active conversation has this layout at the top of the Page: [Presence Icon] Username [Close Conversation Button] So in this case it doesn't make sense that this Page would

[Breeze] [Bug 361885] Kalarm crashes when closing main woindow

2016-04-20 Thread Aleix Pol via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361885 Aleix Pol changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Breeze] [Bug 361885] Kalarm crashes when closing main woindow

2016-04-20 Thread David Jarvie via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361885 David Jarvie changed: What|Removed |Added Product|kalarm |Breeze

[Differential] [Commented On] D1452: Implement volume overdrive

2016-04-20 Thread drosca (David Rosca)
drosca added a comment. In https://phabricator.kde.org/D1452#26974, @colomar wrote: > There is a reason why "100%" is not the actual maximum volume you can get out of a device. > I don't want us to be liable for either burst speakers or burst eardrums... Well, for most devices

[Differential] [Updated, 86 lines] D1452: Implement volume overdrive

2016-04-20 Thread drosca (David Rosca)
drosca updated this revision to Diff 3438. drosca added a comment. Fix showing volume % in applet REPOSITORY rPLASMAPA Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1452?vs=3434=3438 BRANCH volume-overdrive REVISION DETAIL

Review Request 127702: ConfigModel: Don't try to resolve empty source path from package

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

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 40 - Still Failing!

2016-04-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/40/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Apr 2016 20:08:19 + Build duration: 3 min 42 sec CHANGE SET Revision

[Differential] [Commented On] D1452: Implement volume overdrive

2016-04-20 Thread colomar (Thomas Pfeiffer)
colomar added a comment. I know Pavucontrol does the same, but I think this makes it too easy to go into overdrive. There is a reason why "100%" is not the actual maximum volume you can get out of a device. I don't want us to be liable for either burst speakers or burst eardrums...

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 39 - Still Failing!

2016-04-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/39/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Apr 2016 18:56:53 + Build duration: 3 min 47 sec CHANGE SET Revision

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

2016-04-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.6%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/24/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Apr 2016 18:50:38 + Build duration: 5 min 30 sec CHANGE SET Revision

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

2016-04-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.6%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/24/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Apr 2016 18:50:38 + Build duration: 5 min 30 sec CHANGE SET Revision

Re: another Kirigami issue - overlap of header and listview

2016-04-20 Thread Dirk Hohndel
On Wed, Apr 20, 2016 at 07:32:04PM +0200, Marco Martin wrote: > On Wednesday 20 April 2016 09:44:43 Dirk Hohndel wrote: > > On Wed, Apr 20, 2016 at 10:36:05AM +0200, Marco Martin wrote: > > > On Tuesday 19 April 2016, Dirk Hohndel wrote: > > > > go). The latter always appears to have the top item

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 38 - Failure!

2016-04-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/38/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Apr 2016 17:36:18 + Build duration: 3 min 11 sec CHANGE SET Revision

Re: another Kirigami issue - overlap of header and listview

2016-04-20 Thread Marco Martin
On Wednesday 20 April 2016 09:44:43 Dirk Hohndel wrote: > On Wed, Apr 20, 2016 at 10:36:05AM +0200, Marco Martin wrote: > > On Tuesday 19 April 2016, Dirk Hohndel wrote: > > > go). The latter always appears to have the top item aligned with the > > > header, but when I tap on the header and have

Review Request 127698: Always write the theme name to plasmarc, also if the default theme is chosen

2016-04-20 Thread Wolfgang Bauer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127698/ --- Review request for Plasma. Repository: plasma-framework Description

Re: another Kirigami issue - overlap of header and listview

2016-04-20 Thread Dirk Hohndel
On Wed, Apr 20, 2016 at 10:36:05AM +0200, Marco Martin wrote: > On Tuesday 19 April 2016, Dirk Hohndel wrote: > > go). The latter always appears to have the top item aligned with the > > header, but when I tap on the header and have scrollTopAnimation run to: > > 0, then the header overlaps the

Re: Kirigami: a feature needed for subsurface and api change feedback

2016-04-20 Thread Dirk Hohndel
On Wed, Apr 20, 2016 at 05:52:48PM +0200, Marco Martin wrote: > On Wednesday 20 April 2016, Sebastian Kügler wrote: > > On Tuesday 19 April 2016 16:31:41 Kai Uwe Broulik wrote: > > > > 2.) On your Nexus 7, the whole thing indeed is way too small. This > > > > seems to be a DPI scaling issue,

Re: Review Request 127696: add Positive, Neutral, Negative text colors

2016-04-20 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127696/ --- (Updated April 20, 2016, 4:27 p.m.) Status -- This change has been

Re: Kirigami: a feature needed for subsurface and api change feedback

2016-04-20 Thread Marco Martin
On Wednesday 20 April 2016, Sebastian Kügler wrote: > On Tuesday 19 April 2016 16:31:41 Kai Uwe Broulik wrote: > > > 2.) On your Nexus 7, the whole thing indeed is way too small. This > > > seems to be a DPI scaling issue, though > > > > I've seen QScreen physical dots per inch reporting rubbish

Re: Review Request 127696: add Positive, Neutral, Negative text colors

2016-04-20 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127696/#review94733 --- Fix it, then Ship it! Some minor stuff inline. If the

Re: Kirigami: a feature needed for subsurface and api change feedback

2016-04-20 Thread Sebastian Kügler
On Tuesday 19 April 2016 16:31:41 Kai Uwe Broulik wrote: > > 2.) On your Nexus 7, the whole thing indeed is way too small. This seems > > to be a DPI scaling issue, though > I've seen QScreen physical dots per inch reporting rubbish on some Android > devices, I think Nexus 7 was even one of them.

Re: Kirigami: a feature needed for subsurface and api change feedback

2016-04-20 Thread Sebastian Kügler
On Tuesday 19 April 2016 05:03:01 Dirk Hohndel wrote: > > Since it's not something that Kirigami was made for, I fear that at some > > point, it will just break, and it would neither be good if Kirigami had > > to > > adapt to Subsurface-mobile's specialties, nor if you had to work > > "against"

[Differential] [Accepted] D1445: Get the logind session to take over through env variable instead of own PID

2016-04-20 Thread bshah (Bhushan Shah)
bshah accepted this revision. bshah added a reviewer: bshah. bshah added a comment. This revision is now accepted and ready to land. Please adjust commit message to remove reference of KWIN_LOGIN_SESSION before landing this. REPOSITORY rKWIN KWin BRANCH logind-session-arg REVISION

[Differential] [Accepted] D1444: Switch virtual terminal through logind instead of having a VirtualTerminal implementation

2016-04-20 Thread bshah (Bhushan Shah)
bshah accepted this revision. bshah added a reviewer: bshah. bshah added a comment. Tested and works on mobile. REPOSITORY rKWIN KWin BRANCH vt-integration-through-logind REVISION DETAIL https://phabricator.kde.org/D1444 EMAIL PREFERENCES

[Differential] [Updated, 95 lines] D1443: Applet: Move Application streams with drag and drop

2016-04-20 Thread drosca (David Rosca)
drosca updated this revision to Diff 3435. drosca added a comment. Fix issues REPOSITORY rPLASMAPA Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1443?vs=3403=3435 BRANCH applet-movestreams REVISION DETAIL https://phabricator.kde.org/D1443

Re: Review Request 127696: add Positive, Neutral, Negative text colors

2016-04-20 Thread Marco Martin
> On April 20, 2016, 1:04 p.m., Andreas Kainz wrote: > > sorry marco but I'm not that good in code reading. what's the difference in > > the second review? added stuff to be able to use those colors from qml plasmoids as well - Marco

Re: Review Request 127696: add Positive, Neutral, Negative text colors

2016-04-20 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127696/#review94730 --- sorry marco but I'm not that good in code reading. what's

Re: Review Request 127696: add Positive, Neutral, Negative text colors

2016-04-20 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127696/ --- (Updated April 20, 2016, 12:49 p.m.) Review request for Plasma and

[Differential] [Commented On] D1452: Implement volume overdrive

2016-04-20 Thread drosca (David Rosca)
drosca added a comment. F112392: overdrive-kcm.png REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D1452 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca,

[Differential] [Updated] D1452: Implement volume overdrive

2016-04-20 Thread drosca (David Rosca)
drosca added a reviewer: Plasma: Design. REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D1452 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, Plasma, Plasma: Design Cc: plasma-devel, sebas

[Differential] [Request, 86 lines] D1452: Implement volume overdrive

2016-04-20 Thread drosca (David Rosca)
drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY It is now possible to set volume up to 150% from KCM. BUG: 351438 TEST PLAN For applet there will be

Re: Review Request 127696: add Positive, Neutral, Negative text colors

2016-04-20 Thread Marco Martin
> On April 20, 2016, 11:46 a.m., Andreas Kainz wrote: > > there is one problem the icon colors don't fit the colors from the color > > scheme > > > > https://techbase.kde.org/Projects/Usability/HIG/Color > > > > icon red != danger red > > icon green != verdant green > > > > plasma blue is

Re: Review Request 127696: add Positive, Neutral, Negative text colors

2016-04-20 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127696/#review94728 --- there is one problem the icon colors don't fit the colors

[Powerdevil] [Bug 361974] Power saving settings for monitor have no effect

2016-04-20 Thread Christoph Feck via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361974 Christoph Feck changed: What|Removed |Added Assignee|plasma-b...@kde.org |plasma-devel@kde.org

Review Request 127696: add Positive, Neutral, Negative text colors

2016-04-20 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127696/ --- Review request for Plasma. Repository: plasma-framework Description

Re: another Kirigami issue - overlap of header and listview

2016-04-20 Thread Marco Martin
On Tuesday 19 April 2016, Dirk Hohndel wrote: > go). The latter always appears to have the top item aligned with the > header, but when I tap on the header and have scrollTopAnimation run to: > 0, then the header overlaps the top element in the ListView. > > A patch like this seems to fix the

[Differential] [Closed] D1449: Hook up virtual_desktop_changeable state.

2016-04-20 Thread hein (Eike Hein)
This revision was automatically updated to reflect the committed changes. Closed by commit rKWINf2e4e1192936: Hook up virtual_desktop_changeable state. (authored by hein). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1449?vs=3428=3433 REVISION DETAIL

[Differential] [Closed] D1448: Add a virtual_desktop_changable state to the window protocol.

2016-04-20 Thread hein (Eike Hein)
This revision was automatically updated to reflect the committed changes. Closed by commit rKWAYLAND021f67f3f470: Add a virtual_desktop_changable state to the window protocol. (authored by hein). REPOSITORY rKWAYLAND KWayland CHANGES SINCE LAST UPDATE

[Differential] [Updated, 240 lines] D1366: Add Event Sounds stream to Applications list

2016-04-20 Thread drosca (David Rosca)
drosca updated this revision to Diff 3431. drosca added a comment. Store device name for event role stream REPOSITORY rPLASMAPA Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1366?vs=3308=3431 BRANCH event-sounds-stream REVISION DETAIL

Re: Plasma 5.7 Test Days prep meeting #2 notes & log

2016-04-20 Thread Luca Beltrame
Il 19 aprile 2016 14:10:56 CEST, Eike Hein ha scritto: >- No comm yet with other distros >- einar77 may be able to interface with openSUSE for us Thanks for poking me - the ISOs were stuck due to a packaging hiccup. I can say that we'll be able to provide test ISOs. I put links

[Differential] [Accepted] D1448: Add a virtual_desktop_changable state to the window protocol.

2016-04-20 Thread Martin Gräßlin
graesslin accepted this revision. This revision is now accepted and ready to land. REPOSITORY rKWAYLAND KWayland BRANCH master REVISION DETAIL https://phabricator.kde.org/D1448 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: hein, graesslin Cc:

[Differential] [Accepted] D1449: Hook up virtual_desktop_changeable state.

2016-04-20 Thread Martin Gräßlin
graesslin accepted this revision. This revision is now accepted and ready to land. REPOSITORY rKWIN KWin BRANCH master REVISION DETAIL https://phabricator.kde.org/D1449 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: hein, graesslin Cc: plasma-devel,