D19847: Delete/Trash confirmation dialogue: Fix misleading title and make consistent

[Jonathan Marten]

marten created this revision.
marten added reviewers: Frameworks, dfaure.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
marten requested review of this revision.

REVISION SUMMARY
  The misleading title that brought this dialogue to my attention is where there is only one selected item to move to the trash.  In this case the question wording is correct but the message box title says "Delete Permanently?".  This is the code for "case Trash" and "if (prettyList.count() == 1)" in KIO::JobUiDelegate::askDeleteConfirmation().
  
  This should be corrected to say "Move to Trash" as in the other conditional below.
  
  There are some other anomalies which are also corrected in this diff:
  
  a)  The "Delete Permanently" message box title has a question mark, but "Move to Trash" does not.  I can't find a HIG rule for which of those is correct, but for consistency and in accordance with "Do not use the title to explain what to do in the dialog – that’s the purpose of the main instruction" the question marks are removed.
  
  b) Capitalisation of "Trash" made consistent in the question for trash of multiple items.

TEST PLAN
  Built KIO with these changes, checked all combinations of delete/trash operations in Dolphin and verified that the correct message box title and question is shown.

REPOSITORY
  R241 KIO

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

AFFECTED FILES
  src/widgets/jobuidelegate.cpp

To: marten, #frameworks, dfaure
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190317/b7a56557/attachment.html>