elvisangelaccio requested changes to this revision.
elvisangelaccio added a comment.
This revision now requires changes to proceed.
I still think that the rename from "Search" to "Find" should be done either
upstream/globally or not at all.
Anyway, it's unrelated from this commit, which is just about moving to
placeholders everywhere (which I agree with).
INLINE COMMENTS
> dolphinmainwindow.cpp:931-933
> + // Overwrite Find action to Search action
> + QAction *search =
> ac->action(KStandardAction::name(KStandardAction::Find));
> + search->setText(i18n("Search..."));
We shouldn't rename a standard action. From System Settings -> Global Shortcuts
it will still be named "Find..."
REPOSITORY
R318 Dolphin
REVISION DETAIL
https://phabricator.kde.org/D19770
To: ognarb, #dolphin, #vdg, ngraham, elvisangelaccio
Cc: GB_2, kde-doc-english, elvisangelaccio, ngraham, yurchor, kfm-devel,
gennad, alexde, feverfew, meven, spoorun, navarromorales, firef, andrebarros,
skadinna, emmanuelp, mikesomov