Closed Bug 570205 Opened 10 years ago Closed 10 years ago
Target Editable could use is Content Editable
Currently IsTargetEditable works by looking for an HTML element with a non-inherit value of the contentEditable property. The new isContentEditable property does much the same thing, although we still have to find an HTML element; once we do find one we can then return its editableness directly.
Attachment #449595 - Flags: review? → review?(neil)
Comment on attachment 449595 [details] [diff] [review] Changed the return to use IsContentEditable. > for (var node = this.target; node; node = node.parentNode) > if (node instanceof Components.interfaces.nsIDOMNSHTMLElement) >+ return node.isContentEditable; [Nit: too much indentation. Please remove one space before checkin.]
Attachment #449595 - Flags: review?(neil) → review+
Removed a space.
Attachment #449595 - Attachment is obsolete: true
Comment on attachment 449612 [details] [diff] [review] Changed the return to use IsContentEditable. [Checkin: comment 4] http://hg.mozilla.org/comm-central/rev/a8b8bdb983aa
Attachment #449612 - Attachment description: Changed the return to use IsContentEditable. → Changed the return to use IsContentEditable. [Checkin: comment 4]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a3
You need to log in before you can comment on or make changes to this bug.