<table><tr><td style="">mphilippov 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/D25423">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/D25423#589481" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25423#589481</a>, <a href="https://phabricator.kde.org/p/pvuorela/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@pvuorela</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Do you mean this "if ()" part of code? If there is no style "graphic" and inner "if" will not execute. Do you mean it'll be better to handle it in this part of code...or what?</p></blockquote>

<p>The main question was how should the three attributes play together. But another question: why isn't setBackgroundColor enough for the background color?</p></div>
</blockquote>

<p>Ok. 1. There is a bug in calligra with alpla-channel. So with style->setBackgroundColor(QColor()); all transparent cells will be painted with black color.</p>

<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">This part of code isn't handled in all spreadsheet formats either ods or xls or xlsx. Setting brush needs us to paint all the cell with the required color</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R8 Calligra</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25423">https://phabricator.kde.org/D25423</a></div></div><br /><div><strong>To: </strong>mphilippov, denexter, pvuorela, davidllewellynjones, dcaliste<br /><strong>Cc: </strong>dcaliste, davidllewellynjones, denexter, Calligra-Devel-list, Calligra: 3.0, ognarb, cochise, vandenoever<br /></div>