D25423: Fill table cells with color for ODS and XLSX files

[Mikhail Philippov]

mphilippov added a comment.


  In D25423#589430 <https://phabricator.kde.org/D25423#589430>, @pvuorela wrote:
  
  > > loadTableCellProperties() is a method for applying styles for table cells, i.e. cell background. There is no handling for cell background in the place you mentioned.
  >
  > I'm talking about handling in that particular method. It will already set background brush in that method later. Why is that not enough?
  
  
  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?

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20200107/2a06d611/attachment.htm>