Review Request 118939: Add an option that makes it possible to open scripts and desktop files, rather than execute them

[Frank Reininghaus]

> On June 25, 2014, 7:12 p.m., Christoph Feck wrote:
> > dolphin/src/settings/general/behaviorsettingspage.cpp, line 74
> > <https://git.reviewboard.kde.org/r/118939/diff/1/?file=284599#file284599line74>
> >
> >     I think the two verbs (clicking / executing) are backwards here.
> >     
> >     Maybe "Execute scripts (etc.) when clicking them"?

Yes, you're right, that's probably better.

In any case, I tested it on another computer in the mean time, and due to different font settings, the dialog is too narrow here to show the rather long check box text. This is probably straightforward to fix, but I think it's better not to rush this into master right now, just before the string freeze at midnight UTC.

Sorry for the noise - one really should take more time for changes in the GUI. I'll try to implement a proper "What do you want to do with this script" dialog for the next release.


- Frank


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


On June 25, 2014, 4:03 p.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118939/
> -----------------------------------------------------------
> 
> (Updated June 25, 2014, 4:03 p.m.)
> 
> 
> Review request for Dolphin and David Faure.
> 
> 
> Bugs: 275405
>     http://bugs.kde.org/show_bug.cgi?id=275405
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> This patch adds an option to the settings dialog that makes it possible to open scripts and desktop files, rather than execute them. It is a slightly modified version of David's patch from https://bugs.kde.org/show_bug.cgi?id=275405#c12
> 
> These are the modifications:
> 
> 1. Changed the checkbox text from "Left click..." to the more general "Clicking..." because the user can revert left and right mouse buttons in the system settings.
> 
> 2. Changed the type of the option from bool to an enum. The motivation is that this makes it easier to extend this feature: when clicking an executable file, we could show a dialog that offers "Execute", "Open in {insert default application here}", maybe "Execute in an external terminal", "Cancel", and a "Do not ask again" checkbox. Note that this makes the code that deals with this setting rather verbose, unfortunately. I could not come up with a less verbose solution that still makes it clear at first sight (both in the code and the config files) what the setting is about.
> 
> I did try to implement such a dialog using convenience functions/classes like KMessageBox/QDialogButtonBox, but came to the conclusion that this is less convenient in this special case than I thought. Writing a custom dialog would probably be better. It's probably not hard, but not trivial either, and we might want to discuss with the usability team if an "Execute in Terminal" or "Execute in Terminal Panel" button makes sense.
> 
> Since we are running out of time now, I thought that getting David's patch in before the 4.14 feature and string freeze is better than nothing.
> 
> Note that "Execute" is still the default. This means that users who got used to executing scripts by clicking them will not be surprised by the change, but those who prefer to open and edit scripts have to change the setting manually. When we implement a dialog in the future, the default value should be "AlwaysAsk".
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinapplication.cpp 8e83a85 
>   dolphin/src/dolphinmainwindow.cpp c60951d 
>   dolphin/src/dolphinviewcontainer.cpp 57452b9 
>   dolphin/src/settings/dolphin_generalsettings.kcfg 849a9c7 
>   dolphin/src/settings/general/behaviorsettingspage.h 7a9c2f0 
>   dolphin/src/settings/general/behaviorsettingspage.cpp cbbde1d 
> 
> Diff: https://git.reviewboard.kde.org/r/118939/diff/
> 
> 
> Testing
> -------
> 
> Works as expected for me, i.e., scripts are executed or opened in an editor, depending on the setting.
> 
> 
> File Attachments
> ----------------
> 
> Modified settings dialog
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/06/25/6f58e1a0-9425-42c9-9378-1327594f3733__DolphinSettingsDialog-ExecuteScripts.png
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

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