<table><tr><td style="">cryptodude 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/D8787" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Updated to fix the issues.</p>

<p>The leading space was intended, indeed. It makes things look a bit nicer.</p>

<p>I also clarified the description, and as to the question if its still valuable.<br />
The main change is that the original code was intended to not close the password dialog (as Ivan stated on bugs.kde) when an error occurred, but it does and people got confused when nothing happened. This patch make  the intended behaviour actually work.</p>

<p>It may be nice to make the errors from the backends more legible and understandable to end-users, but I think that is a separate issue.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R845 Plasma Vault</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8787" rel="noreferrer">https://phabricator.kde.org/D8787</a></div></div><br /><div><strong>To: </strong>cryptodude, Plasma, ivan<br /><strong>Cc: </strong>ngraham, cfeck, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>