New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 622421 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Not working on Chrome any more
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows , Chrome
Pri: 1
Type: Bug-Regression



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?)

Project Member Reported by agamem...@gmail.com, Jun 22 2016

Issue description

I'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.
 
Components: Blink>CSS

Comment 2 by meade@chromium.org, Jul 12 2016

Labels: -css Needs-Feedback
Owner: agamem...@gmail.com
It's pretty hard to imagine what's going on without a test case. Could you please attach one? Thanks!

Comment 3 by agamem...@gmail.com, Jul 19 2016

Owner: ----
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...

Comment 4 by shans@chromium.org, Jul 20 2016

Status: WontFix (was: Untriaged)
If you have time to create a test case, please reopen this bug at that point. Closing this as unpronounceable until then.

Comment 5 by agamem...@gmail.com, Jul 20 2016

Cc: shans@chromium.org
Status: Unconfirmed (was: WontFix)
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!
Project Member

Comment 6 by sheriffbot@chromium.org, Jul 20 2016

Labels: -Needs-Feedback Needs-Review
Owner: meade@chromium.org
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

Comment 7 by shans@chromium.org, Jul 20 2016

Status: WontFix (was: Unconfirmed)
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.

Comment 8 by agamem...@gmail.com, Jul 20 2016

Status: Unconfirmed (was: WontFix)
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.

Comment 9 by shans@chromium.org, Jul 20 2016

Status: WontFix (was: Unconfirmed)
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.

Comment 10 Deleted

Ok, so I think I understand this a bit better now, thanks.

Sign in to add a comment