<table><tr><td style="">sharvey 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/D12321">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/D12321#249568" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12321#249568</a>, <a href="https://phabricator.kde.org/p/anemeth/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@anemeth</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>(effective screencast)</p></div>
</blockquote>

<p>+1 from me, and you beat me to it. 😃  I was just looking through the code when Thunderbird chimed that I had new mail.</p>

<p>For the sake of clarification, the reason this works and my attempt in <a href="https://phabricator.kde.org/D12094" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12094</a> did not, is because this <tt style="background: #ebebeb; font-size: 13px;">KToggleAction</tt> is, for lack of a better term, unique. There's no second, competing <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Preview</span></span></span> action to confuse the issue and be duplicated in <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Toolbar Preferences</span></span></span>. You <strong>can</strong> bend a <tt style="background: #ebebeb; font-size: 13px;">KToggleAction</tt> to your will; that's pretty straightforward. As you can see in <a href="https://phabricator.kde.org/p/anemeth/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@anemeth</a>'s patch, toggling it on or off (or disabling it entirely) is just a matter of setting the right flags, under the desired conditions. My work in <a href="https://phabricator.kde.org/D12094" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12094</a> does much of the same, but because we were trying to implement two closely-related <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Find</span></span></span> actions, we needed two action entries. Here, we only need one. And because there's just one to worry about, Alex can make it do any trick he wants, without worrying about interfering with or getting confused with another action item.</p>

<p>I think this effect is subtle but very classy and polished. We show previews at higher sizes by default, but they can be toggled on or off at any time. The <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Preview</span></span></span> button is disabled at tiny sizes, because it makes no sense otherwise. Looks good.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12321">https://phabricator.kde.org/D12321</a></div></div><br /><div><strong>To: </strong>anemeth, VDG, Frameworks, ngraham, rkflx, Dolphin<br /><strong>Cc: </strong>sharvey, rkflx, ngraham, Frameworks, michaelh, bruns<br /></div>