pvuorela added a comment.
> 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? The main question was how should the three attributes play together. But another question: why isn't setBackgroundColor enough for the background color? REPOSITORY R8 Calligra REVISION DETAIL https://phabricator.kde.org/D25423 To: mphilippov, denexter, pvuorela, davidllewellynjones, dcaliste Cc: dcaliste, davidllewellynjones, denexter, Calligra-Devel-list, #calligra:_3.0, ognarb, cochise, vandenoever