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

Conversation

@ItsJonQ
Copy link
Contributor

@ItsJonQ ItsJonQ commented Oct 19, 2018

Grid: Refactor + CSS-in-JS

screen recording 2018-10-18 at 08 48 pm

This update refactors the various Grid components with the latest Blue
conventions, such as namespacing, propConnect, and type support.

It also uses CSS-in-JS for styling, powered by Fancy. This allow us to remove @seedcss/seed-grid, which shaves about 9kb in the compiled CSS file size (minified).

For context, the Grid system is one of the larger seed packages.

Performance has also been improved as these components are now
PureComponent classes, rather than SFC.

The tests have also been adjusted to favour mount over shallow rendering.

Jon Q added 4 commits October 18, 2018 20:50
This update refactors the various Grid components with the latest Blue
conventions, such as namespacing, propConnect, and type support.

It also uses CSS-in-JS for styling, powered by Fancy.

Performance has also been improved as these components are now
`PureComponent` classes, rather than SFC.

The tests have also been adjusted to favour `mount` over `shallow` rendering.
@ItsJonQ ItsJonQ self-assigned this Oct 19, 2018
@ItsJonQ ItsJonQ requested review from Charca and knicklabs October 19, 2018 00:55
@ItsJonQ ItsJonQ changed the title ## Grid: Refactor + CSS-in-JS Grid: Refactor + CSS-in-JS Oct 19, 2018
@coveralls
Copy link

coveralls commented Oct 19, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 30b21a0 on grid-css-in-js into e870883 on master.

@ItsJonQ ItsJonQ merged commit 75553ae into master Oct 19, 2018
@ItsJonQ ItsJonQ deleted the grid-css-in-js branch November 8, 2018 18:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants