Closed Bug 1059705 Opened 5 years ago Closed 5 years ago
revert the change which made ns
At least for now nsIEditor should be implementable in JS. We should probably figure out what the Thunderbird addons are doing and give them some other options than re-implementing rather complicated interface...and then make nsIEditor builtinclass again.
Attachment #8480483 - Flags: review?(masayuki)
Comment on attachment 8480483 [details] [diff] [review] patch >- [infallible, noscript] readonly attribute boolean isInEditAction; >+ readonly attribute boolean isInEditAction; Ehsan liked [noscript] this (bug 1053048 comment 17). If it's possible without "builtinclass", it should be, I think.
Attachment #8480483 - Flags: review?(masayuki) → review+
That should work.
Attachment #8480483 - Attachment is obsolete: true
Sigh. Why did we not wait for the discussion on tb-planning to at least get somewhere?
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Because we're close to an uplift and the fix in this case was trivial.
See bug 1060051 "Resolve obstacles that require Conversations to wrap nsIEditor"
FYI: Now, we can make nsIEditor builtinclass again because of bug 1069518.
You need to log in before you can comment on or make changes to this bug.