Spell check deletes words if there's color formatting on the line

QUICK FIX: https://github.com/quilljs/quill/issues/2096#issuecomment-399576957

Steps for Reproduction

1. Visit quilljs.com and go to the third example where you can set a font color
2. Create a colored sentence with a typo, like “I love makig bugs”
3. Right click “makig” to correct the spelling

Expected behavior: The spelling should be corrected just like regular, non-formatted text

Actual behavior: Characters are deleted. Sometimes it’s just the word, sometimes it’s everything before it

Platforms: macOS High Sierra. Chrome and Safari (notably, not Firefox)

Version: 1.3.6

Related:

The issue doesn’t happen when you use classes instead of inline styles, which makes me think it’s some kind of issue with normalized, rgb() values and quill uses hex. I tried forcing the Delta to use rgb() but it didn’t help.

[chearon]

This seems to do everything correctly, I’ll update it with any fixes:

const Inline = Quill.import('blots/inline');

class CustomColor extends Inline {
  constructor(domNode, value) {
    super(domNode, value);

    // Map <font> properties
    domNode.style.color = domNode.color;

    const span = this.replaceWith(new Inline(Inline.create()));

    span.children.forEach(child => {
      if (child.attributes) child.attributes.copy(span);
      if (child.unwrap) child.unwrap();
    });

    this.remove();

    return span;
  }
}

CustomColor.blotName = "customColor";
CustomColor.tagName = "FONT";

Quill.register(CustomColor, true);

[LeeMustache]

It’s still a problem. I think it’s worth fixing instead of relying on programmers to create workarounds. Spell correcting is a pretty basic and commonly used feature.