<table><tr><td style="">pvuorela 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 #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></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>