Skip to content
This repository was archived by the owner on Jun 20, 2022. It is now read-only.

Conversation

@gregberge
Copy link
Member

BREAKING CHANGES:

  • OriginalComponent.withComponent(NewComponent) is replaced by uiAs(OriginalComponent, NewComponent)

  • as={NewComponent} is replaced by uiAs={NewComponent}

  • globalStyle() is now replaced by Normalize component

@gregberge gregberge changed the title feat: support emotion 10 feat: support emotion 10 & remove hacks Jan 15, 2019
@gregberge gregberge force-pushed the emotion-10 branch 3 times, most recently from 9cf04ad to 1456988 Compare January 16, 2019 12:48
BREAKING CHANGE:

`OriginalComponent.withComponent(NewComponent)` is replaced by `uiAs(OriginalComponent, NewComponent)`

`as={NewComponent}` is replaced by `uiAs={NewComponent}`

`globalStyle()` is now replaced by `Normalize` component
@gregberge gregberge merged commit d311640 into master Jan 16, 2019
@gregberge gregberge deleted the emotion-10 branch January 16, 2019 12:55
}

return deepmerge(acc, item, {
clone: false, // No need to clone deep, it's way faster.

Choose a reason for hiding this comment

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

;)

Copy link
Member Author

Choose a reason for hiding this comment

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

Inspired by you!

const propKeys = Object.keys(attrs)
const propCount = propKeys.length
let allStyle = {}
for (let i = 0; i < propCount; i++) {

Choose a reason for hiding this comment

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

for > map :)

Copy link
Member Author

Choose a reason for hiding this comment

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

Complexity n, even with compose ;)

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