diff options
author | Justin Luth <justin_luth@sil.org> | 2018-08-18 11:41:35 +0300 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2018-08-31 09:54:06 +0200 |
commit | 3d81af669ea0886fb16282d29048f08025ace344 (patch) | |
tree | 7dac7ec48d28ac99c7f3ad59d4f8b42c4a42d8f4 /sw/source/filter/ww8 | |
parent | 4235206c340f0a8e8cc38b18403a7f0fbee04f7a (diff) |
tdf#116436 rtf export: add missing table background fill
MS formats only have support for Table and Cell fill.
Interestingly, MS Word doesn't let the cells inherit from
the Table fill setting,
so that value also needs to be written out into every cell.
Since Word apparently ignores the table color, I didn't
bother trying to search out how to export that in RTF
format. (I did add it to DOCX since it was so easy to find.)
Change-Id: I7fe7723fb1603bc60608762aea79ff3b8122e073
Reviewed-on: https://gerrit.libreoffice.org/59282
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
(cherry picked from commit 1a4bc964f15003c6aaec84989e3e47501b7f72df)
Reviewed-on: https://gerrit.libreoffice.org/59667
Diffstat (limited to 'sw/source/filter/ww8')
-rw-r--r-- | sw/source/filter/ww8/rtfattributeoutput.cxx | 28 |
1 files changed, 23 insertions, 5 deletions
diff --git a/sw/source/filter/ww8/rtfattributeoutput.cxx b/sw/source/filter/ww8/rtfattributeoutput.cxx index 99b283f92e9c..edd2830bc6e9 100644 --- a/sw/source/filter/ww8/rtfattributeoutput.cxx +++ b/sw/source/filter/ww8/rtfattributeoutput.cxx @@ -727,6 +727,21 @@ void RtfAttributeOutput::TableDefaultBorders( void RtfAttributeOutput::TableBackgrounds( ww8::WW8TableNodeInfoInner::Pointer_t pTableTextNodeInfoInner) { + const SwTable* pTable = pTableTextNodeInfoInner->getTable(); + const SwTableBox* pTableBox = pTableTextNodeInfoInner->getTableBox(); + const SwTableLine* pTableLine = pTableBox->GetUpper(); + + Color aColor = COL_AUTO; + const SvxBrushItem* pTableColorProp + = pTable->GetFrameFormat()->GetAttrSet().GetItem<SvxBrushItem>(RES_BACKGROUND); + if (pTableColorProp) + aColor = pTableColorProp->GetColor(); + + const SvxBrushItem* pRowColorProp + = pTableLine->GetFrameFormat()->GetAttrSet().GetItem<SvxBrushItem>(RES_BACKGROUND); + if (pRowColorProp && pRowColorProp->GetColor() != COL_AUTO) + aColor = pRowColorProp->GetColor(); + const SwWriteTableRows& aRows = m_pTableWrt->GetRows(); SwWriteTableRow* pRow = aRows[pTableTextNodeInfoInner->getRow()]; const SwWriteTableCell* const pCell @@ -736,11 +751,14 @@ void RtfAttributeOutput::TableBackgrounds( if (pCellFormat->GetAttrSet().HasItem(RES_BACKGROUND, &pItem)) { auto& rBack = static_cast<const SvxBrushItem&>(*pItem); - if (!rBack.GetColor().GetTransparency()) - { - m_aRowDefs.append(OOO_STRING_SVTOOLS_RTF_CLCBPAT); - m_aRowDefs.append(static_cast<sal_Int32>(m_rExport.GetColor(rBack.GetColor()))); - } + if (rBack.GetColor() != COL_AUTO) + aColor = rBack.GetColor(); + } + + if (!aColor.GetTransparency()) + { + m_aRowDefs.append(OOO_STRING_SVTOOLS_RTF_CLCBPAT); + m_aRowDefs.append(static_cast<sal_Int32>(m_rExport.GetColor(aColor))); } } |