<table><tr><td style="">xuetianweng requested changes to this revision.<br />xuetianweng 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/D22535">View Revision</a></tr></table><br /><div><div><p>This change doesn't serve any real functionality of kimpanel. It doesn't show the current input method in tray, doesn't hide or show up. And kimpanel usually consist multiple icons. While it's doubt whether we need that much for certain input method (currently user can blacklist some by hand), but being able to show some important ones should not be removed. (on windows, there're two icon for chinese case).</p>

<p>Also, simply move the existing implementation into expanded tray is not acceptable, the layout is not designed for the expanded tray. There is much more space in the tray so even if we want to keep the similar ui large icon IMHO is prefered.</p>

<p>As for configure, there's already a dbus call in kimpanel for launch the configuration, not sure if want to use that to launch kcm.</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/D22535">https://phabricator.kde.org/D22535</a></div></div><br /><div><strong>To: </strong>guoyunhe, ngraham, VDG, Plasma, xuetianweng<br /><strong>Cc: </strong>xuetianweng, gpark, hein, mart, ngraham, apol, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>