<table><tr><td style="">rkflx updated this revision to Diff 35808.<br />rkflx edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-u3z43i7xtgmjanm/">(Show Details)</a><br />rkflx edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-kocfwabo6ro7xwo/">(Show Details)</a><br />rkflx 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/D13390">View Revision</a></tr></table><br /><div><div><p>Okay, great. Thanks for investigating! I now changed to what you proposed, so we have something for 5.13.1.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>As that property is not available from QML, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">we reset the text colour to<br />
solve the problem. It's still not using the proper frame colour, though,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">we use an alternative</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">but that does not seem like too much of an issue.<br />
<br />
Alternatively we could use a `Frame` and a `Label`,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">approach by setting the state to `readOnly`.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">but this approach</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">To also indicate</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">has even more problems, e.g.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">visually  that nothing can be typed into the textedit,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the frame does not respect the rounded</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">we add</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">border mandated by Breeze, and the horizontal alignment of each row</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">`Kirigami.Theme.inherit: true` which results in changing the</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">would need to be adapted</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">background from white to grey</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>After: {F589<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">4424}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">6658}</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Ideal</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">How it</span> loo<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">k (</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">ked like in </span>Plasma 5.12<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">)</span>: {F5894423}</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D13390?vs=35719&id=35808">https://phabricator.kde.org/D13390?vs=35719&id=35808</a></div></div><br /><div><strong>BRANCH</strong><div><div>fonts-kcm-readability (branched from Plasma/5.13)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13390">https://phabricator.kde.org/D13390</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/fonts/package/contents/ui/FontWidget.qml</div></div></div><br /><div><strong>To: </strong>rkflx<br /><strong>Cc: </strong>mart, broulik, Zren, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>