D15357: [Bookmarks Runner] Remove duplicate results for bookmarks

bruns added a comment. qdbus-qt5 org.kde.krunner /App org.kde.krunner.App.querySingleRunner bookmarks kde Current state: F6298174: Bookmarks_Current.png Deduplicated: F6298173: Bookmarks_Deduplicated.png

D15357: [Bookmarks Runner] Remove duplicate results for bookmarks

bruns added a comment. I would really appreciate if we could get this into the imminent Plasma release. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D15357 To: bruns, #plasma Cc: davidedmundson, zzag, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesli

D15892: [Devicenotifications Engine] Keep at most one notification per UDI

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Each notification was created as new datasource, and was newer removed as long as the engine exist

D15892: [Devicenotifications Engine] Keep at most one notification per UDI

bruns marked an inline comment as done. bruns added inline comments. INLINE COMMENTS > broulik wrote in devicenotificationsengine.cpp:39 > This potentially breaks applets relying on the structure of that name? Not > sure how big of an issue this is as far as "API promise" we give for > dataengi

D15892: [Devicenotifications Engine] Keep at most one notification per UDI

This revision was automatically updated to reflect the committed changes. bruns marked an inline comment as done. Closed by commit R120:17380886d9b8: [Devicenotifications Engine] Keep at most one notification per UDI (authored by bruns). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D1589

D15357: [Bookmarks Runner] Remove duplicate results for bookmarks

