Skip to content
This repository was archived by the owner on May 4, 2020. It is now read-only.

Conversation

@lusimeon
Copy link
Contributor

@lusimeon lusimeon commented Feb 4, 2019

Close issue #5

  • Add a grid-gutter map variable in _grid.scss.
  • Add a gutter function who call the right gutter width for a specific breakpoint.

@lusimeon lusimeon added the enhancement New feature or request label Feb 4, 2019
Copy link
Contributor

@titouanmathis titouanmathis left a comment

Choose a reason for hiding this comment

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

Nice!

I think we can delete the $space-gutter variable defined in the _spaces.scss file, it will not be used anymore.

I will add a simple test strategy to be able to test (manually for the moment), the output of the configuration.

Copy link
Contributor

@titouanmathis titouanmathis left a comment

Choose a reason for hiding this comment

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

I added a folder with test files and the @studiometa/gulp-config package to test how the files are compiled.

I will add information on these test in the PR #3.

@titouanmathis titouanmathis added this to the v1.1.0 milestone Feb 4, 2019
@titouanmathis titouanmathis merged commit 05b698d into develop Feb 5, 2019
@titouanmathis titouanmathis deleted the feature/mutliple-grid-gutters branch February 5, 2019 14:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants