Opened 9 years ago
Closed 9 years ago
#14605 closed Bug (fixed)
Test for pagebreak fails in Chrome
| Reported by: | Tomasz Jakut | Owned by: | Tade0 |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.5.9 |
| Component: | General | Version: | 4.5.9 |
| Keywords: | Cc: |
Description
Open http://tests.ckeditor.dev:1031/tests/plugins/pagebreak/pagebreak#tests%2Fplugins%2Fpagebreak%2Fpagebreak%20test%20paste in newest stable Chrome (50). The test fails.
Change History (7)
comment:1 Changed 9 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 9 years ago by
comment:3 Changed 9 years ago by
| Owner: | set to Tade0 |
|---|---|
| Status: | confirmed → assigned |
comment:4 Changed 9 years ago by
| Status: | assigned → review |
|---|
More specifically currently in Chrome, when the style page-break-after: always is applied, it's represented as such in the DOM and dev tools, *but*, when getting the DOM element style in JS, the style object properties are as follows:
pageBreakAfter: ""
breakAfter: "page"
Changes pushed to branch:t/14605.
comment:6 Changed 9 years ago by
| Status: | review → review_passed |
|---|
This http://dev.ckeditor.com/ticket/14605#comment:4 is what exactly happens. I only added ticket reference in the fixed test. Looks good, R+.
comment:7 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with git:669f6f555d (merged to master).
Note: See
TracTickets for help on using
tickets.
Apparently current Chrome uses the newer property
break-afterinstead ofpage-break-after.I'll modify the test so that it takes this change into account.