Review Request: Remove additional directories from shortcuts scheme export path

[Michael Pyne]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104981/#review14150
-----------------------------------------------------------

Ship it!


In response to the ping, I like the concept of the bugfix and the patch appears sound. I haven't tested it though but if no one has complained in patch review yet then it's probably safe to commit. ;)

- Michael Pyne


On May 24, 2012, 10:01 p.m., Burkhard Lück wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104981/
> -----------------------------------------------------------
> 
> (Updated May 24, 2012, 10:01 p.m.)
> 
> 
> Review request for kdelibs and Andreas Pakulat.
> 
> 
> Description
> -------
> 
> The Configure Shortcuts dialog has an Action to export a scheme 
> (Details->More Actions->Export Scheme)
> 
> Using this action the user is asked for a export location and has to select a directory.
> Then the current scheme 'schemename' in application 'appname' is exported to a file 
> named appnameschemenameshortcuts.rc.
> 
> But this file is not saved in the selected directory as Joe User would expect, but in 
> shortcuts/share/apps/appname/ below the selected folder.
> 
> This patch removes the additional directories shortcuts/share/apps/appname/ from 
> the export path to make it easier for the user to find the scheme file and move/copy 
> it via command line (there is no GUI to import a scheme).
> 
> 
> Diffs
> -----
> 
>   kdeui/dialogs/kshortcutschemeseditor.cpp 34a485a 
> 
> Diff: http://git.reviewboard.kde.org/r/104981/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120524/850f723a/attachment.htm>