Skip to content
This repository was archived by the owner on Aug 5, 2019. It is now read-only.

adds State#10

Closed
mattlub wants to merge 2 commits intomasterfrom
state
Closed

adds State#10
mattlub wants to merge 2 commits intomasterfrom
state

Conversation

@mattlub
Copy link
Copy Markdown
Contributor

@mattlub mattlub commented Mar 29, 2018

  • adds state component

<State> and <Bind> seems slightly strange for names, maybe we should namespace all the components, e.g. <AmpBind>

@nuc thoughts?

@mattlub mattlub requested a review from nuc March 29, 2018 15:25
README.md Outdated
```jsx
<Bind text="'Hello ' + foo" className="bar">
<p>I will get the [text] and [class] attributes</p>
</Bind>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

oops, just noticed the examples are the wrong way round

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is fixed now

@mattlub mattlub force-pushed the master branch 2 times, most recently from 69d2771 to 9a649f8 Compare June 14, 2018 13:29
@mattlub mattlub mentioned this pull request Nov 13, 2018
@mattlub
Copy link
Copy Markdown
Contributor Author

mattlub commented Nov 13, 2018

closing, see #31 which is the updated version

@mattlub mattlub closed this Nov 13, 2018
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.

2 participants