Skip to content

Commit

Permalink
Update docs to remove 'React ContentEditable Warning' issue
Browse files Browse the repository at this point in the history
The issue with 'React ContentEditable Warning' was resolved when React
added the 'suppressContentEditableWarning' option[1] and then we used it
in Draft.js[2]. This doesn't need to be in the docs any more.

[1]: facebook/react#6112
[2]: facebookarchive#98
  • Loading branch information
flarnie authored and ouchxp committed Apr 7, 2017
1 parent fdabce6 commit b9b0d7a
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions docs/Advanced-Topics-Issues-and-Pitfalls.md
Original file line number Diff line number Diff line change
Expand Up @@ -46,17 +46,6 @@ likely need to replicate much of the default styling.

## Known Issues

### React ContentEditable Warning

Within the React core, a warning is used to ward off engineers who wish to
use ContentEditable within their components, since by default the
browser-controlled nature of ContentEditable does not mesh with strict React
control over the DOM. The Draft editor resolves this issue, so for our case,
the warning is noise. You can ignore it for now.

We are currently looking into removing or replacing the warning to alleviate
the irritation it may cause: https://github.com/facebook/react/issues/6081

### Custom OSX Keybindings

Because the browser has no access to OS-level custom keybindings, it is not
Expand Down

0 comments on commit b9b0d7a

Please sign in to comment.