New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for page-break-before: avoid; and page-break-after: avoid; for <tr> elements #2004
Conversation
Thanks for the PR. in order to be incorporated, please add a reasonable test coverage and include a changelog entry. |
Thanks to the unit tests I found a copy&paste error, so the change should be fine now. |
Can you also please move the test to a separate class under Issues/? Thanks. |
You should be able to check CS locally easily by just running |
which issue number shall I use? |
Well, this one, 2004 |
Great, thanks! Can you also prepare a PR to update the supported CSS docs at https://github.com/mpdf/mpdf.github.io/blob/master/css-stylesheets/supported-css.md? |
done |
This code is sponsored by Launix (https://launix.de) and released under GPL 2.0
In order to not get into fork hell, we appreciate a fast release of the feature in the next minor version.