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

Design

Torger Angeltveit
Torger Angeltveit
11,228 Points

My first website up and running, feedback please :)

I published my first website after getting into webdesign and would be grateful for some feedback! its just an easy website with a couple of pages.

It's a Christmas calendar for my girlfriend :)

http://carinascalendar.com/

Thank you :)

9 Answers

wow really nice good job

Wayne Priestley
Wayne Priestley
19,579 Points

Hi Torger,

Nice I like it and its a great idea. :)

Might i suggest having the presents fade in ever so slightly, maybe round the corners a few pixels too.
I don't know how your coding your site but something like this might help you on your way.

#mydiv {
transition: background-image 1s ease;
background-image:url(β€œ/firstimg.jpg”);
}

#mydiv:hover {
background-image:url(β€œ/secondimg.jpg”);
}

Hope this helps.

Ron McCranie
Ron McCranie
7,837 Points

Good job. Better than my first website.

I would try to compress the big background image a little more so it loads faster. 1.1Mb is a big file to send.

Torger Angeltveit
Torger Angeltveit
11,228 Points

Thanks for advice guys, I really appreciate it!

And Wayne, I like that proposal i'm definitely going do it, i felt like something were missing in that area on the site. I think you might found it hehe, thank you.

Torger Angeltveit
Torger Angeltveit
11,228 Points

Transitions can't be used on background-image? only background-color, or am i wrong ?

Brian O'Connor
Brian O'Connor
13,226 Points

really great first website, I would add a back button to the individual day screens to bring your girlfriend back to the calendar page. Also is there a control on the individual links to not make them active until the relevant date or is that a manual thing?

Torger Angeltveit
Torger Angeltveit
11,228 Points

Thanks for tip Brian! Yeah that would definitely be more user friendly. No, i upload them manually for each day. Maybe there is a way of doing this in PHP? You have any idea of how this could be done?

Looks good! I only have a couple suggestions:

I believe someone pointed out fading the presents a little... The way they are now seems too... Harsh... With all the soft edges and colors, the presents stand out way too much for my liking. I like where you're going with it, just soften the edges a little.

And second, when you hover over the days and the present pops up, you can still see the underline for the link. I would try to remove that if you can.

Other than that, great job!