This revision was automatically updated to reflect the committed changes. Closed by commit R120:e02f3fcb1347: [Bookmarks Runner] Remove duplicate results for bookmarks (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15357?vs=42615&

D15885: Clean UI file for `details` portion of UI

bruns requested changes to this revision. bruns added a comment. This revision now requires changes to proceed. Can you ammend this with the removals of appname/m_appname? If you need any assistance, please ask! REPOSITORY R121 Policykit (Polkit) KDE Agent REVISION DETAIL https://phabri

D16402: Cleanup - remove unnecessary QIcon include

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY QIcon is only used in the implementation, not in the interface. TEST PLAN compile tested, no func

D16403: Add services and appstream runner debug categories to categories file

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Make the debug categories visible in kdebugsettings REPOSITORY R120 Plasma Workspace BRANCH ma

D16404: [Bookmarks Runner] Cleanup tests CMakeList

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Build the test dependencies as static library. Remove duplicate and commented out lines. Use ecm_a

D16402: Cleanup - remove unnecessary QIcon include

This revision was automatically updated to reflect the committed changes. Closed by commit R120:9945aecaf2eb: Cleanup - remove unnecessary QIcon include (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16402?vs=44159&id=44163 REVIS

D16405: Add debug category for bookmarks runner

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Dependency for followup patches, no functional change TEST PLAN make REPOSITORY R120 Plasma Wo

D16405: Add debug category for bookmarks runner

bruns added a comment. Depends on the cleanup in D16404 , as the autogenerated `bookmarks_debug.cpp` is not linked to the tests/testChromeBookmarks binary otherwise. REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator

D16405: Add debug category for bookmarks runner

bruns added a comment. Only applies cleanly after D16403 . REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D16405 To: bruns, #frameworks, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai

D16409: [Bookmarks Runner] Open database connection in the query thread

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY QSqlDatabase connection instances are global reference counted objects which can be crated using t

D16410: [Bookmarks Runner] Avoid leaking FetchSqlite instances

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The favicon and places instances are created in the prepare() call, leaking the instance from the

D16417: Improve replacement text when action description is not provided

bruns created this revision. bruns added reviewers: Plasma, ngraham, davidedmundson. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The polkit rules specification mandates a description for actions, https://ww

D12498: Fully remove `Application Name` from Details panel

bruns updated this revision to Diff 44184. bruns added a comment. Update UI file Split out "description" change REPOSITORY R121 Policykit (Polkit) KDE Agent CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D12498?vs=42375&id=44184 BRANCH arcpatch-D12498 REVISION DETAIL https:

D16417: Improve replacement text when action description is not provided

bruns added inline comments. INLINE COMMENTS > ngraham wrote in AuthDialog.cpp:356 > So I gather `.isEmpty()` works now? Yes. > ngraham wrote in AuthDialog.cpp:359 > I would prefer "not available" over "not provided", but I won't choose this > as my hill to die on. :) > > Do we actually need

D16403: Add services and appstream runner debug categories to categories file

This revision was automatically updated to reflect the committed changes. Closed by commit R120:17a0d34d0e83: Add services and appstream runner debug categories to categories file (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D164

D15885: Clean UI file for `details` portion of UI

bruns abandoned this revision. bruns added a comment. Folded back into D12498 REPOSITORY R121 Policykit (Polkit) KDE Agent REVISION DETAIL https://phabricator.kde.org/D15885 To: bruns, ngraham, davidedmundson, sharvey Cc: bruns, plasma-devel, ragreen,

D16417: Improve replacement text when action description is not provided

bruns added inline comments. INLINE COMMENTS > ngraham wrote in AuthDialog.cpp:359 > I would prefer "not available" over "not provided", but I won't choose this > as my hill to die on. :) > > Do we actually need to repeat the word 'Description' here? Btw, "not provided" is your own proposal ;-

D16417: Improve replacement text when action description is not provided

This revision was automatically updated to reflect the committed changes. Closed by commit R121:9cf71f424950: Improve replacement text when action description is not provided (authored by bruns). REPOSITORY R121 Policykit (Polkit) KDE Agent CHANGES SINCE LAST UPDATE https://phabricator.kde.o

D12498: Fully remove `Application Name` from Details panel

This revision was automatically updated to reflect the committed changes. Closed by commit R121:eb9c4c080427: Fully remove `Application Name` from Details panel (authored by sharvey, committed by bruns). REPOSITORY R121 Policykit (Polkit) KDE Agent CHANGES SINCE LAST UPDATE https://phabricat

D16530: Drop X11 core fonts code from startkde/startplasma

bruns created this revision. bruns added reviewers: Plasma, fvogt. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Qt and GTK based applications have switched to using fontconfig for fonts exclusively, so this c

D16404: [Bookmarks Runner] Cleanup tests CMakeList

bruns added a comment. Ping! REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D16404 To: bruns, #frameworks Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D16530: Drop X11 core fonts code from startkde/startplasma

bruns added a comment. Friendly reminder ... REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D16530 To: bruns, #plasma, fvogt, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D16530: Drop X11 core fonts code from startkde/startplasma

bruns added a comment. In D16530#355078 , @fvogt wrote: > Looks good to me, but I don't know enough about old X11 stuff like this to accept it. @fvogt - maybe you can poke a coworker? Someone else? REPOSITORY R120 Plasma Workspace

D16404: [Bookmarks Runner] Cleanup tests CMakeList

bruns added a comment. Anyone? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D16404 To: bruns, #frameworks Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D16530: Drop X11 core fonts code from startkde/startplasma

This revision was automatically updated to reflect the committed changes. Closed by commit R120:5374513266ee: Drop X11 core fonts code from startkde/startplasma (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16530?vs=44499&id=4515

D16404: [Bookmarks Runner] Cleanup tests CMakeList

This revision was automatically updated to reflect the committed changes. Closed by commit R120:5ba0a08b8312: [Bookmarks Runner] Cleanup tests CMakeList (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16404?vs=44161&id=45351 REVIS

D16405: Add debug category for bookmarks runner

This revision was automatically updated to reflect the committed changes. Closed by commit R120:62804f6becb9: Add debug category for bookmarks runner (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16405?vs=44164&id=45352 REVISION

D16409: [Bookmarks Runner] Open database connection in the query thread

This revision was automatically updated to reflect the committed changes. Closed by commit R120:145caeac1050: [Bookmarks Runner] Open database connection in the query thread (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16409?vs=

D16410: [Bookmarks Runner] Avoid leaking FetchSqlite instances

This revision was automatically updated to reflect the committed changes. Closed by commit R120:9e90fdd2a593: [Bookmarks Runner] Avoid leaking FetchSqlite instances (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16410?vs=44170&id=

[kio-extras] [Bug 329687] previewing an HTML file from a local file system causes network retrievals while generating the thumbnail

https://bugs.kde.org/show_bug.cgi?id=329687 Stefan Brüns changed: What|Removed |Added CC||stefan.bruens@rwth-aachen.d

D18890: Add button to reset index database and repair Baloo crashing

bruns added a comment. In D18890#409359 , @ngraham wrote: > Thanks! This is pretty good as-is, and I can confirm that it works just fine. However once the user presses the button, there's no further feedback, which could encourage them to repeat

D18890: Add button to reset index database and repair Baloo crashing

bruns added inline comments. INLINE COMMENTS > kcm.cpp:197 > +const QString rm = QStandardPaths::findExecutable(QStringLiteral("rm")); > +const QString balooctl = > QStandardPaths::findExecutable(QStringLiteral("balooctl")); > +const QString baloofile = > QStandardPaths::findExecuta

D18890: Add button to reset index database and repair Baloo crashing

bruns added a comment. In D18890#410137 , @davidedmundson wrote: > If we know the DB is corrupted, and it's just a cache, why do we need a user facing button? There are many possible ways of corruption: - non-decodable values - en

D19963: Avoid serializing base64 encoded favicon data twice

bruns created this revision. bruns added reviewers: Plasma, broulik. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The URL is ignored in case the image data is available. Avoid pushing it twice via DBus. TEST

D19963: Avoid serializing base64 encoded favicon data twice

bruns added a comment. Ping! REPOSITORY R856 Plasma Browser Integration REVISION DETAIL https://phabricator.kde.org/D19963 To: bruns, #plasma, broulik Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D19963: Avoid serializing base64 encoded favicon data twice

bruns added a comment. Ping! REPOSITORY R856 Plasma Browser Integration REVISION DETAIL https://phabricator.kde.org/D19963 To: bruns, #plasma, broulik Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D19963: Avoid serializing base64 encoded favicon data twice

This revision was automatically updated to reflect the committed changes. Closed by commit R856:c3611dc98170: Avoid serializing base64 encoded favicon data twice (authored by bruns). REPOSITORY R856 Plasma Browser Integration CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19963?vs=54

D19823: Port to the new install directory for knsrc files

bruns added a comment. Now plasma-workspace depends on KNewStuff 5.57 REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D19823 To: apol, ngraham Cc: bruns, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts,

D20382: [KRunner] Move trivial check in addToHistory to the front

bruns created this revision. bruns added reviewers: Plasma, broulik, apol. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY No functional change. TEST PLAN compiles, runs REPOSITORY R120 Plasma Workspace REV

D20383: [KRunner] Avoid writing history if addToHistory is idempotent

bruns created this revision. bruns added reviewers: Plasma, broulik, apol. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY It is not uncommon to run the same command repeatedly. In this case, the history is actu

D20383: [KRunner] Avoid writing history if addToHistory is idempotent

bruns updated this revision to Diff 55793. bruns added a comment. use const REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20383?vs=55748&id=55793 BRANCH arcpatch-D20383 REVISION DETAIL https://phabricator.kde.org/D20383 AFFECTED FILES kru

D20382: [KRunner] Move trivial check in addToHistory to the front

This revision was automatically updated to reflect the committed changes. Closed by commit R120:4a5f62fee952: [KRunner] Move trivial check in addToHistory to the front (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20382?vs=55747&

D20383: [KRunner] Avoid writing history if addToHistory is idempotent

This revision was automatically updated to reflect the committed changes. Closed by commit R120:9686c05c7226: [KRunner] Avoid writing history if addToHistory is idempotent (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20383?vs=55

D25991: [KFilePlacesModel] Fix supported scheme check for devices

bruns added inline comments. INLINE COMMENTS > kfileplacesmodel.cpp:758 > +KFilePlacesItem *item = nullptr; > +if (deviceAvailable) { > +item = new KFilePlacesItem(bookmarkManager, > bookmark.address(), udi); you have dropped the `allowedHere`

D27579: Remove attempt to start long gone baloo_file_cleaner

bruns created this revision. bruns added reviewers: Baloo, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The standalone index cleaner has been replaced almost 5 years by functionality implemented in b

D27579: Remove attempt to start long gone baloo_file_cleaner

This revision was automatically updated to reflect the committed changes. Closed by commit R119:616a99165eda: Remove attempt to start long gone baloo_file_cleaner (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27579?vs=76181&id=7619

D27611: Invert logic of shouldShowMountPoint -> ignoreMountPoint

bruns created this revision. bruns added reviewers: Baloo, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Instead of negating the logic values in several places, just invert the logic. REPOSITORY R1

D27612: Remove unused functions

bruns created this revision. bruns added reviewers: Baloo, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY showMessage() is only declared, but never defined. includeFolders() and fetchMountPoint(url)

D27611: Invert logic of shouldShowMountPoint -> ignoreMountPoint

bruns updated this revision to Diff 76251. bruns added a comment. move out of class REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27611?vs=76249&id=76251 REVISION DETAIL https://phabricator.kde.org/D27611 AFFECTED FILES kcms/baloo/filteredfold

D27612: Remove unused functions

bruns updated this revision to Diff 76252. bruns added a comment. rebase REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27612?vs=76250&id=76252 REVISION DETAIL https://phabricator.kde.org/D27612 AFFECTED FILES kcms/baloo/filteredfoldermodel.cpp

D27611: Invert logic of shouldShowMountPoint -> ignoreMountPoint

bruns added a comment. In D27611#616556 , @usta wrote: > 2 Question : > 1- isnt that method should be const'ified ? Its not a member function ... REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27611

D27611: Invert logic of shouldShowMountPoint -> ignoreMountPoint

bruns updated this revision to Diff 76253. bruns added a comment. ignoreMountPoint -> ignoredMountPoint whitespace REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27611?vs=76251&id=76253 REVISION DETAIL https://phabricator.kde.org/D27611 AFFECTE

D27613: Remove bogus mount points check when starting baloo on save

bruns created this revision. bruns added reviewers: Baloo, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The check stops baloo if non of the included pathes is *currently* mounted, i.e. if a device is

D27611: Invert logic of shouldShowMountPoint -> ignoredMountPoint

bruns updated this revision to Diff 76329. bruns retitled this revision from "Invert logic of shouldShowMountPoint -> ignoreMountPoint" to "Invert logic of shouldShowMountPoint -> ignoredMountPoint". bruns edited the summary of this revision. bruns added a comment. Update commit message (I ha

D27611: Invert logic of shouldShowMountPoint -> ignoredMountPoint

This revision was automatically updated to reflect the committed changes. Closed by commit R119:31e823b6a67a: Invert logic of shouldShowMountPoint -> ignoredMountPoint (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27611?vs=76329&id

D27612: Remove unused functions

This revision was automatically updated to reflect the committed changes. Closed by commit R119:cd52299862ea: Remove unused functions (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27612?vs=76252&id=76331 REVISION DETAIL https://p

D27613: Remove bogus mount points check when starting baloo on save

This revision was automatically updated to reflect the committed changes. Closed by commit R119:9dc634bdb1ec: Remove bogus mount points check when starting baloo on save (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27613?vs=76254&

D27636: Remove redundant match for mountpoints

bruns created this revision. bruns added reviewers: Baloo, ngraham, davidedmundson. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY m_excludeList is populated from m_settings->excludeFolders(), so one of the che

D27636: Remove redundant match for mountpoints

This revision was automatically updated to reflect the committed changes. Closed by commit R119:0acbfba3b0e4: Remove redundant match for mountpoints (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27636?vs=76333&id=76432 REVISION DE

D27917: [Baloo] Fix firstRun and config change logic

bruns created this revision. bruns added reviewers: Baloo, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY FirstRun should only be set if baloo was *never ever* run before, and this case is already hand

D27917: [Baloo] Fix firstRun and config change logic

This revision was automatically updated to reflect the committed changes. Closed by commit R119:550cf9cafb19: [Baloo] Fix firstRun and config change logic (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27917?vs=77183&id=77456 REVIS

D28024: [Baloo KCM] Expose "Index hidden" config setting

bruns created this revision. bruns added reviewers: Baloo, VDG, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Also inset both "Index file content" and "Index hidden" so it is more obvious both are sub

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns created this revision. bruns added reviewers: Baloo, VDG, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The current "Excluded folders" list in the KCM is quite awkward: 1. It tries to mimic b

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns added a comment. F8174362: Screenshot_20200313_145738.png F8174361: Screenshot_20200313_145811.png REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D28025 To: bruns, #balo

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns added inline comments. INLINE COMMENTS > main.qml:169 > +id: removeFolder > + visible: model.deletable > +icon.name: "user-trash" Spurious tab REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D28025 To

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns updated this revision to Diff 77574. bruns added a comment. tabs REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28025?vs=77560&id=77574 BRANCH baloo_config_rework REVISION DETAIL https://phabricator.kde.org/D28025 AFFECTED FILES kcms/b

D28024: [Baloo KCM] Expose "Index hidden" config setting

bruns added inline comments. INLINE COMMENTS > ngraham wrote in main.qml:59 > Instead adding these new layouts, you could just set the following on each > checkbox that needs to be indented: > > Layout.leftMargin: !LayoutMirroring.enabled ? Kirigami.Units.largeSpacing : > 0 > Layout.rightMa

D28024: [Baloo KCM] Expose "Index hidden" config setting

This revision was automatically updated to reflect the committed changes. Closed by commit R119:491ce0f1d78f: [Baloo KCM] Expose "Index hidden" config setting (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28024?vs=77558&id=77662 R

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns added a comment. In D28025#630729 , @ngraham wrote: > I have some UI suggestions: > > - Have a button to add an exclusion path as well as a button to add an inclusion path, rather than a single Add Setting button, which is a rather jar

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns added a comment. In D28025#630882 , @ngraham wrote: > In D28025#630849 , @bruns wrote: > > > Having only inclusion/exclusion is a temporary state. I plan to add more settings for each path, so

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns updated this revision to Diff 78064. bruns added a comment. Use Kirigami.Action for Trash REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28025?vs=77574&id=78064 BRANCH baloo_config_rework REVISION DETAIL https://phabricator.kde.org/D28025

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns added a comment. F8185503: Screenshot_20200320_022339.png REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D28025 To: bruns, #baloo, #vdg, ngraham, mart Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, j

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns added a comment. TODO: T12840 REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D28025 To: bruns, #baloo, #vdg, ngraham, mart Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas,

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns added a comment. In D28025#631358 , @ngraham wrote: > If you're okay with me modifying the UI in a follow-up patch, I'll accept the UI in its current form. Is that acceptable? Thats the intention of all this - get it in a usable sta

D28208: Move sni icon handling logic from data engine to applet

bruns added inline comments. INLINE COMMENTS > kmaterka wrote in StatusNotifierItem.qml:90 > Yes, you are correct, my mistake. So `StatusNotifierItemSource` has no margin > and `KIconLoader` has 5% of icon size - another inconsistency. > What about 'always round when multiplying by a non-integer

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns marked 2 inline comments as done. bruns added inline comments. INLINE COMMENTS > ngraham wrote in filteredfoldermodel.cpp:36 > this handy little function feels like it wants to be in KCoreAddons or > something Now its likely inlined by the compiler ... > ngraham wrote in main.qml:112 > S

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns updated this revision to Diff 78332. bruns marked 2 inline comments as done. bruns added a comment. update REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28025?vs=78064&id=78332 BRANCH baloo_config_rework REVISION DETAIL https://phabricat

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

bruns updated this revision to Diff 78333. bruns marked 2 inline comments as done. bruns added a comment. update2 REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28025?vs=78332&id=78333 BRANCH baloo_config_rework REVISION DETAIL https://phabrica

D28025: [Baloo KCM] Complete overhaul of the include/exclude folder list

This revision was automatically updated to reflect the committed changes. Closed by commit R119:7498b41a1979: [Baloo KCM] Complete overhaul of the include/exclude folder list (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28025?vs=7

D28280: [kcms/baloo] Refine UI and prepare for more indexing options in the future

bruns added a comment. Try do imagine how the combobox will look like when you add 'Index hidden' to the config options. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D28280 To: ngraham, #vdg, #baloo, #plasma Cc: bruns, plasma-devel, Orage, LeGast00n, The-Fe

D28280: [kcms/baloo] Refine UI and prepare for more indexing options in the future

bruns added a comment. For the "Add folder setting...": How about a config dialog **after** the File Dialog? This may seem over the top for now, but the next step is to make the excludeFilters and mimetype filters configurabe from the KCM, and then also per folder. REPOSITORY R119 P

D28280: [kcms/baloo] Refine UI and prepare for more indexing options in the future

bruns added a comment. In D28280#634405 , @ngraham wrote: > In D28280#634388 , @bruns wrote: > > > Try do imagine how the combobox will look like when you add 'Index hidden' to the config options.

D28280: [kcms/baloo] Refine UI and prepare for more indexing options in the future

bruns added a comment. In D28280#634471 , @ngraham wrote: > Such a complicated setup is probably not going to be typical, I expect. But for that we could have an Advanced Settings sheet that allows people to customize things with great detail. H

D28196: BUGFIX BookmarksRunner: Fix wrong default profile extraction, remove caching

bruns added a comment. Can you split out the early-return part for the QSQLite driver and submit it as a preparation patch? It adds quite some noise to the diff due to the whitespace changes (though I fully agree it is the better coding pattern). REPOSITORY R120 Plasma Workspace REVI

D28196: BUGFIX BookmarksRunner: Fix wrong default profile extraction, remove caching

bruns added a comment. Regarding the summary: 1. Wrap the lines at <80 characters 2. Describe the screenshot textually REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D28196 To: alex, bruns, z3ntu, apol, broulik, davidedmundson, #plasma Cc: plasma-deve

D28196: BUGFIX BookmarksRunner: Fix wrong default profile extraction, remove caching

bruns added a comment. Also keep code lines at < 80 characters REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D28196 To: alex, bruns, z3ntu, apol, broulik, davidedmundson, #plasma Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus

D28196: BUGFIX BookmarksRunner: Fix wrong default profile extraction, remove caching

bruns requested changes to this revision. bruns added a comment. This revision now requires changes to proceed. This should be 3 submission: 1. early return/indentation changes (preparation) 2. always read default profile from FF config, do not write "dbfile" to runner config 3. fix l

D28439: BoorkmarksRunner: Set QIcon instead of Favicon pointer in BookmarkMatch

bruns added a comment. In D28439#638216 , @alex wrote: > In D28439#638188 , @davidedmundson wrote: > > > QIcon isn't thread safe. > > > Sorry, I wasn't aware about that and haven't found anythi

D28439: BoorkmarksRunner: Set QIcon instead of Favicon pointer in BookmarkMatch

bruns added a comment. In D28439#638188 , @davidedmundson wrote: > QIcon isn't thread safe. I think this is related to https://phabricator.kde.org/D5889?id=14601 ? I think we are save here, as the QIcon is created from an image store

D28439: BoorkmarksRunner: Set QIcon instead of Favicon pointer in BookmarkMatch

bruns added a comment. @davidedmundson - is `QIcon(QImage(...)) `save? the AppstreamRunner suffers from the same problem, it uses `match->setIcon(/*wrapped*/QIcon::fromTheme())`. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D28439 To: alex, apol, bro

D28473: BookmarksRunner: Early return and formatting

bruns added inline comments. INLINE COMMENTS > firefox.cpp:198 > -if (profilePath.isEmpty()) { > -//qDebug() << "No default firefox profile found"; > -return; Can you leave this in, and even promote it to an active qWarning* - if we have a db and prof

D28473: BookmarksRunner: Early return and formatting

bruns added inline comments. INLINE COMMENTS > firefox.cpp:166 > +if (!QSqlDatabase::isDriverAvailable(QStringLiteral("QSQLITE"))) { > +qWarning() << "SQLITE driver isn't available"; > +return; Thanks, though I was wrong about "no categories yet". #include "bookmarks_debug

D28473: BookmarksRunner: Early return and formatting

bruns added a comment. LGTM - @broulik ? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D28473 To: alex, bruns, broulik, #plasma Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, h

D28619: BookmarksRunner: Remove caching of profile database path

bruns added a comment. There are two possible ways of changing the caching, and each one will break behavior for one of two different groups of users: 1. People who rely on the krunner following the default profile. 2. People who rely on krunner using the dbfile config entry. I wou

D28619: BookmarksRunner: Remove caching of profile database path

bruns requested changes to this revision. bruns added a comment. This revision now requires changes to proceed. Please just remove the `writeEntry`. You can then propose "But what about renaming the dbfile entry, maybe to alternate_dbfile?" as another review. REPOSITORY R120 Plasma Wor

<    1   2   3   >