Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Update documentation concerning immutable updates #1884

Conversation

cojennin
Copy link

@cojennin cojennin commented Oct 1, 2018

Summary
Adding some inline examples in the documentation around immutable updates for EditorState, ContentState and SelectionState based on the suggestions of #1445.

Let me know if there's more areas in the documentation that could use similar examples or if some editing needs to be done to what I've added.

Test Plan

N/A, but I did view in online markdown editor and it looks ok

Copy link
Contributor

@niveditc niveditc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - thanks for adding these examples.

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

niveditc is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

jdecked pushed a commit to twitter-forks/draft-js that referenced this pull request Oct 9, 2019
Summary:
**Summary**
Adding some inline examples in the documentation around immutable updates for `EditorState`, `ContentState` and `SelectionState` based on the suggestions of facebookarchive#1445.

Let me know if there's more areas in the documentation that could use similar examples or if some editing needs to be done to what I've added.

**Test Plan**

N/A, but I did view in online markdown editor and it looks ok
Pull Request resolved: facebookarchive#1884

Differential Revision: D10188040

fbshipit-source-id: c91d935d3ffdb7aee9ccf826af68c91cf1596a0a
alicayan008 pushed a commit to alicayan008/draft-js that referenced this pull request Jul 4, 2023
Summary:
**Summary**
Adding some inline examples in the documentation around immutable updates for `EditorState`, `ContentState` and `SelectionState` based on the suggestions of #1445.

Let me know if there's more areas in the documentation that could use similar examples or if some editing needs to be done to what I've added.

**Test Plan**

N/A, but I did view in online markdown editor and it looks ok
Pull Request resolved: facebookarchive/draft-js#1884

Differential Revision: D10188040

fbshipit-source-id: c91d935d3ffdb7aee9ccf826af68c91cf1596a0a
aforismesen added a commit to aforismesen/draft-js that referenced this pull request Jul 12, 2024
Summary:
**Summary**
Adding some inline examples in the documentation around immutable updates for `EditorState`, `ContentState` and `SelectionState` based on the suggestions of #1445.

Let me know if there's more areas in the documentation that could use similar examples or if some editing needs to be done to what I've added.

**Test Plan**

N/A, but I did view in online markdown editor and it looks ok
Pull Request resolved: facebookarchive/draft-js#1884

Differential Revision: D10188040

fbshipit-source-id: c91d935d3ffdb7aee9ccf826af68c91cf1596a0a
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants