[Differential] [Closed] D1831: Add rule for chromium to taskmanagerrulesrc for proper grouping.

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAWORKSPACE110f58471510: Add rule for chromium to taskmanagerrulesrc for proper grouping.` (authored by pinakahuja). REPOSITORY rPLASMAWORKSPACE Plasma Workspace CHANGES SINCE LAST UPDATE https://

[Differential] [Request, 6 lines] D1831: Add rule for chromium to taskmanagerrulesrc for proper grouping.

pinakahuja created this revision. pinakahuja added a reviewer: Plasma. pinakahuja set the repository for this revision to rPLASMAWORKSPACE Plasma Workspace. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REPOSITORY rPLASMAWORKSPACE Plasm

Re: Review Request 125129: Update baloo's D-Bus interface name in KCM

marked as submitted. Review request for Plasma and Vishesh Handa. Changes --- Submitted with commit 93e437c684ed57fefcf2f5821edd52008e60c2a2 by Pinak Ahuja to branch master. Repository: plasma-desktop Description --- Also use the new Baloo::IndexerConfig::refresh() Diffs

Re: Review Request 125129: Update baloo's D-Bus interface name in KCM

Handa. Repository: plasma-desktop Description (updated) --- Also use the new Baloo::IndexerConfig::refresh() Diffs (updated) - kcms/baloo/kcm.cpp 8012a3b Diff: https://git.reviewboard.kde.org/r/125129/diff/ Testing --- Thanks, Pinak Ahuja

Re: Review Request 125129: Update baloo's D-Bus interface name in KCM

> On Sept. 15, 2015, 9:33 p.m., Vishesh Handa wrote: > > Maybe we could just add this to our `IndexerConfig` class so that users of > > our API don't need to know about dbus interfaces? Specially since I'm not > > too keen on maintaining API comatibility in the d

Re: Review Request 125129: Update baloo's D-Bus interface name in KCM

> On Sept. 15, 2015, 9:33 p.m., Vishesh Handa wrote: > > Maybe we could just add this to our `IndexerConfig` class so that users of > > our API don't need to know about dbus interfaces? Specially since I'm not > > too keen on maintaining API comatibility in the d

Re: Review Request 125369: Baloo runner: look also for presentations/spreadsheets/text files

> On Sept. 24, 2015, 11:50 p.m., Vishesh Handa wrote: > > runners/baloo/baloosearchrunner.cpp, line 165 > > > > > > Have a look at basicindexingjob.txt. We define that all Presentation > > and Spreadsheets are a

Re: Review Request 125129: Update baloo's D-Bus interface name in KCM

it.reviewboard.kde.org/r/125129/#review85462 --- On Sept. 10, 2015, 5:56 a.m., Pinak Ahuja wrote: > > --- > This is an automatically generated e-mail. To reply, visit: &g

Review Request 125129: Update baloo's D-Bus interface name in KCM

Description --- Also generate interface from XML Diffs - kcms/baloo/CMakeLists.txt 7415289 kcms/baloo/kcm.cpp d85f615 Diff: https://git.reviewboard.kde.org/r/125129/diff/ Testing --- Thanks, Pinak Ahuja ___ Plasma-devel

Re: Review Request 124962: Fix CMAKE_INSTALL_LIBDIR not respecting installation prefix

marked as submitted. Review request for Plasma and Martin Klapetek. Changes --- Submitted with commit 105cd18dbfe644e510e772a7e8226d8e965477f9 by Pinak Ahuja to branch master. Repository: kwallet-pam Description --- pam_kwallet5.so was being installed to /lib(64) irrespective of

Re: Review Request 124962: Fix CMAKE_INSTALL_LIBDIR not respecting installation prefix

> > > > Also, are pam modules actually supported outside of the default dir (I > > have no clue)? > > Pinak Ahuja wrote: > According to this: > http://www.cmake.org/cmake/help/v3.0/module/GNUInstallDirs.html the syntax > I've used is correct. >

Re: Review Request 124962: Fix CMAKE_INSTALL_LIBDIR not respecting installation prefix

e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124962/#review84546 ------- On Aug. 28, 2015, 7:39 a.m., Pinak Ahuja wrote: > > --- > This is an automatically ge

Re: Review Request 124962: Fix CMAKE_INSTALL_LIBDIR not respecting installation prefix

--- pam_kwallet5.so seems to install under the directory set as CMAKE_INSTALL_PREFIX Thanks, Pinak Ahuja ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: baloo monitor visibility

monitor's functionality, which will show up in the tray. Maybe we can hide the monitor from applications and add a button to plasmoid? i.e. if we do end up shipping the plasmoid. * Generic name makes sense. -- Pinak Ahuja On 15 July 2015 at 13:56, Mark Gaiser wrote: > > >> Also just my t

Re: Review Request 123088: Adding libkactivities-stats to the build

On 21-Mar-2015 2:38 am, "Ivan Čukić" wrote: > > This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123088/ > > Review request for Plasma, Eike Hein and Marco Martin. > By Ivan Čukić. > Repository: plasma-desktop > Description > > The experimental library

Re: Review Request 123040: Make selection in krunner wrapping

- lib/qml/ResultsView.qml 0eac7b8 Diff: https://git.reviewboard.kde.org/r/123040/diff/ Testing --- Compiled and tested on my pc, works as described. Thanks, Pinak Ahuja ___ Plasma-devel mailing list Plasma-devel@kde.org https

Re: Review Request 123040: Make selection in krunner wrapping

on wrap to bottom. M +1-1lib/qml/ResultsView.qml http://commits.kde.org/milou/05570618369027c8636db9abbe3726ece6fe8e72 - Pinak Ahuja On March 20, 2015, 4:14 p.m., Pinak Ahuja wrote: > > --- > This is an automatically g

Re: Review Request 123040: Make selection in krunner wrapping

d.kde.org/r/123040/#review77712 --- On March 18, 2015, 7:06 p.m., Pinak Ahuja wrote: > > --- > This is an automatically generated e-mail. To reply, visit: &g

Re: Review Request 123040: Make selection in krunner wrapping

: https://git.reviewboard.kde.org/r/123040/diff/ Testing --- Compiled and tested on my pc, works as described. Thanks, Pinak Ahuja ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 123040: Make selection in krunner wrapping

/ Testing --- Compiled and tested on my pc, works as described. Thanks, Pinak Ahuja ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 122328: Fix apply button in breeze window decoration config

ion/config/breezeconfigwidget.cpp e160c57 Diff: https://git.reviewboard.kde.org/r/122328/diff/ Testing --- Compiled and installed, apply button works as expected. Thanks, Pinak Ahuja ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.o

Re: Review Request 122328: Fix apply button in breeze window decoration config

hip It! > > PS: do you have commit access ? Otherwise I can commit for you. > > > - Hugo > > On January 30th, 2015, 11:42 a.m. UTC, Pinak Ahuja wrote: > Review request for Plasma and Hugo Pereira Da Costa. > By Pinak Ahuja. > > *Updated Jan. 30, 2015, 11:42 a.m.* &

Re: Review Request 122328: Fix apply button in breeze window decoration config

I might be wrong, but shouldn't the changed signal have a bool parameter? The changed() without a parameter refers to a slot(can we emit a slot? I'm new to this). I didn't compile and check if the behaviour was fixed in the master branch without this patch, I have the problem with the plasma 5.2 pa

Review Request 122328: Fix apply button in breeze window decoration config

Testing --- Compiled and installed, apply button works as expected. Thanks, Pinak Ahuja ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Plasma 5.2 beta

It would be better, if you could verify these things in 5.2 stable. If you are able to reproduce them, file a bug report at https://bugs.kde.org ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Idea for plasma media center

ugh I haven't really thought of how I would implement this. Thanks, Pinak Ahuja ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel