soggybag / Front-end-Web-Class-8

0 stars 0 forks source link

https://coreyarch1234.github.io/ #2

Closed coreyarch1234 closed 7 years ago

coreyarch1234 commented 8 years ago

Take a look and give some feedback.

juanpablofernandez commented 8 years ago

I like your website however you could add/fix a few things:

outdoorsole commented 8 years ago

Hi Corey!

Feedback:

Good job Corey!

Nkdream commented 8 years ago

The content itself is good, but I feel as though the design could use some work, for instance I feel like the colors do not synergize very well. Some of the elements on the page are not aligned exactly and a tiny bit off center. It is a small detail but I feel as though it would make a huge difference

davidkc0 commented 8 years ago

Very simple and elegant! The few changes I would make are:

  1. keep the font color the same. in the orange box you have black font but for the rest of the site you have white.
  2. The site has a side scrolling issue. Make sure it fits the page.
  3. Add a logo to the header. I understand this is a new idea, so you might not have a logo yet, but when you do make sure to make it the enter point of your header section.
  4. Add "Made with ♥ in San Francisco, CA" to the footer. All the cool hipster startups do it. ;)
tlambrou commented 8 years ago

Nice job Corey! I really like the colors and the big prominent text.

brianhans commented 8 years ago

I like the simple design and color scheme. I would center the content on the main screen though and remove the margin on the body so that the blocks extend to the edge of the screen. The bars get misplaced when the screen gets too small. I'm gonna try to make a pull request with some of my suggested edits later today so you can see if you like the changes.