Changes between Initial Version and Version 1 of Ticket #12621, comment 7


Ignore:
Timestamp:
Nov 10, 2014, 7:01:39 PM (9 years ago)
Author:
Piotrek Koszuliński
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #12621, comment 7

    initial v1  
    22
    331. Result of `style.removeFromRange` is correct. The range is placed in: `<p>foo<strong>\u200b</strong>[]</p>`.
    4 2. When `selection.selectRanges` is executed it checks whether the selection needs a filling char.
     42. When `selection.selectRanges` is executed it checks whether the selection needs a filling char. **Note:** by "correct" I mean that it has not changed recently, although it is a bug that the `removeFromRange()` method does not remove the empty inline element.
    553. In this case the algorithm (implemented by the `rangeRequiresFix` function) should find that the range is placed right after `<strong>\u200b</strong>` and return true - the range requires the filling char.
    664. For some odd reason the algorithm uses the `walker.invisible()` method. Apparently it did return `true` for the `<strong>\u200b</strong>` element. But it does not do this anymore.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy