D5950: Implement the new panel icon size ceiling also for Activities.

2017-05-30 Thread Eike Hein
hein closed this revision. hein added a comment. This was committed as https://phabricator.kde.org/R119:a8d9038a592e6d3ed87852bc2fb3a0f93449897d, not sure why the ticket didn't get closed. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D5950 To: hein, #plasm

D3650: lower mouse acceleration limit to 0.0

2017-05-30 Thread Jens Reuterberg
jensreuterberg added a comment. Ping? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3650 To: sitter, plasma-devel Cc: graesslin, broulik, jensreuterberg, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, abetts, sebas, apol, mart, lukas

D5974: Wizard: Use emblem-success and emblem-error icons

2017-05-30 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R97 Bluedevil BRANCH master REVISION DETAIL https://phabricator.kde.org/D5974 To: drosca, #plasma, davidedmundson Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jen

D1231: Add Remote Access interface to KWayland

2017-05-30 Thread David Edmundson
davidedmundson added a comment. This revision now requires changes to proceed. > Also caused by QtWayland not exposing the wl_output in the native interface. This might have been true at the time of writing,. It's not the case now. nativeResourceForScreen will return a wl_output, we

D1231: Add Remote Access interface to KWayland

2017-05-30 Thread David Edmundson
davidedmundson added a comment. Seaprate question, In wl_surface when we attach a new buffer we also mark what areas are damaged. Here we're passing an even bigger buffer. Would it benefit from a series of damage events being sent in the org_kde_kwin_remote_buffer before the gbm_h

Re: Plasma 5.10 video NOT FINAL

2017-05-30 Thread Łukasz Sawicki
Hi, > I guess it's two different videos pieces together in which those two icons > were manually moved between the two. That indeed was the case, fixed now. https://youtu.be/VtdTC2Mh070 I also added subtitles Cheers, Łukasz

D5768: Use the first thumbnail as icon for KNS resources

2017-05-30 Thread Marco Martin
mart added a comment. In https://phabricator.kde.org/D5768#112114, @apol wrote: > Or we apply it for master and hope Kirigami 2.2 is released before next Plasma release. we can make it happen soon. i just wanted to know if scroll wheel behavior is alright now, then for what i'm

D1230: GBM remote access support for KWin

2017-05-30 Thread David Edmundson
davidedmundson requested changes to this revision. davidedmundson added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > drm_buffer.cpp:131 > #if HAVE_GBM > +if (m_deferred) { > +return; If it's deferred it means someone else is doing the gbm_surface

D1230: GBM remote access support for KWin

2017-05-30 Thread Oleg Chernovskiy
Kanedias added inline comments. INLINE COMMENTS > davidedmundson wrote in drm_buffer.cpp:131 > If it's deferred it means someone else is doing the gbm_surface_release. > > But we still need to set m_bo to nullptr. Otherwise it's potentially left > dangling here after the RemoteAccessManager has

D6020: add session and power mgmt runners to kickoff

2017-05-30 Thread Harald Sitter
sitter created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY It's a bit weird that kickoff's leave tab has power management actions but you cannot actually invoke them via a search. CHANGELOG: Appli

D6020: add session and power mgmt runners to kickoff

2017-05-30 Thread Eike Hein
hein added a comment. Please also add to kicker/dashboard if missing. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D6020 To: sitter, hein, broulik Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, m

D6020: add session and power mgmt runners to kickoff

2017-05-30 Thread Harald Sitter
sitter added a comment. already in there https://phabricator.kde.org/source/plasma-desktop/browse/master/applets/kicker/package/contents/ui/main.qml;7a4fdbbd6124757ccd9d07fabd8e82ac62bc8995$156 REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D6020 To: sitter,

D6020: add session and power mgmt runners to kickoff

2017-05-30 Thread Eike Hein
hein accepted this revision. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop BRANCH Plasma/5.10 REVISION DETAIL https://phabricator.kde.org/D6020 To: sitter, hein, broulik Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterbe

D6020: add session and power mgmt runners to kickoff

2017-05-30 Thread Harald Sitter
This revision was automatically updated to reflect the committed changes. Closed by commit R119:d73760978877: add session and power mgmt runners to kickoff (authored by sitter). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6020?vs=14966&id=14968 REVI

D5974: Wizard: Use emblem-success and emblem-error icons

2017-05-30 Thread David Rosca
This revision was automatically updated to reflect the committed changes. Closed by commit R97:f3b2736856c0: Wizard: Use emblem-success and emblem-error icons (authored by drosca). REPOSITORY R97 Bluedevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5974?vs=14857&id=14970 REVISIO

D6024: Fix detection of sys/event.h on FreeBSD < 12

2017-05-30 Thread Tobias C. Berner
tcberner created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Unfortunately I only tested the kevent stuff on FreeBSD 12. FreeBSD < 12 has an older clang, which seems to add less headers automatically.

Plasma 5.10.0

2017-05-30 Thread Jonathan Riddell
Plasma 5.10.0 is now released https://www.kde.org/announcements/plasma-5.10.0.php

Re: Plasma 5.10.0

2017-05-30 Thread Harald Sitter
On Tue, May 30, 2017 at 2:13 PM, Jonathan Riddell wrote: > Plasma 5.10.0 is now released > https://www.kde.org/announcements/plasma-5.10.0.php It's a beauty! Thanks to everyone ❤

D6024: Fix detection of sys/event.h on FreeBSD < 12

2017-05-30 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R133 KScreenLocker BRANCH Plasma/5.10 REVISION DETAIL https://phabricator.kde.org/D6024 To: tcberner, #freebsd, davidedmundson Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-

