- Issue created by @agentrickard
- Status changed to Needs review
about 1 year ago 3:17pm 10 November 2023 - Open on Drupal.org βCore: 10.1.4 + Environment: PHP 8.1 & MySQL 8last update
about 1 year ago Waiting for branch to pass - πΊπΈUnited States agentrickard Georgia (US)
And a patch to show the output change.
- πΊπΈUnited States bluegeek9
I think a settings form to allow site builders to control mPDF's settings.
Like β¨ DomPDF settings Needs review - Status changed to Needs work
about 1 year ago 4:03pm 10 November 2023 - πΊπΈUnited States agentrickard Georgia (US)
OK, I'll queue it up and use the patch in the meantime.
- Status changed to Closed: works as designed
about 1 year ago 4:55pm 10 November 2023 - πΊπΈUnited States agentrickard Georgia (US)
For the record -- it turns out that image 1 is more correct.
The problem we were having was with REM units not being defined. https://mpdf.github.io/css-stylesheets/supported-css.html
Specifically:
Note: Support for rem was added in mPDF 5.7. Unlike the CSS3 specification, the basic size used for rem in the document is based on the font-size set on the element (rather than the element).
Our default css specifies font-size in REM on the body, assuming inheritance from HTML.
So we don't need the patch, and I am going to set this back to a closed support request. We can open another ticket for settings if needed.