vcl/source/gdi/pdfwriter_impl.cxx | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
New commits:
commit c9f801f527634a4ec1b5712d345a5ff669e26bbd
Author: Michael Stahl <michael.st...@allotropia.de>
AuthorDate: Mon Aug 5 14:09:47 2024 +0200
Commit: Michael Stahl <michael.st...@allotropia.de>
CommitDate: Mon Aug 5 17:33:02 2024 +0200
tdf#155228 vcl: appease Adobe Acrobat Pro, always produce /Tabs
Reportedly Adobe Acrobat Pro complains if any page is missing "/Tabs"
even though ISO 14289-1:2014 requires it only for a page with
annotations.
Change-Id: I84d750da12ca55519aa61c196d2d93aa33f1abad
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171486
Reviewed-by: Michael Stahl <michael.st...@allotropia.de>
Tested-by: Jenkins
diff --git a/vcl/source/gdi/pdfwriter_impl.cxx
b/vcl/source/gdi/pdfwriter_impl.cxx
index 70058f24e426..99f528c84572 100644
--- a/vcl/source/gdi/pdfwriter_impl.cxx
+++ b/vcl/source/gdi/pdfwriter_impl.cxx
@@ -776,11 +776,13 @@ bool PDFPage::emit(sal_Int32 nParentObject )
aLine.append( ((i+1)%15) ? " " : "
" );
}
aLine.append( "]
" );
- if (PDFWriter::PDFVersion::PDF_1_5 <= m_pWriter->m_aContext.Version)
- {
- // ISO 14289-1:2014, Clause: 7.18.3
- aLine.append( "/Tabs/S
" );
- }
+ }
+ if (PDFWriter::PDFVersion::PDF_1_5 <= m_pWriter->m_aContext.Version)
+ {
+ // ISO 14289-1:2014, Clause: 7.18.3 requires it if there are
annotations
+ // but Adobe Acrobat Pro complains if it is ever missing so just
+ // write it always.
+ aLine.append( "/Tabs/S
" );
}
if( !m_aMCIDParents.empty() )
{
