<table><tr><td style="">davidedmundson 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/D10118" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>it is not clear if this code is functional at this moment or not</p></blockquote>

<p>I'm -1 for disabling if the change is based on a false premise that it doesn't work.</p>

<p>Is it something you've already lost several hours into trying to fix? If so this proposal has a lot more merit.</p>

<p>If it is to be disabled, commenting something out when it's already in an if() block is not a good approach.<br />
Also any hacks always always need a code comment.</p>

<hr class="remarkup-hr" />

<p>FWIW:  ktouch has it's own layout parser / preview code.   It does update between UK and french and move the preview with the letters about, but switching to arabic returns an error, whereas the kcm preview works fine. Probably would need layer support too.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10118" rel="noreferrer">https://phabricator.kde.org/D10118</a></div></div><br /><div><strong>To: </strong>bshah, Plasma<br /><strong>Cc: </strong>sitter, ngraham, hein, broulik, graesslin, anthonyfieroni, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>