Re: Animate plasma tooltip windows?

2016-09-11 Thread Michail Vourlakos
On 12/09/2016 02:29 πμ, David Edmundson wrote: On Mon, Sep 12, 2016 at 12:21 AM, Michail Vourlakos > wrote: Hello everyone, I am using in one of my plasmoids the following, PlasmaCore.Dialog{ id:

Re: Animate plasma tooltip windows?

2016-09-11 Thread David Edmundson
On Mon, Sep 12, 2016 at 12:21 AM, Michail Vourlakos wrote: > Hello everyone, > > > I am using in one of my plasmoids the following, > > PlasmaCore.Dialog{ > id: windowsPreviewDlg > type: PlasmaCore.Dialog.Tooltip > location: plasmoid.location > >

Animate plasma tooltip windows?

2016-09-11 Thread Michail Vourlakos
Hello everyone, I am using in one of my plasmoids the following, PlasmaCore.Dialog{ id: windowsPreviewDlg type: PlasmaCore.Dialog.Tooltip location: plasmoid.location ... } because I can not use the ToolTipArea, I need hovering very

[Differential] [Closed] D2589: Make the AbstractListItem a Rectangle

2016-09-11 Thread apol (Aleix Pol Gonzalez)
apol closed this revision. apol added a comment. Submitted as of: commit 9261ecc98d10f38c4eade84746eb5a9b71bff4ff Author: Aleix Pol Date: Tue Sep 6 17:33:20 2016 +0200 I'm unsure why it didn't close the request. REPOSITORY rKIRIGAMI Kirigami

Re: Review Request 128892: Open baloo lmdb database read-only beside in baloo_file/baloo_file_extractor + balooctl (for some commands) + unit tests

2016-09-11 Thread Christoph Cullmann
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128892/ --- (Updated Sept. 12, 2016, 12:39 a.m.) Status -- This change has been

Re: Review Request 128892: Open baloo lmdb database read-only beside in baloo_file/baloo_file_extractor + balooctl (for some commands) + unit tests

2016-09-11 Thread Christoph Cullmann
> On Sept. 11, 2016, 9:35 p.m., Vishesh Handa wrote: > > This is awesome. Ship it. Lets hope it is awesome correct. - Christoph --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 128892: Open baloo lmdb database read-only beside in baloo_file/baloo_file_extractor + balooctl (for some commands) + unit tests

2016-09-11 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128892/#review99115 --- Ship it! This is awesome. Ship it. - Vishesh Handa On

[Spam] [Bug 368627] ®((((~> 18885987978 <~)))) WireLess OKI Printer Technical Support Phone Number CANADA...++18885987978+

2016-09-11 Thread Myriam Schweingruber via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368627 Myriam Schweingruber changed: What|Removed |Added Product|Breeze |Spam

[Differential] [Commented On] D2383: [Notifications] Add "Clear Notifications" context menu entry

2016-09-11 Thread colomar (Thomas Pfeiffer)
colomar added a comment. In https://phabricator.kde.org/D2383#50991, @andreaska wrote: > ordinary I had more often the problem that the notification stuff is empty but I'd like to know what happend in the past. > > most of the time I see no new notification but I'd like to see the

[Differential] [Commented On] D1771: [Workspace Options] Add option to disable OSD

2016-09-11 Thread andreaska (Andreas Kainz)
andreaska added a comment. would it help if you add an "screenshot" where you show the information tip and osd? REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1771 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/

[Differential] [Commented On] D1771: [Workspace Options] Add option to disable OSD

2016-09-11 Thread colomar (Thomas Pfeiffer)
colomar added a comment. In https://phabricator.kde.org/D1771#50973, @broulik wrote: > Can we have something more concise than "Visual feedback (On-Screen Display) for status changes not triggered via a graphical user interface". Perhaps just "Visual feedback for status changes"?

[Differential] [Commented On] D2383: [Notifications] Add "Clear Notifications" context menu entry

2016-09-11 Thread andreaska (Andreas Kainz)
andreaska added a comment. ordinary I had more often the problem that the notification stuff is empty but I'd like to know what happend in the past. most of the time I see no new notification but I'd like to see the old ones (5.7.4) maybe 5.8 change this behavior REPOSITORY

[Differential] [Commented On] D2383: [Notifications] Add "Clear Notifications" context menu entry

2016-09-11 Thread colomar (Thomas Pfeiffer)
colomar added a comment. Okay, let's stick with the context menu for now and see if we can find a good place in the popup at some point. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2383 EMAIL PREFERENCES

Re: Review Request 128870: [ToolButtonStyle] Fix menu arrow

2016-09-11 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128870/#review99114 --- src/declarativeimports/plasmastyle/ToolButtonStyle.qml

[Differential] [Commented On] D2741: [Window Switcher] Cleanup

2016-09-11 Thread colomar (Thomas Pfeiffer)
colomar added a comment. Why is the font for the window titles so large? I don't really see much benefit from that... REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2741 EMAIL PREFERENCES

Plasma 5.8 and look and feel layouts

2016-09-11 Thread Łukasz Sawicki
Hi, In the last week I tested a new Plasma 5.8 feature "Look and feel layouts" to have something for our Plasma 5.8 promo materials. Let me share some experiences here. 1. Random crashes (especially when I switch "Look and feel layouts" for the first time. Unfortunately I wasn't able to

[Differential] [Commented On] D2727: don't list "NoDisplay" panels

2016-09-11 Thread broulik (Kai Uwe Broulik)
broulik added a comment. And there's some other madness going on, even with NoDisplay it is not shown as it: - is not a LayoutTempate (so it is not caught by the filter lambda) - it doesnt have X-Plasma-ContainmentType set (so it's not added by the other kservice query) What's

[Differential] [Commented On] D2727: don't list "NoDisplay" panels

2016-09-11 Thread broulik (Kai Uwe Broulik)
broulik added a comment. Now it no longer lists the "Empty Panel" as it is marked as "NoDisplay". REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2727 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mart,

[Differential] [Commented On] D2383: [Notifications] Add "Clear Notifications" context menu entry

2016-09-11 Thread broulik (Kai Uwe Broulik)
broulik added a comment. This was less of a techincal ping but more of a "should we add this thing now" usability question (see hidden comments above) :) REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2383 EMAIL PREFERENCES

[Differential] [Accepted] D2383: [Notifications] Add "Clear Notifications" context menu entry

2016-09-11 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. This revision is now accepted and ready to land. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2383 EMAIL PREFERENCES

[Differential] [Request, 75 lines] D2742: Simplify Panel lastSpacer + fix crash

2016-09-11 Thread davidedmundson (David Edmundson)
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY The panel contains an invisible Item lastSpacer, who will fill the width of the layout if no

[Differential] [Commented On] D1771: [Workspace Options] Add option to disable OSD

2016-09-11 Thread broulik (Kai Uwe Broulik)
broulik added a comment. Can we have something more concise than "Visual feedback (On-Screen Display) for status changes not triggered via a graphical user interface". Perhaps just "Visual feedback for status changes"? REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL

[Differential] [Updated] D2019: [Task Manager] Add media controls in context menu

2016-09-11 Thread broulik (Kai Uwe Broulik)
broulik added a comment. Ping. Once https://phabricator.kde.org/D2029 is in, the duplicates shouldn't be a problem anymore. I checked some players (such as Cantata) and they do actually set the appropriate only show in fields. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL

[Differential] [Commented On] D2033: Clear idle timeouts when session becomes inactive

2016-09-11 Thread broulik (Kai Uwe Broulik)
broulik added a comment. So, should we go with this patch then? Powerdevil is now a separate binary, so IdleTime::instance()->removeAllIdleTimeouts(); shouldn't cause side-effects on other modules and also should fix it for non-weird X setups like mine. ;) REPOSITORY

[Differential] [Commented On] D2383: [Notifications] Add "Clear Notifications" context menu entry

2016-09-11 Thread broulik (Kai Uwe Broulik)
broulik added a comment. Ping REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2383 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, #plasma:_design, mck182 Cc: colomar, plasma-devel,

[Differential] [Request, 155 lines] D2741: [Window Switcher] Cleanup

2016-09-11 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, Plasma: Design. broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This cleans up

[Breeze] [Bug 368627] New: ®((((~> 18885987978 <~)))) WireLess OKI Printer Technical Support Phone Number CANADA...++18885987978+

2016-09-11 Thread Johnson via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368627 Bug ID: 368627 Summary: ®~> 18885987978 <~ WireLess OKI Printer Technical Support Phone Number CANADA...++18885987978+ Product: Breeze Version: unspecified Platform:

Re: Review Request 128892: Open baloo lmdb database read-only beside in baloo_file/baloo_file_extractor + balooctl (for some commands) + unit tests

2016-09-11 Thread Christoph Cullmann
> On Sept. 11, 2016, 7:19 p.m., Boudhayan Gupta wrote: > > Can you add Vishesh and poke him if possible? This seems like a gigantic > > patch and I'm not sure I understand all of it right now. Hi, I can poke him, sure. Given there is no documentation what the things shall do, I am not 100%

Re: Review Request 128892: Open baloo lmdb database read-only beside in baloo_file/baloo_file_extractor + balooctl (for some commands) + unit tests

2016-09-11 Thread Boudhayan Gupta
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128892/#review99110 --- Can you add Vishesh and poke him if possible? This seems

Review Request 128892: Open baloo lmdb database read-only beside in baloo_file/baloo_file_extractor + balooctl (for some commands) + unit tests

2016-09-11 Thread Christoph Cullmann
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128892/ --- Review request for KDE Frameworks, Plasma and Boudhayan Gupta.

Re: Review Request 128890: Make e.g. Baloo::Query thread safe.

2016-09-11 Thread Christoph Cullmann
> On Sept. 11, 2016, 6:32 p.m., Anthony Fieroni wrote: > > src/engine/database.cpp, line 128 > > > > > > It's needed *m_env = 0;* > > Or you can add method close to call it in all way. > > Christoph

Re: Review Request 128890: Make e.g. Baloo::Query thread safe.

2016-09-11 Thread Anthony Fieroni
> On Септ. 11, 2016, 9:32 след обяд, Anthony Fieroni wrote: > > src/engine/database.cpp, line 128 > > > > > > It's needed *m_env = 0;* > > Or you can add method close to call it in all way. > > Christoph

Re: Review Request 128890: Make e.g. Baloo::Query thread safe.

2016-09-11 Thread Christoph Cullmann
> On Sept. 11, 2016, 6:32 p.m., Anthony Fieroni wrote: > > src/engine/database.cpp, line 128 > > > > > > It's needed *m_env = 0;* > > Or you can add method close to call it in all way. I don't understand

Re: Review Request 128890: Make e.g. Baloo::Query thread safe.

2016-09-11 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128890/#review99103 --- src/engine/database.cpp (line 128)

Re: Review Request 128890: Make e.g. Baloo::Query thread safe.

2016-09-11 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128890/#review99102 --- src/engine/database.h (line 60)

Re: Review Request 128890: Make e.g. Baloo::Query thread safe.

2016-09-11 Thread Christoph Cullmann
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128890/ --- (Updated Sept. 11, 2016, 9:27 p.m.) Status -- This change has been

Re: Review Request 128890: Make e.g. Baloo::Query thread safe.

2016-09-11 Thread Boudhayan Gupta
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128890/#review99101 --- Ship it! Ship It! - Boudhayan Gupta On Sept. 11, 2016,

Review Request 128890: Make e.g. Baloo::Query thread safe.

2016-09-11 Thread Christoph Cullmann
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128890/ --- Review request for KDE Frameworks, Plasma and Boudhayan Gupta.

Re: Taking over maintainership of Baloo

2016-09-11 Thread Dominik Haumann
Hi, indeed I also did take a look at the bug list, and out of the 200 bugs, without much exaggeration almost all of them are crashes. In fact, I believe it's important to CC the plasma-devel mailing list here: Given we are about to release a LTS Plasma 5.8, and baloo is tightly integrated into

[Differential] [Request, 2 lines] D2740: Use correct config property for show only current screen in pager

2016-09-11 Thread davidedmundson (David Edmundson)
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. TEST PLAN Tested it worked. Also checked there were no warnings any more. REPOSITORY rPLASMADESKTOP Plasma

Re: Review Request 128684: Proofread + update khtml-general kcm docbook

2016-09-11 Thread Luigi Toscano
> On Ago. 16, 2016, 9:05 a.m., David Faure wrote: > > doc/kcontrol/khtml-general/index.docbook, line 51 > > > > > > Well, qt5-webkit and kwebkitpart do still exist. They're just not > > really maintained (but

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

2016-09-11 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/320/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 11 Sep 2016 09:42:47 + Build duration: 17 min CHANGE SET Revision

Re: Review Request 128728: Update plasma handbook

2016-09-11 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128728/#review99077 --- ping? - Burkhard Lück On Aug. 25, 2016, 3:34 nachm.,

Re: Review Request 128771: powerdevil lcm docbook update to 5.7

2016-09-11 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128771/#review99073 --- Please review, thanks - Burkhard Lück On Aug. 26, 2016,

Re: Review Request 128741: Update paths docbook

2016-09-11 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128741/ --- (Updated Sept. 11, 2016, 9:42 a.m.) Status -- This change has been