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

<p>Hi! :)</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Second: I agree that the current code is bug prone, but your foreseen change is not satisfactory either. <br />
 The idea with the current code was to try create the shadows only "once" and if it fails for some reason, leave it failed for all widgets for which it is requested. <br />
 In your modified code however, if the creation of shadowtiles fails once, it keeps being called over and over again for each new widget. <br />
 I think this should be avoided.</p></blockquote>

<p>I've tried to preserve old behavior. The current code is always trying to create shadows.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>One solution is possibly to call shadowTiles once and only once, in breezeStyle::loadConfiguration, and just test whether it is valid whenever used. <br />
 what do you think ?</p></blockquote>

<p>Yes, that's a good idea. How about calling <tt style="background: #ebebeb; font-size: 13px;">shadowTiles</tt> in <tt style="background: #ebebeb; font-size: 13px;">ShadowHelper::reset()</tt>?</p></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/D11533">https://phabricator.kde.org/D11533</a></div></div><br /><div><strong>To: </strong>zzag, Breeze, hpereiradacosta<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>