Bummer! You must be logged in to access this page.

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 trial

General Discussion

Posted by Jess Taylor
Jess Taylor
5,160 Points

My portfolio!

So I have done the HTML and CSS parts of the Frontend Web Development track, just started on the JS Basics yesterday.

As I have went along I have been making a portfolio for myself, its not 100% finished, there is a little bug with Firefox not showing the Social icons, the contact form hasn't been done yet, and I obv have no work to show off.... but its almost there! :)

Just wanting some opinions on how it looks :)

http://jessicatee.github.io/portfolio

lihaoquan
lihaoquan
12,045 Points

One thing about contrast is, when the background is dark, use bright colors for text, when background is bright, use dark colors for text. In your case, your text is above a dark background, so I think you can make it white. One great tool to check is http://contrastchecker.com/

2 Answers

Maxim Levikov
Maxim Levikov
6,546 Points
Maxim Levikov
Maxim Levikov
6,546 Points

Hi!

About bug in Firefox you need to change path to font-awesome.

Change this:

<link rel="stylesheet" href="font-awesome-4.4.0\css\font-awesome.min.css">

To this:

<link rel="stylesheet" href="font-awesome-4.4.0/css/font-awesome.min.css">
Jess Taylor
Jess Taylor
5,160 Points

Thanks, will have a go at changing it tonight, will this still make it work for other browsers?

lihaoquan
lihaoquan
12,045 Points
lihaoquan
lihaoquan
12,045 Points

I think you need more white space ( padding ) between the 3 sections ( Web Design, Responsive Design, Web development ) so that it'll be more clearer that they are three separate sections.

Your hero image and the orange text are conflicting, its quite hard to see the contrast, which causes readability problems. ( I had to use extra effort to see that the words are "Web Dev Extraordinaire" ).

Other than that I think it looks great, the colors at the "About Me" sections are nice.

Jess Taylor
Jess Taylor
5,160 Points

The header text colour hasn't been sitting right with me for a while whilst designing it, I think it looks clearer when it is white, or another brighter color?

Will also change the whitespace as I agree.

Thanks for the critique :)