Fixed dolphin properties dialog shortcut
Review Request #106753 - Created Oct. 7, 2012 and submitted
Fixes Bug 307603 - Alt+Enter should open item properties dialog as with Alt+Return, reported by Syam. Both Alt+Return and Alt+Enter have now been set as the default shortcuts for the properties dialog in dolphin.
Tested and confirmed working with KDE SC 4.9.2 on openSUSE 12.2.
Thanks for the patch! Looks good and can go into the 4.9 branch :-) Just one little idea to make it a bit more consistent with the other actions: could you change it slightly to set the multiple shortcuts like it's done for the showHiddenFiles action? With your current approach, the KShortcut which you create with 'primary' and 'alternate' shortcuts is casted to a QList<QKeySequence> implicitly. If you create the QList directly, you save the cast. It's just a small issue and mostly a matter of taste, but I think that the code will remain more readable if we do it the same way for all actions. Feel free to push this to the 4.9 branch. If you don't have a git account, just let me know and I'll do it for you. Thanks again for your help!