Skip to content
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

showHeaderOnEveryPage #263

Closed
wants to merge 2 commits into from
Closed

showHeaderOnEveryPage #263

wants to merge 2 commits into from

Conversation

Sgkhour
Copy link
Contributor

@Sgkhour Sgkhour commented Jan 10, 2021

looping back on the open issue #222 and some code changes i am proposing to address it (in line with #262 by David Moore). code is meant to fix some missing offset to line up headers and table spacing on every page. code can benefit from some refactoring but just tested it in 2.3.3 and looks good .

account for space.header and space.footer in page margins calculations
if table has a header that repeats on everypage, make sure the cells are shifted by headerHeight and adjust the position of the size wrt minOffset. also add table padding to the surrounding cells
@Sgkhour Sgkhour closed this Jan 10, 2021
@Sgkhour
Copy link
Contributor Author

Sgkhour commented Jan 10, 2021

submitted by mistake against master not develop base

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant