D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added inline comments. INLINE COMMENTS > broulik wrote in dolphinmainwindow.cpp:2289 > > It is still impossible to detect (un)installation of such tool. > > There's signals when the KSycoca service database changes which we use in the > application menu to update when apps are

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added inline comments. INLINE COMMENTS > broulik wrote in dolphinmainwindow.cpp:2289 > > (can be modified with More Search Tools -> More -> Configure) > > Then there should be a signal in `KMoreTools` a user of this class can > connect to to get notified when user preference

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added inline comments. INLINE COMMENTS > broulik wrote in dolphinmainwindow.cpp:2289 > > It is a rare occasion that user changes preferred search tools, > > So, the URL does not have any impact on the search tool chosen? I thought > that was the main reason for using the URL and

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added inline comments. INLINE COMMENTS > broulik wrote in dolphinmainwindow.cpp:943 > `exec()` on a job is generally bad, but looks like it was like this already? Yes, it was like that in DolphinMainWindow::openTerminal() before. It was needed as explained in this comment: // If

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added inline comments. INLINE COMMENTS > ngraham wrote in dolphinmainwindow.cpp:1162 > Since this patch was made, we've re-organized the hamburger menu a bit to > simplify it and hide not-commonly-used functionality. I think this feature > counts, since it's an alternative to the

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added inline comments. INLINE COMMENTS > ngraham wrote in dolphinmainwindow.cpp:1162 > Since this patch was made, we've re-organized the hamburger menu a bit to > simplify it and hide not-commonly-used functionality. I think this feature > counts, since it's an alternative to the

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski updated this revision to Diff 69855. pdabrowski marked 5 inline comments as done. REPOSITORY R318 Dolphin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22594?vs=67500=69855 REVISION DETAIL https://phabricator.kde.org/D22594 AFFECTED FILES doc/index.docbook

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski marked 2 inline comments as done. pdabrowski added inline comments. INLINE COMMENTS > elvisangelaccio wrote in dolphinmainwindow.cpp:916-919 > I don't get this connection. Can you explain what's needed for? There is no other way to update the open_preferred_search_tool action

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski updated this revision to Diff 67500. pdabrowski marked 4 inline comments as done. REPOSITORY R318 Dolphin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22594?vs=66163=67500 REVISION DETAIL https://phabricator.kde.org/D22594 AFFECTED FILES doc/index.docbook

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski updated this revision to Diff 66163. pdabrowski added a comment. Rebased. REPOSITORY R318 Dolphin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22594?vs=66098=66163 REVISION DETAIL https://phabricator.kde.org/D22594 AFFECTED FILES doc/index.docbook

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added a comment. Done. REPOSITORY R318 Dolphin REVISION DETAIL https://phabricator.kde.org/D22594 To: pdabrowski, #dolphin, ngraham, elvisangelaccio Cc: pkloc, kfm-devel, kde-doc-english, iasensio, fprice, gennad, MrPepe, fbampaloukas, alexde, Codezela, feverfew, meven,

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski updated this revision to Diff 66098. pdabrowski marked 2 inline comments as done. REPOSITORY R318 Dolphin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22594?vs=62910=66098 REVISION DETAIL https://phabricator.kde.org/D22594 AFFECTED FILES doc/index.docbook

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski edited the summary of this revision. REPOSITORY R318 Dolphin REVISION DETAIL https://phabricator.kde.org/D22594 To: pdabrowski, #dolphin, ngraham, elvisangelaccio Cc: pkloc, kfm-devel, kde-doc-english, aprcela, vmarinescu, fprice, gennad, fbampaloukas, alexde, feverfew, meven,

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski updated this revision to Diff 62910. pdabrowski added a comment. Small fix for the "Open Preferred Search Tool" as a toolbar button. REPOSITORY R318 Dolphin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22594?vs=62212=62910 REVISION DETAIL

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski edited the summary of this revision. REPOSITORY R318 Dolphin REVISION DETAIL https://phabricator.kde.org/D22594 To: pdabrowski, #dolphin, ngraham, elvisangelaccio Cc: pkloc, kfm-devel, kde-doc-english, aprcela, vmarinescu, fprice, gennad, fbampaloukas, alexde, feverfew, meven,

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added a comment. In D22594#503270 , @elvisangelaccio wrote: > In D22594#498890 , @ngraham wrote: > > > As for the feature itself, for the past few years I'd been opposed to this, on the

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added a comment. In D22594#503270 , @elvisangelaccio wrote: > I'd like to remind that the menubar is hidden by default and the average user won't know about the `CTRL+Shift+F` shortcut. > So is this feature only aimed at advanced

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added a comment. I did some more testing, and everything seems to work fine: sudo su cd /usr/share/applications sleep 2 && mv org.kde.kfind.desktop org.kde.kfind.desktop_# no KFind now sleep 2 && mv org.kde.kfind.desktop_ org.kde.kfind.desktop# KFind

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added a comment. No problem. I'm glad it works :) REPOSITORY R318 Dolphin REVISION DETAIL https://phabricator.kde.org/D22594 To: pdabrowski, #dolphin, ngraham, elvisangelaccio Cc: kfm-devel, kde-doc-english, aprcela, fprice, gennad, fbampaloukas, alexde, feverfew, meven,

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added a comment. Hmm... Are you using the latest diff (https://phabricator.kde.org/D22594?id=62212)? REPOSITORY R318 Dolphin REVISION DETAIL https://phabricator.kde.org/D22594 To: pdabrowski, #dolphin, ngraham, elvisangelaccio Cc: kfm-devel, kde-doc-english, aprcela, fprice,

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added a comment. > The way you've implemented the menu item naming seems a bit fragile though True. It's really hard to update it reliably. KMoreTools* do not provide an easy way to get notified that the tools changed. Even current solution in "More Search Tools" menu

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski added a comment. > 2. For the keyboard shortcut let's use Alt instead of Shift. It is an alternate search, after all. I found `Ctrl+Shift+F` to be much more popular for "search", "advanced search", "search in all ...", "find in ...", "advanced find", etc.:

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski updated this revision to Diff 62212. pdabrowski edited the summary of this revision. pdabrowski added a comment. > 1. Instead of showing a generic text, how about making it actually say "Search with https://phabricator.kde.org/D22594?vs=62211=62212 REVISION DETAIL

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski updated this revision to Diff 62211. pdabrowski added a comment. > 3. Whenever you change anything in a .rc file, you need to bump the version number that appears at the top of the file. Done. REPOSITORY R318 Dolphin CHANGES SINCE LAST UPDATE

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski edited the summary of this revision. REPOSITORY R318 Dolphin REVISION DETAIL https://phabricator.kde.org/D22594 To: pdabrowski, #dolphin, ngraham Cc: kfm-devel, kde-doc-english, aprcela, fprice, gennad, fbampaloukas, alexde, feverfew, meven, spoorun, navarromorales, firef,

D22594: [Dolphin] Open Preferred Search Tool action

pdabrowski created this revision. pdabrowski added reviewers: Dolphin, ngraham. Herald added projects: Dolphin, Documentation. Herald added subscribers: kde-doc-english, kfm-devel. pdabrowski requested review of this revision. REVISION SUMMARY Added "Open Preferred Search Tool" action to Tools