<table><tr><td style="">ngraham 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/D22627">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D22627#500454" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D22627#500454</a>, <a href="https://phabricator.kde.org/p/filipf/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@filipf</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Yeah an empty entry is bad UI but it works for the code since it ends up just grabbing the text from the combobox and writing it to the config file :P ... it was a quick solution I'll admit.</p></div>
</blockquote>

<p>Instead of just passing the string in the combobox directly to whatever writes the value to the file, you'd need to translate the value first.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The feature request was this, a much more straightforward idea: <a href="https://forum.kde.org/viewtopic.php?f=285&t=161536&p=420073#p420073" class="remarkup-link" target="_blank" rel="noreferrer">https://forum.kde.org/viewtopic.php?f=285&t=161536&p=420073#p420073</a></p>

<p>But having that checkbox seemed too specific and it seemed like having a general input method chooser might be more useful. What do you think?</p></blockquote>

<p>The generic solution you've implemented here makes sense if the features you're exposing actually work in SDDM with all themes (in which case, we need to improve the UI here). If not, maybe do the simple checkbox as suggested in that forum post.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R123 SDDM Configuration Panel (KCM)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22627">https://phabricator.kde.org/D22627</a></div></div><br /><div><strong>To: </strong>filipf, Plasma, VDG, ngraham, davidedmundson<br /><strong>Cc: </strong>plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>