Detected contentEditable element.

[Alecyrus]

I don't get it why you detect contentEditable element.

Description

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

[idiotWu]

The problem is that the value of contentEditable attribute may be inherit (default value), which means you have to traverse through all the ancestor elements to see if this one is editable.

[idiotWu]

A simple but high-cost way to fix it:

while (elem) {
  if (elem.contentEditable === "true") {
     // do something
    break;
  }

  elem = elem.parentElement;
}

[idiotWu]

Just got an answer from stack overflow:

- elem.contentEditable === "true"
+ elem.isContentEditable

@Alecyrus can you modify your code like the above snippet?

[Alecyrus]

Change the rules for detecting ContentEditable elements

[idiotWu]

Thanks!

[idiotWu]

v8.4.1 is out