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

[Mikhail Philippov]

mphilippov added a comment.


  In D25423#586953 <https://phabricator.kde.org/D25423#586953>, @pvuorela wrote:
  
  > Not sure about all the details but two things look suspicious here:
  >
  > 1. Above this there's handling for transparent and now the handling isn't symmetric for background brush.
  > 2. There's some handling on the file for KoXmlNs::draw / style-name & KoXmlNs::calligra / fill-style-name which do background brush setting. What's the relation between these three?
  
  
  
  
  1. I see it. Ok
  2. 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.

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/81b61de0/attachment.htm>