Skip to content

fix: Themed breakpoint not being applied #62

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Jul 15, 2017
Merged

Conversation

aaronvanston
Copy link
Owner

@aaronvanston aaronvanston commented Jul 1, 2017

Fixes: #61

Properly includes the breakpoint value when creating the media queries.

Help wanted:

Currently, with the way I am passing the template string as the second argument it now includes commas separating the string and the values, I'm having to now split on comma and rejoin. this is a very hacky solution.

@codecov
Copy link

codecov bot commented Jul 1, 2017

Codecov Report

Merging #62 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #62   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          11     11           
  Lines          72     72           
  Branches       17     17           
=====================================
  Hits           72     72
Impacted Files Coverage Δ
src/components/Col/Col.js 100% <100%> (ø) ⬆️
src/helpers/mediaQuery/mediaQuery.js 100% <100%> (ø) ⬆️
src/theme/theme.js 100% <100%> (ø) ⬆️
src/components/Row/Row.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6736fdd...1515060. Read the comment docs.

@codecov
Copy link

codecov bot commented Jul 1, 2017

Codecov Report

Merging #62 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #62   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          11     11           
  Lines          72     78    +6     
  Branches       17     20    +3     
=====================================
+ Hits           72     78    +6
Impacted Files Coverage Δ
src/components/Col/Col.js 100% <100%> (ø) ⬆️
src/helpers/mediaQuery/mediaQuery.js 100% <100%> (ø) ⬆️
src/components/Row/Row.js 100% <100%> (ø) ⬆️
src/theme/theme.js 100% <100%> (ø) ⬆️
src/helpers/columnWidth/columnWidth.js 100% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6736fdd...ae725f2. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant