Skip to content
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

Handle case where css float property is converted to cssFloat by autoprefixer camelCase function #224

Merged
merged 3 commits into from
Aug 2, 2017

Conversation

tkh44
Copy link
Member

@tkh44 tkh44 commented Aug 2, 2017

No description provided.

@tkh44 tkh44 requested a review from emmatown August 2, 2017 17:31
Copy link
Member

@emmatown emmatown left a comment

Choose a reason for hiding this comment

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

We should handle this for styles in tagged template literals too.

@codecov-io
Copy link

codecov-io commented Aug 2, 2017

Codecov Report

Merging #224 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #224   +/-   ##
=======================================
  Coverage   90.39%   90.39%           
=======================================
  Files          22       22           
  Lines         979      979           
  Branches      265      265           
=======================================
  Hits          885      885           
  Misses         76       76           
  Partials       18       18
Impacted Files Coverage Δ
src/ast-object.js 89.13% <ø> (ø) ⬆️

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 57af90c...f45292f. Read the comment docs.

@tkh44 tkh44 merged commit 89f17ba into master Aug 2, 2017
@emmatown emmatown deleted the float-fix branch November 18, 2017 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants