D24247: Expose full set of script execution options in settings window

[Elvis Angelaccio]

elvisangelaccio accepted this revision.
elvisangelaccio added a comment.
This revision is now accepted and ready to land.


  While the Confirmation tab is not perfect, I don't think that moving this setting to the Behavior or Navigation tabs would be much better.

INLINE COMMENTS

> confirmationssettingspage.cpp:134
>      KConfigGroup scriptExecutionGroup(kioConfig, "Executable scripts");
> -    if (m_confirmScriptExecution->isChecked()) {
> +    int index = m_confirmScriptExecution->currentIndex();
> +    switch (index) {

`const`

REPOSITORY
  R318 Dolphin

BRANCH
  multi-state-script-execution-control (branched from master)

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

To: ngraham, elvisangelaccio, #dolphin, #vdg
Cc: kfm-devel, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190929/ecfe38c6/attachment.htm>