Changes between Version 2 and Version 3 of Ticket #12621, comment 7


Ignore:
Timestamp:
Nov 10, 2014, 7:31:44 PM (5 years ago)
Author:
Piotrek Koszuliński
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #12621, comment 7

    v2 v3  
    331. Result of `style.removeFromRange` is correct. The range is placed in: `<p>foo<strong>\u200b</strong>[]</p>`.
    442. 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.
    5 3. 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.
     53. 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 `false` for the `<strong>\u200b</strong>` element. But it does not do this anymore.
    77
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy