D12605: added 'add_to_places' action to file menu and as a shortcut

[Méven Car]

meven added a comment.


  D26590 <https://phabricator.kde.org/D26590> reminded me of this one.

INLINE COMMENTS

> dolphinmainwindow.cpp:1248
> +    addToPlaces->setText(i18nc("@action:inmenu Add current folder to places", "Add to Places"));
> +    actionCollection()->setDefaultShortcuts(addToPlaces, {Qt::CTRL + Qt::Key_D});
> +    connect(addToPlaces, &QAction::triggered, this, &DolphinMainWindow::addToPlaces);

I don't think we need a shortcut for this, and this one in particular will be used by the duplicate feature D8208 <https://phabricator.kde.org/D8208>

REVISION DETAIL
  https://phabricator.kde.org/D12605

To: rpetrov, #dolphin, ngraham
Cc: meven, markg, kfm-devel, ngraham, pberestov, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200113/f790793f/attachment.htm>