related tdf#99602 writerfilter TODO: subscript - use ParaStyle fontsize
The existing code handled two situations:
1.) if both position(escapement) and fontsize set via direct properties
2.) if fontsize came from the document default properties.
That misses both paragraph style fontsize (very common)
and character style fontsize. This patch adds checking for
the paragraph style's fontsize.
Change-Id: I25312cd4544cfd1be09b21c96a270cb05e4f5b8f
Reviewed-on: https://gerrit.libreoffice.org/80179
Reviewed-by: Justin Luth <justin_luth@sil.org>
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport10.cxx b/sw/qa/extras/ooxmlexport/ooxmlexport10.cxx
index 911c343..17b91ec 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport10.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport10.cxx
@@ -213,8 +213,10 @@ DECLARE_OOXMLEXPORT_TEST(testFontEsc, "test_tdf120412.docx")
return;
// don't lose the run with superscript formatting
assertXPath(pXmlDoc, "/w:document/w:body/w:p/w:r", 2);
// raising is greater than 100%
assertXPath(pXmlDoc, "/w:document/w:body/w:p/w:r[2]/w:rPr/w:position","val", "24");
// Superscript should be raised by 100% (11pt). Was 110% (12pt)
// calculated using docDefault with fontsize 10pt (note only w:szCs defined as 11pt, not w:sz)
// instead of inherited normal paraStyle fontsize 11pt (related to tdf#99602)
assertXPath(pXmlDoc, "/w:document/w:body/w:p/w:r[2]/w:rPr/w:position","val", "22");
}
DECLARE_OOXMLEXPORT_TEST(testMceWpg, "mce-wpg.docx")
diff --git a/writerfilter/source/dmapper/DomainMapper.cxx b/writerfilter/source/dmapper/DomainMapper.cxx
index b461367..3d7216e 100644
--- a/writerfilter/source/dmapper/DomainMapper.cxx
+++ b/writerfilter/source/dmapper/DomainMapper.cxx
@@ -1699,9 +1699,6 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, const PropertyMapPtr& rContext )
xCharStyle->setPropertyValue(getPropertyName(PROP_CHAR_HEIGHT), aVal);
}
}
// Make sure char sizes defined in the stylesheets don't affect char props from direct formatting.
if (!IsStyleSheetImport())
m_pImpl->deferCharacterProperty( nSprmId, uno::makeAny( nIntValue ));
m_pImpl->appendGrabBag(m_pImpl->m_aInteropGrabBag, (nSprmId == NS_ooxml::LN_EG_RPrBase_sz ? OUString("sz") : OUString("szCs")), OUString::number(nIntValue));
}
break;
@@ -2803,55 +2800,26 @@ void DomainMapper::processDeferredCharacterProperties( const std::map< sal_Int32
rProp.second >>= sStringValue;
switch( Id )
{
case NS_ooxml::LN_EG_RPrBase_sz:
case NS_ooxml::LN_EG_RPrBase_szCs:
break; // only for use by other properties, ignore here
case NS_ooxml::LN_EG_RPrBase_position:
{
double nEscapement = 0;
sal_Int8 nProp = 100;
if(nIntValue == 0)
nProp = 0;
else
sal_Int8 nProp = 0;
if ( nIntValue )
{
std::map< sal_Int32, uno::Any >::const_iterator font = deferredCharacterProperties.find( NS_ooxml::LN_EG_RPrBase_sz );
PropertyMapPtr pDefaultCharProps = m_pImpl->GetStyleSheetTable()->GetDefaultCharProps();
boost::optional<PropertyMap::Property> aDefaultFont = pDefaultCharProps->getProperty(PROP_CHAR_HEIGHT);
if( font != deferredCharacterProperties.end())
{
double fontSize = 0;
font->second >>= fontSize;
if (fontSize != 0.0)
nEscapement = nIntValue * 100 / fontSize;
}
// TODO if not direct formatting, check the style first, not directly the default char props.
else if (aDefaultFont)
{
double fHeight = 0;
aDefaultFont->second >>= fHeight;
if (fHeight != 0.0)
{
// fHeight is in points, nIntValue is in half points, nEscapement is in percents.
nEscapement = nIntValue * 100 / fHeight / 2;
}
}
nProp = 100;
double fFontSize = 0;
m_pImpl->GetAnyProperty(PROP_CHAR_HEIGHT, rContext) >>= fFontSize;
if ( fFontSize )
// nIntValue is in half-points, fontsize is in points, escapement is a percentage.
nEscapement = round( nIntValue/2.0 / fFontSize * 100 );
else
{ // TODO: Find out the font size. The 58/-58 values were here previous, but I have
// no idea what they are (they are probably some random guess that did fit whatever
// specific case somebody was trying to fix).
nEscapement = ( nIntValue > 0 ) ? 58: -58;
}
nEscapement = nIntValue > 0 ? DFLT_ESC_SUPER : DFLT_ESC_SUB;
}
// tdf#120412 up to 14400% (eg. 1584 pt with 11 pt letters)
if ( nEscapement > MAX_ESC_POS )
{
nEscapement = MAX_ESC_POS;
}
else if ( nEscapement < -MAX_ESC_POS )
{
nEscapement = -MAX_ESC_POS;
}
rContext->Insert(PROP_CHAR_ESCAPEMENT, uno::makeAny( sal_Int16(nEscapement) ) );
rContext->Insert(PROP_CHAR_ESCAPEMENT_HEIGHT, uno::makeAny( nProp ) );
