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

Global UI Styles for Habitat Web #345

Merged
merged 5 commits into from
Apr 5, 2016
Merged

Conversation

ryankeairns
Copy link
Contributor

This pull request is the first go at adding some global UI styles based upon current mockups. This stuff is not vetted as it will change once we get the Mortar branding guidelines at the end of April... but because that leaves us very little time I've gone ahead and started tidying up the HTML, adding in Sass variables for color/typography, etc., so that the end push is not as daunting.

@@ -6,6 +6,9 @@

@import "normalize";

// Include web fonts
@import url(https://fonts.googleapis.com/css?family=Montserrat:400,700|Titillium+Web:400,400italic,600);
Copy link
Contributor

Choose a reason for hiding this comment

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

Note that if we're going to ship this for on-prem use, we'll need to include the fonts on disk instead of calling out to Google's CDN. It's fine here for now.

@smith
Copy link
Contributor

smith commented Apr 4, 2016

Wow nice!

A few things:

@ryankeairns
Copy link
Contributor Author

@smith figured out the profile dropdown, I think this is ready!

@smith
Copy link
Contributor

smith commented Apr 5, 2016

@delivery approve

@chef-delivery
Copy link
Contributor

Failed to approve change: b9f0fd67-16d6-4a1a-97d4-513280b7f7fb
Change must pass verify before approving. @smith

@smith smith mentioned this pull request Apr 5, 2016
@chef-delivery chef-delivery merged commit 2f81d9f into master Apr 5, 2016
@juliandunn juliandunn deleted the rk/base-web-styles branch April 8, 2016 02:34
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