<table><tr><td style="">nicolasfella added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D5876" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I implemented your suggestion with a little change. I separated between Plugins that could not be loaded because of lack of permission and Plugins that could not be loaded because of other errors and have separate lists. This way, it is more clear to the user. Maybe we should add an option to hide the list in case the user really doesn't want to grant a permission. This separation also makes it easier for Plugin developers. They only have to list the desired permissions and the Plugin class takes care of requesting them. This way, getErrorDialog() is only used for real errors and  getPermissionExplanationDialog() for Permissions. Plugins can provide detailed explanation why permissions are necessary by setting permissionExplanation, otherwise a generic explanation is used.</p>

<p>What do you think?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5876" rel="noreferrer">https://phabricator.kde.org/D5876</a></div></div><br /><div><strong>To: </strong>nicolasfella, KDE Connect<br /><strong>Cc: </strong>albertvaka, aboudhar, seebauer, progwolff, MayeulC, menasshock, ach, apol, hkaelberer<br /></div>