<table><tr><td style="">mglb added inline comments.
</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/D20816">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D20816#inline-122992">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ngraham</span> wrote in <span style="color: #4b4d51; font-weight: bold;">MainWindow.cpp:742</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;"><tt style="background: #ebebeb; font-size: 13px;">preferences-system-users</tt> isn't terrible, but maybe if the two were combined somehow it would be better. Like two terminal windows, each with a user icon inside it, with one in front of the other.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;"><tt style="background: #ebebeb; font-size: 13px;">utilities-terminal</tt> is already used by "General" page. I think the icon should represent themes concept, as those profiles are not related to multiple users (what <tt style="background: #ebebeb; font-size: 13px;">preferences-system-users</tt> icon suggests) and are mostly about look.<br />
<a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> maybe just one rectangle with a white prompt on blue background and second rectangle with green prompt on black background (colors are just an example). Or just one rectangle divided diagonally, one half uses one theme and the other half another theme. Might be made more generic by using some lines (representing text instead of prompt).</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20816">https://phabricator.kde.org/D20816</a></div></div><br /><div><strong>To: </strong>mglb, Konsole, VDG, ngraham<br /><strong>Cc: </strong>GB_2, tcanabrava, ngraham, hindenburg, VDG, konsole-devel, Konsole, cblack, arvidhansson, ian, jguidon, hannahk, Ghost6, jraleigh, fbampaloukas, squeakypancakes, alexde, IohannesPetros, trickyricky26, thsurrel, mglb, crozbo, ndavis, firef, skadinna, maximilianocuria, aaronhoneycutt, mbohlender<br /></div>