<table><tr><td style="">subdiff 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/D5250" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Script looks very nice! I will test run it tomorrow. One thing we have to keep in mind when adding additional cursor sizes is the automatic (resolution dependent) cursor size mode. Adding many sizes with small differences could make the chosen size ambiguous or the algorithm to select the size become inefficient. But I would start from the premise that this is not case. Let's make sure though by looking into it shortly as well.</p>

<p>Regarding your comments in your bug report: After my initial comment I hadn't looked into it again till today when I noticed your Diff here on Phab, so that's why I haven't responded. The Breeze repo is also currently without maintainer, so that's why there were not many other reactions. Thanks for the later responses to Dominik and Andreas therefore!</p></div></div><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/D5250#inline-21629" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">build.sh:1</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; "><span class="ch">#!/bin/bash</span>
</div><div style="padding: 0 8px; margin: 0 4px; "><span style="color: #74777d"># Open initial output.</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Leave this file at mode 644.</p></div></div><br /><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/D5250#inline-21626" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">build.sh:14</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; "><span style="color: #001294">ALIASES</span><span style="color: #aa2211">=</span><span style="color: #766510">"src/cursorList"</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #001294">SIZES</span><span style="color: #aa2211">=</span><span style="color: #766510">"24 32 40 48 64 96"</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">What about the 36px size? It is scaling factor 1.5, therefore quite common.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R31 Breeze</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5250" rel="noreferrer">https://phabricator.kde.org/D5250</a></div></div><br /><div><strong>To: </strong>jamesl<br /><strong>Cc: </strong>subdiff, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>