D15609: Add "Create New" menu to "Control" button

Elvis Angelaccio noreply at phabricator.kde.org
Thu Oct 4 22:12:43 BST 2018


elvisangelaccio added a comment.


  Btw we broke the "Create New" button on read-only folders. Time to write an unit test for that...

INLINE COMMENTS

> dolphinmainwindow.cpp:923
> +    // Add "Create New" menu
> +    QMenu* createNewMenu = m_newFileMenu->menu();
> +    menu->addMenu(createNewMenu);

Why here though? "Create New" belongs to the File menu, so it should probably be the very first thing in the Control menu.

REPOSITORY
  R318 Dolphin

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

To: ngraham, #dolphin, broulik
Cc: elvisangelaccio, kfm-devel, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20181004/ef7d9c9a/attachment.htm>


More information about the kfm-devel mailing list