Review Request 130028: Add a dedicated 'Copy To Stash' context menu button for the upcoming stash:/ ioslave

[Boudhayan Gupta]

> On March 19, 2017, 4:48 a.m., Boudhayan Gupta wrote:
> > Could you also post a couple of screenshots detailing the UI changes?
> 
> arnav dhamija wrote:
>     Yes, I have posted an instance of the context menu with the copy to stash action.

What about just "Stash" instead of "Copy to Stash"? I'd change the order of the actions too, so that Stash came above Move to Trash.


- Boudhayan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/130028/#review102899
-----------------------------------------------------------


On March 19, 2017, 3:20 p.m., arnav dhamija wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/130028/
> -----------------------------------------------------------
> 
> (Updated March 19, 2017, 3:20 p.m.)
> 
> 
> Review request for Dolphin, Boudhayan Gupta, David Faure, and Emmanuel Pescosta.
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> This patch adds a dedicated Copy to Stash button in Dolphin's context menu when items are selected in a group. Triggering this action adds the selected items to the clipboard and pastes it on the stash:/ ioslave.
> 
> I have encountered some problems when writing this patch. For one, though the QAction appears in the context menu as expected, it does not show up in the Menubar's File Menu though it is added under that in the dolphinui.rc file. Apart from this, the patch works fine in manual testing.
> 
> 
> Diffs
> -----
> 
>   src/dolphincontextmenu.cpp 890494f 
>   src/dolphinmainwindow.h d741eb2 
>   src/dolphinmainwindow.cpp de8601d 
>   src/dolphinpart.rc 725320b 
>   src/dolphinui.rc c446656 
>   src/views/dolphinview.h 5e69fd3 
>   src/views/dolphinview.cpp 63f6252 
> 
> Diff: https://git.reviewboard.kde.org/r/130028/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> Context menu
>   https://git.reviewboard.kde.org/media/uploaded/files/2017/03/19/085a8a51-86a7-4647-b5ae-fc8f0f599026__Screenshot_20170319_095338.png
> 
> 
> Thanks,
> 
> arnav dhamija
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20170319/5cb6164f/attachment.htm>