Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

General Discussion

Adam Smallman
Adam Smallman
815 Points

What do you think of my website ?

Let me know what you think of my website :)

http://www.bro-creative.com/

Thanks

Adam

3 Answers

Jason Ziegler
seal-mask
.a{fill-rule:evenodd;}techdegree
Jason Ziegler
Full Stack JavaScript Techdegree Student 36,647 Points

Instead of welcome as the only word on the front page, I would use this first impression space to grab the attention of the visitor or give a concise intro to your company. Perhaps let them know what problems you solve for them.

Also, I think it is strange that on the front page the 4 portfolio images dim on hover. This seems like you would want to illuminate on hover and dim out the other 3.

Hope this helps!

Adam Smallman
Adam Smallman
815 Points

Hello Jason thanks for your message.

One: Thats a grate idea about adding more information instead of just welcome, I will get on that now.

Two: I am working on that part now, I am adding in text which will appear when a user hovers over a project.

Thanks again :)

Zack Ward
Zack Ward
9,196 Points

Your site looks nice! I had two little suggestions that might help the site.

  • On your footer, if you add an ampersand character in front of the word copy, and a semicolon after the word copy ( & + copy + ; = © ), it will produce the copyright character.
  • On your services page, you may want to include the text for each service in the link. That would make it a bigger click target, and it will be much easier for those using touch screens to expand each section.

Just my two cents. =)

Camden Mounts
Camden Mounts
9,453 Points

Adam - your website looks awesome!

However, when I viewed your homepage on a 1920 x 1080 monitor, the grey-lonely-beach picture repeated horizontally. Either fixing the image size options or centering the image with white-space on the side would fix this issue. In your code, I noticed that your had actually set the background to 'repeat-x' , so if this is the desired effect, then I retake my critique.

Additionally, on your "About" page, the "Portfolio" and "Services" button hover-colors did not blend well together. The combination of the white page background, light-grey button background and the white button text made it hard to read when highlighted. I don't want to mess with your color-scheme, but perhaps making the background color a bit darker would improve the readability. Something along this line...

.MainAbout .introAbout .buttonWork:hover {
  background: #c6c6c6;
  text-decoration: none;
  color: #272727;
}

Keep working on it, can't wait to see how it turns out!

<CM>