Skip to content

Support passing a context variable to the custom format prepareEditTreeValue#11630

Merged
youknowriad merged 1 commit intotry/prepare-editablefrom
add/context-support
Nov 9, 2018
Merged

Support passing a context variable to the custom format prepareEditTreeValue#11630
youknowriad merged 1 commit intotry/prepare-editablefrom
add/context-support

Conversation

@youknowriad
Copy link
Copy Markdown
Contributor

This PR adds an identifier prop to the RichText component.
It also passes the "clientId" and "richTextIdentifier" as "context" to the format API.

@youknowriad youknowriad added the [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable label Nov 8, 2018
@youknowriad youknowriad added this to the 4.3 milestone Nov 8, 2018
@youknowriad youknowriad self-assigned this Nov 8, 2018
@youknowriad youknowriad requested review from a team and ellatrix November 8, 2018 15:30
Copy link
Copy Markdown
Member

@tofumatt tofumatt left a comment

Choose a reason for hiding this comment

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

Makes sense to me; seems like it'd be a good fit for some unit tests, but I know timelines are tight and it's experimental 🤷‍♂️

@gziolo
Copy link
Copy Markdown
Member

gziolo commented Nov 8, 2018

Any plans to add identifier to other blocks?

@youknowriad
Copy link
Copy Markdown
Contributor Author

@gziolo Could be done as needed. This is just a first iteration to implement the concept.

@youknowriad youknowriad merged commit 89539fd into try/prepare-editable Nov 9, 2018
@youknowriad youknowriad deleted the add/context-support branch November 9, 2018 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants