Skip to content
This repository has been archived by the owner on Dec 31, 2020. It is now read-only.

Update Observer component definition #376

Merged
merged 1 commit into from
Dec 27, 2017

Conversation

RafalFilipek
Copy link
Contributor

Hi!

Right now we have to wrap observerd value with HTML tag like:

const C => <Observer>{() => <div>{obser.vable}</div>}</Observer>

With this change you can just return observed value

const C => <Observer>{() => obser.vable}</Observer>

Thank you!

@mweststrate
Copy link
Member

Great, thanks!

@mweststrate mweststrate merged commit a1e05d9 into mobxjs:master Dec 27, 2017
@danielkcz
Copy link
Contributor

@mweststrate Do you plan to release a new version soon? This one is particularly annoying since I cannot return null from there.

@mweststrate
Copy link
Member

@FredyC yeah, I want to release if #403 is in. If that is not today, I'll probably cut a release at the end of the day anyway

@github-actions github-actions bot mentioned this pull request Oct 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants