We are migrating CKEditor issue tracking to GitHub. Please, use GitHub to report any new issues.

The former tracking system (this website) will still be available in the read-only mode. All issues reported in the past will still be available publicly and can be referenced.

Important: we decided not to transfer all the tickets to GitHub, as many of them are not reproducible anymore or simply no longer requested by the community.
If the issue you are interested in, can be still reproduced in the latest version of CKEditor, feel free to report it again on GitHub.
At the same time please note that issues reported on this website are still taken into consideration when picking up candidates for next milestones.

To report a new issue, go to https://github.com/ckeditor/ckeditor-dev/issues and follow the instructions in the issue template.

Context Navigation

← Previous Change
Ticket Comment History
Next Change →

Changes between Initial Version and Version 1 of Ticket #16682, comment 18

Timestamp:: Dec 23, 2016, 11:04:32 PM (7 years ago)
Author:: Marek Lewandowski
Comment:

Legend:

: Unmodified
: Added
: Removed
: Modified

Ticket #16682, comment 18

initial	v1
3	3	So the thing is that this fixture produces an edge case `magin:0`, which made `val` equal to 0 [https://github.com/cksource/ckeditor-dev/blob/39e58ab/plugins/pastefromword/filter/default.js#L1662 inside of levels lookup map] having levels equal to 0 it was messing up the transformations later on.
4	4
5		The fix here is to [https://github.com/cksource/ckeditor-dev/blob/39e58ab/plugins/pastefromword/filter/default.js#L1666-L1670 bump ~~_every_ level]~~if a case like this happens.
	5	The fix here is to [https://github.com/cksource/ckeditor-dev/blob/39e58ab/plugins/pastefromword/filter/default.js#L1666-L1670 bump every level] if a case like this happens.
6	6
7	7	I've added proper integration and unit test for this. Also there's an integration test for negative margins (which are possible to create in Word).