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/draft-js#98
  • Loading branch information
aforismesen authored and flarnie committed Sep 20, 2016
1 parent 6190a9d commit 08c690f
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 08c690f

Please sign in to comment.