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

Hooks useTheme #786

Merged
merged 2 commits into from
Jan 28, 2019
Merged

Hooks useTheme #786

merged 2 commits into from
Jan 28, 2019

Conversation

daniel-martic-sociomantic
Copy link
Contributor

As suggested here:

  • removed createCssMap
  • created utils/customHooks containing useTheme that builds cssMap

After each component is converted to functions that are using hooks, useTheme should be called instead of current createCssMap to generate cssMap

Copy link
Contributor

Choose a reason for hiding this comment

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

LG 🚀

Copy link
Contributor

@damian-rodriguez-sociomantic damian-rodriguez-sociomantic left a comment

Choose a reason for hiding this comment

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

LGTM; just a small note; should we add also this in this PR or we keep it separate?

@conor-cafferkey-sociomantic
Copy link
Contributor

Yeah would be good to get that in too. But I guess it should be a separate PR

@mahmoud-zakria-sociomantic
Copy link
Contributor

This is Awesome 👍 👍

@daniel-martic-sociomantic daniel-martic-sociomantic merged commit 82a754d into Milestone-Hooks! Jan 28, 2019
@daniel-martic-sociomantic daniel-martic-sociomantic deleted the Hooks-useTheme branch January 28, 2019 12:56
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.

4 participants