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 trialChristian Lüthy
18,715 PointsFeedback on my personal project
Hi everyone,
I’ve been useing the skills I’ve learned to create a new photography website.
Take a look at http://gws-chl.ch. (Maybe also take a look at it on your phone for the mobile version.) I used everything from html and css to php, javascript and jquery.
I would love to know what you think and to tell me of any bugs you may find.
Thanks, Chris
2 Answers
Holt Hunter
4,629 PointsI can't find any problems, but I think it would be neat if when you click on a picture it slides out instead of just suddenly becoming bigger. Not sure how possible that is but it's just an idea.
Łukasz Czuliński
8,646 PointsI love the styling and overall look. Nice use of white space.
Functionality-wise I think the scrolling method has to change. It took me a few seconds to realize how to go left and right. I also think you need to increase the font size a bit in the photo captions, and possibly heavier font-weight.
Overall great work though. I viewed on 1980x1020 laptop in Chrome.
Christian Lüthy
18,715 PointsThank you for your feedback. What do you mean by scrolling method?
Łukasz Czuliński
8,646 PointsI just mean to go left and right. I had to bring the mouse to the very bottom of the page using the scroll bar there.
Christian Lüthy
18,715 PointsI'm just so used to my trackpad on the notebook and my magicMouse that it didn't even occur to me that you also have to provide a way to physically click and drag a scroll bar. thanks
Christian Lüthy
18,715 PointsChristian Lüthy
18,715 PointsYes that would be cool. I haven't thought of that. I've found a little issue there. Sometimes the images enlarge to the left side and other times to the right side and I have no idea why or how to fix it.
Holt Hunter
4,629 PointsHolt Hunter
4,629 PointsWhat kind of code are you using for that?
Christian Lüthy
18,715 PointsChristian Lüthy
18,715 PointsI've written some jquery that toggles classes on the images and the p tags for the image captions:
javascript:
div of 1 image in the markup:
in the css I just changed the percentage of the height of the image in the .photo class from 61.8% to 100% in the .photoBig class.
Holt Hunter
4,629 PointsHolt Hunter
4,629 PointsYou could try this CSS, all you really need to do is add
transition: all 1s ease;
to each class.This should work. I tested it using the developer tools on Chrome. I am not sure if it will work on all browsers though