Skip to content
This repository has been archived by the owner on Dec 3, 2017. It is now read-only.

Contact form styles #627

Merged
merged 6 commits into from
Feb 8, 2017
Merged

Contact form styles #627

merged 6 commits into from
Feb 8, 2017

Conversation

noahmanger
Copy link
Contributor

@codecov-io
Copy link

codecov-io commented Feb 8, 2017

Codecov Report

Merging #627 into master will not impact coverage.

@@           Coverage Diff           @@
##           master     #627   +/-   ##
=======================================
  Coverage   79.52%   79.52%           
=======================================
  Files          29       29           
  Lines        1470     1470           
  Branches      216      216           
=======================================
  Hits         1169     1169           
  Misses        301      301

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 2218aee...b58b360. Read the comment docs.

Copy link
Contributor

@xtine xtine left a comment

Choose a reason for hiding this comment

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

Just a small hr class naming thing, but otherwise looks good.

background-position: right 10px top 50%;
}

.hr--soft {
Copy link
Contributor

Choose a reason for hiding this comment

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

Just kind of a minor nitpick, but I think the hr naming convention might make more sense to follow the color variables, so like .hr and .hr--dark (and .hr--light if using $gray-light). What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good call.

@xtine xtine merged commit b1285f9 into master Feb 8, 2017
@xtine xtine deleted the contact-form-styles branch February 8, 2017 19:15
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