Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trialAdam Smallman
815 PointsWhat do you think of my website ?
3 Answers
Jason Ziegler
Full Stack JavaScript Techdegree Graduate 39,583 PointsInstead 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!
Zack Ward
9,196 PointsYour 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
9,453 PointsAdam - 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>
Adam Smallman
815 PointsAdam Smallman
815 PointsHello 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 :)