draft-js-multidecorators icon indicating copy to clipboard operation
draft-js-multidecorators copied to clipboard

fix: pass contentState through to decorators

Open echenley opened this issue 8 years ago • 4 comments

Adds support for DraftJS v0.10.0, which expects contentState to be passed in to each decorator.

See https://facebook.github.io/draft-js/docs/v0-10-api-migration.html and https://github.com/facebook/draft-js/blob/master/src/model/decorators/DraftDecoratorType.js.

echenley avatar Jan 31 '17 21:01 echenley

~~Can this be merged into master please? :+1: Hm unfortunately this doenst work for me. Does it still work for you @echenley ? decorator.getDecorations is not a function~~

Well okay, the example in the readme is just not complete. I checked tests and it needs to be new MultiDecorator([new Draft.CompositeDecorator([..])]). Doing that it works for me :+1:

sashless avatar Mar 20 '17 15:03 sashless

@SamyPesse, merge this please

i11v avatar Apr 21 '17 09:04 i11v

@SamyPesse please merge matey <3

juliankrispel avatar May 17 '17 13:05 juliankrispel

@SamyPesse please merge ;)

christophepas avatar Aug 02 '17 14:08 christophepas