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

Catherine Millington
Catherine Millington
3,754 Points

Feedback on my website please

Hi everyone, I have made my website but I know there are some tweaks I need to make in terms of how it re-sizes for different screens and mobiles etc. Please could you have a look on your machines and let me know what I could do to make it better? www.thelunesdale.co.uk

Thanks!

10 Answers

Stone Preston
Stone Preston
42,016 Points

I am not much of a designer but here are some things that jumped out at me. Viewing in chrome on a macbook pro:

your navigation links open the page in a new tab every time, it probably shouldnt.

Id also maybe find a higher resolution photo for the background, its pretty grainy or not use a background photo at all.

The horizontal scroll bar appears about 10px above the bottom of the screen instead of being right up against the bottom of the screen so your footer ends up beneath it for some reason.

On your menu page I would probably center the list of PDFs itself in that container, but not each individual list item (this way the icons line up with each other instead of that 3rd one being unaligned with the rest. It will look a bit neater that way)

I might also change the way those images are stacked right on top of each other in the contact page.

I like the layout and design though and the image galleries are very nice, as well as the twitter and facebook feeds. Very cool and impressive.

Catherine Millington
Catherine Millington
3,754 Points

Thanks for taking your time to look at this and for your feedback Stone. I'll have a look at those!

Emma Price
Emma Price
1,302 Points

Hi Catherine,

Nice website! A couple of things, some of which are reiterating what Stone said. When viewing in Chrome on iMac;

  • Background image is a little pixelated, a higher resolution image would be great
  • Horizontal scroll bar appears about 10px above the bottom of the screen
  • For me, when on the homepage if I click on another page it opens in a new tab, but once this second tab is open and I click on another page, the page loads in the same second tab. So it is something to do with that first click from the homepage
  • If I was being picky, I would take the full stops out from the address and contact details at the bottom of the page and separate with a tab space. As I am from the UK too I immediately decided to see how far you were from me and copied your details into Google Maps but got the full stop in by accident. This might make it even more user friendly when people try to find you quickly for that all important pint!
  • If you want to keep the full stops, just add a space between the full stop and the start of the email address

Otherwise website looks great and I might be passing at some point!

Cheers,

Emma

Catherine Millington
Catherine Millington
3,754 Points

Thanks Emma, that's a great tip regarding the address and I hadn't thought about that.

When I view it on my iphone it doesn't minimize properly but looks ok in portrait view on the ipad. I have the following in my CSS;

@media (min-width: 1px) and (max-width: 600px) { .grid-container > [class^="grid-"] { padding-top: 5px; padding-bottom: 5px; } .hide-mobile { display: none; } }

@media (min-width: 801px) {

and if I'm honest I don't really understand it properly. Do you guys know what do I need to change to make this work?

Rob Magnusson
Rob Magnusson
4,314 Points

Looks like a great start! In addition to the other feedback, I'd strongly consider dropping all caps to improve readability throughout.

Catherine Millington
Catherine Millington
3,754 Points

Hi Emma, yes i did but i'll check my links to it.

Thanks for that Rob, I've been toying with changing the font for ages (especially after watching the topography tutorials - Guil made me feel really guilty at using a caps one). The font doesn't have a non-capitalized version which is really annoying because i really like it. I will think on this one, but i know what you mean.

Catherine Millington
Catherine Millington
3,754 Points

Thanks guys i have already changed the background pic and address bits and you're right it looks heaps better. I think i just got used to looking at it.

stjarnan
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
stjarnan
Front End Web Development Techdegree Graduate 56,488 Points

Hi Catherine, i don't really have any feedback for you, as the only thing i really noticed was that the navigation links opened new tabs which has already been said. Reason that i am still writing an answer is because i wanted to say that i think that you've done a great job, in my opinion the site looks great! I am using a 14" Chromebook, and it's looking good.

I would also like to take the chance to ask: Did you have any prior experience before starting with Treehouse or have you learned from the beginning here?

Catherine Millington
Catherine Millington
3,754 Points

Hi Jonas, thanks for the feedback and I'm really glad it looks good on a 14" as I hadn't checked that. No, no prior experience. I have just done the HTML and CSS sections of the front-end web development track so far and have been able to put this together which is great to achieve (esp proud of the slideshow as Guil doesn't show you how to make one of those on the CSS so I had to pool my understanding of animations to make it - although I still can't get it to work in IE which is more than frustrating). I'm a perfectionist though hence the plea for tweaking tips! Thanks for the confidence boost!