D6024: Fix detection of sys/event.h on FreeBSD < 12

2017-05-30 Thread Tobias C. Berner
This revision was automatically updated to reflect the committed changes. Closed by commit R133:51210aa52144: Fix detection of sys/event.h on FreeBSD < 12 (authored by tcberner). REPOSITORY R133 KScreenLocker CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6024?vs=14974&id=14977 REVI

Re: Next Plasma LTS and openSUSE

2017-05-30 Thread Sebastian Kügler
On maandag 29 mei 2017 12:56:49 CEST David Edmundson wrote: > Stealing this LTS thread, there's something from Qt I'd like to copy: > >Currently, Qt 5.6 LTS is in the ‘strict’ phase where it only receives > >selected critical fixes and next it is moving into the ‘very strict’ > >phase. During this

D3650: lower mouse acceleration limit to 0.0

2017-05-30 Thread Aleix Pol Gonzalez
apol added a comment. In https://phabricator.kde.org/D3650#80584, @graesslin wrote: > According to my comments in the Wayland port: the value 1.0 means no acceleration. Values below 1.0 mean a de-acceleration or a slow down. But it's still a factor. I'm also afraid that putting a 0.0 the

D5768: Use the first thumbnail as icon for KNS resources

2017-05-30 Thread Aleix Pol Gonzalez
apol added a comment. In https://phabricator.kde.org/D5768#112774, @mart wrote: > In https://phabricator.kde.org/D5768#112114, @apol wrote: > > > Or we apply it for master and hope Kirigami 2.2 is released before next Plasma release. > > > we can make it happen soon. > i just

D6026: Add setScale option to kscreendoctor

2017-05-30 Thread David Edmundson
davidedmundson created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Adds set scale option to kscreen doctor. Can I put this in 5.10 branch too please. TEST PLAN Ran it, my screen changed awesomely.

D6027: Use existing parseInt function

2017-05-30 Thread David Edmundson
davidedmundson created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY There's already a QString::toInt() that validates input. We don't need to make our own. Also there was a bug in the current point

D6028: Use own fork of QQuickStyleItem

2017-05-30 Thread Marco Martin
mart created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY use a local copy of qqc1 qquickstyleitem, remove all qqc1 usage TEST PLAN all of kirigami gallery works with the new style REPOSITORY R858 Q

D5768: Use the first thumbnail as icon for KNS resources

2017-05-30 Thread Aleix Pol Gonzalez
apol added a comment. @leinir remember to close `BUG: 360675` with this one. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D5768 To: leinir, apol Cc: mart, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, s

D6028: Use own fork of QQuickStyleItem

2017-05-30 Thread David Edmundson
davidedmundson accepted this revision. davidedmundson added a comment. This revision is now accepted and ready to land. looks good. (well except for how horrible that borrowed Qt class is) We need to fix the OS X code path. (even if we just comment out the current stuff) INLINE COMMENTS

D6028: Use own fork of QQuickStyleItem

2017-05-30 Thread Marco Martin
This revision was automatically updated to reflect the committed changes. Closed by commit R858:9623a35f63d2: Use own fork of QQuickStyleItem (authored by mart). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D6028?vs=14984&id=14988#toc REPOSITORY R858 Qt Quick Controls 2: Desktop Style

[kio-extras] [Bug 380302] smb:// KIO still tries SMBv1 first which is blocked by modern servers

2017-05-30 Thread Rex Dieter
https://bugs.kde.org/show_bug.cgi?id=380302 Rex Dieter changed: What|Removed |Added CC||rdie...@gmail.com --- Comment #1 from Rex Dieter

[kio-extras] [Bug 380302] smb:// KIO still tries SMBv1 first which is blocked by modern servers

2017-05-30 Thread Rex Dieter
https://bugs.kde.org/show_bug.cgi?id=380302 --- Comment #2 from Rex Dieter --- Or rather, do other things work as expected, like 'smbclient' ? -- You are receiving this mail because: You are the assignee for the bug.

D6035: Use right QProcess API

2017-05-30 Thread David Edmundson
davidedmundson created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY KAccess is responsible for: visual system bell stick keys/slow keys starting orca None of this will work in wayland as-is;

D5994: Force kaccess to run under X

2017-05-30 Thread David Edmundson
davidedmundson abandoned this revision. davidedmundson added a comment. Fair enough. My rationale was that we might end up running two kaccesses in different modes in future, but at that point I'll just make two executables. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabrica

D6035: Force KAccess to run under XCB

2017-05-30 Thread David Edmundson
davidedmundson retitled this revision from "Use right QProcess API" to "Force KAccess to run under XCB". REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D6035 To: davidedmundson, #plasma Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jens

[plasmashell] [Bug 380374] New: WindowList should group by Activity

2017-05-30 Thread Don Nguyen
https://bugs.kde.org/show_bug.cgi?id=380374 Bug ID: 380374 Summary: WindowList should group by Activity Product: plasmashell Version: master Platform: Other OS: Linux Status: UNCONFIRMED Severity: wishlist

[plasmashell] [Bug 380374] WindowList should group by Activity

2017-05-30 Thread Christoph Feck
https://bugs.kde.org/show_bug.cgi?id=380374 Christoph Feck changed: What|Removed |Added Assignee|plasma-devel@kde.org|plasma-b...@kde.org -- You are receiving this

D6041: Port a bunch of classes away from Plasma::Package

2017-05-30 Thread Aleix Pol Gonzalez
apol created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Ported away some classes, removes many unneeded casts that happen when starting. TEST PLAN Tests still pass, plasma starts properly