Initial Commit - #2
Conversation
New design, layout, updated information
|
Email must be created, fonts must be updated to match logo |
| @@ -1,2 +1,26 @@ | |||
| # projectreflect.github.io | |||
| Organization Website | |||
| # [Start Bootstrap](http://startbootstrap.com/) - [Grayscale](http://startbootstrap.com/template-overviews/grayscale/) | |||
|
i agree with @Alexander-Wen, also can we find another cover image? I really don't like the Made in USA, because this isn't |
| <title>Reflect - The Smart Mirror</title> | ||
|
|
||
| <!-- Bootstrap Core CSS --> | ||
| <link href="css/bootstrap.min.css" rel="stylesheet"> |
There was a problem hiding this comment.
Use a CDN, lets not host ourself, it bloats our repo
|
|
||
|
|
||
| ## Creator | ||
| The Reflect website was created, and is maintained by Adrian. |
There was a problem hiding this comment.
Not super necessary, will remove if you want.
There was a problem hiding this comment.
We don't have a creator on any other repo
|
I am still trying to figure out how to do the vertical centering without affecting the other elements in the nav bar |
|
if its too much of a hassle now, its fine. Just raise an issue on github so we remember to deal with it later. |
|
This is just personal preference and you don't have to do this if you don't want to, but can you squash your commits into 1 to make the git tree more tidy |
Cleaning up JS, removing unecessary code, converting to CDN
Fixed spacing issue near the bottom of the page
|
Just add bottom padding or margin to that element or negative padding/margin to the top |
| <div class="col-lg-8 col-lg-offset-2"> | ||
| <h2>Contact Us</h2> | ||
| <p>Feel free to email us to provide some feedback on Reflect, give us suggestions for new features and widgets, or to just say hello!</p> | ||
| <p><a href="mailto:[email protected]">[email protected]</a> |
|
No one does yet though, we should grab a domain soon anyways, since we are putting this site up |
|
Doesn't make sense to have a contact email that doesn't exist |
|
Also, vertical spacing of nav bar logo still off and I think a sans-serif body font makes more sense when you have a serif title font |


New design, layout, updated information