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

Reply via email to