<table><tr><td style="">fvogt created this revision.<br />Restricted Application added a project: Plasma.
</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/D5227" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The show password button only makes sense with a virtual keyboard<br />
on touchscreens. On desktop PCs it's not only not useful, but it<br />
can also be used to show (partially) typed in passwords.<br />
It also makes it possible to display clipboard entries in plaintext,<br />
which is why the clipboard is cleared when the screen locks.<br />
By adding a configuration option, the user can decide whether the<br />
show password functionality is useful to him.<br />
The default is set to show the button to now change the default<br />
behaviour.</p>

<p>Once the clipboard issue is fixed properly in all cases, the configuration<br />
option can be reevaluated, but currently it is in my opinion the best<br />
way to fix both issues at once.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Ran kscreenlocker_greet --testing with and without the option<br />
changed in the kcm. Worked as expected.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R133 KScreenLocker</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5227" rel="noreferrer">https://phabricator.kde.org/D5227</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>greeter/greeterapp.cpp<br />
kcfg/kscreenlockersettings.kcfg<br />
kcm/kcm.ui</div></div></div><br /><div><strong>To: </strong>fvogt, Plasma, graesslin<br /><strong>Cc: </strong>plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>