Re: Plasma Mobile on the Librem 5 (please ignore previous two mails)

2017-09-02 Thread Thomas Pfeiffer
Hi Zlatan, please ignore my previous two mails mail to you which were both sent from the wrong address (sorry for that, new computer with no proper email client yet and GMail is terrible when it comes to multiple email addresses). On Sat, Sep 2, 2017 at 5:23 PM, Zlatan Todoric wrote: > (Cleared a

D3805: Per-activity favorites (Final, again?)

2017-09-02 Thread Ivan Čukić
ivan added a comment. The last patch supports Kickoff, Kicker and Dash. I've tested what I can, found an small issue in KAMD (update KAMD to latest master before testing) and it looks like we are on the finish line. REVISION DETAIL https://phabricator.kde.org/D3805 To: ivan, mart, hein Cc

D3805: Per-activity favorites (Final, again?)

2017-09-02 Thread Ivan Čukić
ivan updated this revision to Diff 19108. ivan edited the summary of this revision. CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3805?vs=19034&id=19108 REVISION DETAIL https://phabricator.kde.org/D3805 AFFECTED FILES applets/kicker/CMakeLists.txt applets/kicker/package/contents

D3805: Per-activity favorites (Final, again?)

2017-09-02 Thread Ivan Čukić
ivan edited the summary of this revision. ivan edited the test plan for this revision. REVISION DETAIL https://phabricator.kde.org/D3805 To: ivan, mart, hein Cc: Zren, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D7663: Correct text style

2017-09-02 Thread patrick j pereira
patrickelectric added a reviewer: Kirigami. REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D7663 To: patrickelectric, tcanabrava, mart, #kirigami Cc: plasma-devel, apol, mart, hein

D7663: Correct text style

2017-09-02 Thread patrick j pereira
patrickelectric added reviewers: tcanabrava, mart. REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D7663 To: patrickelectric, tcanabrava, mart Cc: plasma-devel, apol, mart, hein

D7663: Correct text style

2017-09-02 Thread patrick j pereira
patrickelectric created this revision. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Signed-off-by: Patrick José Pereira Update deploy information Signed-off-by: Patrick José Pereira Update desktop bui

D7640: QtCurve: reduce progressbar timer overhead

2017-09-02 Thread René J . V . Bertin
rjvbb added a comment. Implementing on-demand/JIT starting of the timer also allowed to kill the timer when all progressbars stopped animating, in the end. No more need for the idling frequency trick. REPOSITORY R626 QtCurve REVISION DETAIL https://phabricator.kde.org/D7640 To: rjvbb,

D7640: QtCurve: reduce progressbar timer overhead

2017-09-02 Thread René J . V . Bertin
This revision was automatically updated to reflect the committed changes. Closed by commit R626:d9d2bb0dcb01: reduce progressbar animation timer overhead (authored by rjvbb). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D7640?vs=19073&id=19096#toc REPOSITORY R626 QtCurve CHANGES SINC

Re: Plasma Mobile on the Librem 5

2017-09-02 Thread Zlatan Todoric
(Cleared all previous mails to make this one more visible, feel free to respond on this one while quoting previous replies in this thread) Hi all, so I had discussion with our CEO and he is of course absolutely in. To make clear our standpoint - both GNOME and KDE *will* be first class citizen on

Re: start menu icon

2017-09-02 Thread Franklin Weng
Hi Marco & Eike, Marco Martin 於 2017年08月31日 03:51 寫道: > On mercoledì 30 agosto 2017 15:07:26 CEST Franklin Weng wrote: >> *looking at Eike with shining eyes* >> Could you please tell me in more detail about "ship an applet config >> initialization script in look-n-feel package"? In the package >

D7640: QtCurve: reduce progressbar timer overhead

2017-09-02 Thread René J . V . Bertin
rjvbb added a comment. Never too old to learn, thanks :) REPOSITORY R626 QtCurve REVISION DETAIL https://phabricator.kde.org/D7640 To: rjvbb, yuyichao, #plasma Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D7640: QtCurve: reduce progressbar timer overhead

2017-09-02 Thread Yichao Yu
yuyichao added a comment. > But I don't see how could do either for startTimer()? `const_cast(this)->startTimer()` REPOSITORY R626 QtCurve REVISION DETAIL https://phabricator.kde.org/D7640 To: rjvbb, yuyichao, #plasma Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai,

D7654: Move XRandR event filter into XRandRScreens

2017-09-02 Thread Martin Flöser
graesslin created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY The code in events.cpp was problematic as it was called in a Wayland session. So KWin changed outputs, this gets mirrored to XWayland an

D7653: Do not hard runtime depend on X11 in RuleBook

2017-09-02 Thread Martin Flöser
graesslin created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY The RuleBook is created during Workspace startup, so it's a required component for the overall KWin session. It uses a KXMessages object

D7651: Protect readProperty and deleteProperty in case of no X11

2017-09-02 Thread Martin Flöser
graesslin created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY Read(Window)Property and deleteProperty should only operate if there is an X11 connection available. In addition the methods are ch

D7640: QtCurve: reduce progressbar timer overhead

2017-09-02 Thread René J . V . Bertin
rjvbb added a comment. > This should not be necessary. You can add `mutable` or do const cast for certain members. Right. Never used that myself it wasn't my 1st reflex. But I don't see how could do either for startTimer()? REPOSITORY R626 QtCurve REVISION DETAIL https://phabrica

D7650: Drop XRandR dependency from Options's currentRefreshRate

2017-09-02 Thread Martin Flöser
graesslin created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY This code branch was dead. It was for the case that the number of screens is 0, but KWin::Screens never has a count of zero, especially

D3805: Per-activity favorites (Final, again?)

2017-09-02 Thread Eike Hein
hein added a comment. The Kickoff thing looks more like a model bug to me - there's a duplicate delegate for "Firefox", and one of them has a broken row index (-1). Stuff like that is usually the result of confusing the view with broken model transactions ... the QML is very simple (that bin