D10960: Ask for confirmation when Closing Dolphin windows with a terminal panel running a program

[Elvis Angelaccio]

elvisangelaccio added inline comments.

INLINE COMMENTS

> rkflx wrote in dolphinmainwindow.cpp:401
> What you say applies for regular dialogs. However, here we are in a //confirmation// dialog, where we already saved the user from a destructive action. Now we should present the sensible choice by default, which IMO is quitting. Or do you want //another// safety net?
> 
> Anyway, do what you want and talk to #dolphin <https://phabricator.kde.org/tag/dolphin/>'s maintainers, I don't have time to discuss this right now. Maybe we'll change it back later when this is unified across all apps.

> What you say applies for regular dialogs. However, here we are in a confirmation dialog, where we already saved the user from a destructive action. Now we should present the sensible choice by default, which IMO is quitting.

+1

REPOSITORY
  R318 Dolphin

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

To: rominf, #dolphin, ngraham, elvisangelaccio
Cc: elvisangelaccio, markg, ngraham, rkflx, broulik, #dolphin, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180313/ebdc4269/attachment.htm>