Issue metadata
Sign in to add a comment
|
Using Javascript, trying to set "page-break-before:always" on an element sets it to "page" instead. (CSS issue?) |
||||||||||||||||||||||
Issue descriptionI'm using Windows 7 Chrome Canary, latest as of 6/22/2016. I have potentially a similar issue as this: https://bugs.chromium.org/p/chromium/issues/detail?id=596353&can=1&q=page-break-before&colspec=ID%20Pri%20M%20Stars%20ReleaseBlock%20Component%20Status%20Owner%20Summary%20OS%20Modified I add a div that has a page-break-before:always CSS property before another div, which has a table as a grandchild. It potentially stopped working recently. (last few months) I'm on Canary. I don't know what version because the About page is broken. It's the Canary version as of 6/22/2016. The issue is that instead of "always", the property magically changes to "page". I found a workaround, which is to set the style attribute; E.G.: page_break_div.setAttribute ("style", "page-break-before:always") I don't have a sample right now... maybe later. RIP.
,
Jul 12 2016
It's pretty hard to imagine what's going on without a test case. Could you please attach one? Thanks!
,
Jul 19 2016
It basically doesn't seem to work... if I have time then I will add a test case... needs a CSS person here who can actually make changes though...
,
Jul 20 2016
If you have time to create a test case, please reopen this bug at that point. Closing this as unpronounceable until then.
,
Jul 20 2016
Why can't someone at Google create a test case? I am not likely going to create one in the near future.. so that's it? No more bug fixes at Google unless the submitter does this kind of legwork? I'm re-opening this. If Google abandoned Chromium then feel free to close it again and maybe in a few months I'll do a test case, if I remember!
,
Jul 20 2016
Thank you for providing more feedback. Adding requester "meade@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 20 2016
agamemnus@, thanks for your response. Typically we ask for test cases because we can't reproduce the issue locally, and need a test case to make sure we're looking at the right thing. For example, https://jsfiddle.net/cjn0udyf/1/ reproduces the conditions you described in your report, yet behaves as expected on Canary. Without a test case that is known failing for you, there is nothing else we can do. I'd also like to invite you to read https://www.chromium.org/conduct and help us ensure that future communication is conducted in line with providing a respectful and constructive community.
,
Jul 20 2016
I am reciprocating by working a bit more and creating the test case. It seems that the issue is actually just in the .innerHTML part of it (seemingly). Take a look: https://jsfiddle.net/cjn0udyf/2/ Separately, given that I'm not being paid to post reports or do test cases, and it looked like no well-paid Google employees were given an opportunity to at least create a sample test with my description, it seemed like I was being coerced to write a test case, or have the bug get shut down. Your test case, though, inspired me to finally find the bug with my own test case, so thanks for that.
,
Jul 20 2016
This behavior is per-spec: https://www.w3.org/TR/css-break-3/#page-break-properties (specifically "For compatibility with CSS Level 2, UAs that conform to [CSS21] must alias the page-break-before, page-break-after, and page-break-inside properties to break-before, break-after, and break-inside by treating the page-break-* properties as shorthands for the break-* properties with the following value mappings: always: page "). There's no bug here. Incidentally, this is precisely why we ask for reporters to provide test cases when they can, so that the behavior they're describing is made clear. Both meade@ and I tried to reproduce your issue from the description but couldn't - please assume good intentions from engineers on the project.
,
Jul 21 2016
Ok, so I think I understand this a bit better now, thanks. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by cbiesin...@chromium.org
, Jul 11 2016