rade added a comment.
In D11731#236223 <https://phabricator.kde.org/D11731#236223>, @martinkostolny wrote: > Sorry about that, I should have waited a bit longer. So how about this? > > // removing action from previous menu is necessary > // otherwise it wouldn't be displayed in currently populating menu > if (_mainBookmarkPopup) { > _mainBookmarkPopup->removeAction(_quickSearchAction); > } > Looks good, although I think it should be "won't" not "wouldn't" as it is an action that will not happen in the future not an action that has not happened in the past. REPOSITORY R167 Krusader REVISION DETAIL https://phabricator.kde.org/D11731 To: martinkostolny, #krusader, rade, nmel Cc: kde-doc-english, nmel, mjanczara, miroslavm, janlepper, abika, martinkostolny, asensi, gengisdave, andreaska
