D8206: Add Duplicate feature

[Mark Gaiser]

markg added inline comments.

INLINE COMMENTS

> dolphinviewactionhandler.cpp:134
> +    duplicateAction->setIcon(QIcon::fromTheme(QStringLiteral("edit-duplicate")));
> +    m_actionCollection->setDefaultShortcut(duplicateAction, Qt::CTRL | Qt::Key_D);
> +    duplicateAction->setEnabled(false);

No..  Not another app with another binding for CTRL+D.. :(

In Chrome CTRL+D is bookmark.
In Konsole CTRL+D is exits the current session
..

I'd prefer to just not have a key binding for it by default. The users can - if they want - add a binding themselves in the shortcut settings. Not sure if you need to do something for it to appear in that settings page though.

REPOSITORY
  R318 Dolphin

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

To: ngraham, #dolphin, #kde_applications, dfaure, elvisangelaccio
Cc: markg, fazevedo, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180422/db6f49e0/attachment.htm>