[Differential] [Accepted] D2227: Fix testing for encrypted archives

elvisangelaccio (Elvis Angelaccio) noreply at phabricator.kde.org
Tue Jul 19 19:38:08 UTC 2016


elvisangelaccio accepted this revision.
elvisangelaccio added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> part.cpp:481
>          bool supportsTesting = ArchiveFormat::fromMetadata(m_model->archive()->mimeType(), metadata).supportsTesting();
>          m_testArchiveAction->setEnabled(!isBusy() &&
> +                                        supportsTesting &&

Also add a tooltip explaining the reason for disabling, like we did for "Add Files"

REPOSITORY
  rARK Ark

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: rthomsen, elvisangelaccio
Cc: kde-utils-devel, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20160719/c4d39693/attachment.html>


More information about the Kde-utils-devel mailing list