<table><tr><td style="">ppalaga updated this revision to Diff 11562.<br />ppalaga 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/D4550" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>New or noteworthy about the second iteration:</p>

<p>(1) It implements the proposal c) of Eike.</p>

<p>(2) There is is just one "Icon size" control in the configuration dialog, instead of two controls in the previous iteration.<br />
<a href="https://phabricator.kde.org/F2516001" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F2516001: pasted_file</a></p>

<p>(2) The "Icon size" combo and its label are visible only if the task manager is positioned vertically</p>

<p>(3) The "Icon size" preference is effective only for vertical task managers. It basically limits the size the icon is allowed to stretch to. See the changes in preferredMaxHeight() function.</p>

<p>(4) The "5 literal" meaning the number of 'm' characters whose joint width must be available in the task button label so that the button text is rendered at all was refactored to minimumMColumns() function in layout.js. minimumMColumns() returns 5 for horizontal layouts and 3 for vertical layouts.</p>

<p>(5) The present proposal does not change the behavior of horizontal task manager in any way.</p>

<p>(6) The present proposal reaches the goal of making it possible to show some text in a vertical task manager through choosing a smaller icon size in the configuration. OTOH, users interested in icon-only layouts may reach their aim by selecting larger icons or narrower vertical task manager.</p>

<p>Is the present proposal acceptable?</p></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/D4550?vs=11175&id=11562" rel="noreferrer">https://phabricator.kde.org/D4550?vs=11175&id=11562</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4550" rel="noreferrer">https://phabricator.kde.org/D4550</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/taskmanager/package/contents/code/layout.js<br />
applets/taskmanager/package/contents/config/main.xml<br />
applets/taskmanager/package/contents/ui/ConfigGeneral.qml<br />
applets/taskmanager/package/contents/ui/Task.qml</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>ppalaga, davidedmundson, hein, broulik<br /><strong>Cc: </strong>plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>