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 trialTorger Angeltveit
11,228 PointsMy 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 :)
Thank you :)
9 Answers
Mohd Maqbool Alam
Full Stack JavaScript Techdegree Student 5,715 Pointswow really nice good job
Wayne Priestley
19,579 PointsHi 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
7,837 PointsGood 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
11,228 PointsThanks 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
11,228 PointsTransitions can't be used on background-image? only background-color, or am i wrong ?
Ross Litzenberger
11,706 PointsLOOK GREAT!!!! congrates
Brian O'Connor
13,226 Pointsreally 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
11,228 PointsThanks 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?
Byron Twogood
14,631 PointsLooks 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!