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

Handle break-* in columns #1963

Open
pmjdebruijn opened this issue Sep 13, 2023 · 3 comments
Open

Handle break-* in columns #1963

pmjdebruijn opened this issue Sep 13, 2023 · 3 comments
Labels
bug Existing features not working as expected

Comments

@pmjdebruijn
Copy link
Contributor

pmjdebruijn commented Sep 13, 2023

Example: page-break-column-bug.zip

I'm wondering whether this is a bug that is rooted in intentional behavior for column-less pages.

When break-before: page is applied to a page without columns, you wouldn't want it to trigger a new page, when you're already on top of a clean new page (otherwise, it would effectively result in an empty page being generated).

When break-page: page is applied to a page with columns, you would want to get a clean new page, even if it's at the top of a brand new column.

@liZe liZe added the bug Existing features not working as expected label Sep 15, 2023
@liZe liZe changed the title break-before: page seems to be ignored at top of column Handle break-before in columns Sep 15, 2023
@liZe
Copy link
Member

liZe commented Sep 15, 2023

I'm wondering whether this is a bug that is rooted in intentional behavior for column-less pages.

It’s a bug! break-before and break-after is broken in columns.

@pmjdebruijn
Copy link
Contributor Author

Thanks for the confirmation.

I've also noticed where it seems like a page break actually generates a column break instead, so that might be related...

@liZe
Copy link
Member

liZe commented Sep 19, 2023

I've also noticed where it seems like a page break actually generates a column break instead, so that might be related...

Yes, it’s totally broken.

(Also related to #1856.)

@liZe liZe changed the title Handle break-before in columns Handle break-* in columns Sep 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Existing features not working as expected
Projects
None yet
Development

No branches or pull requests

2 participants