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!

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

WordPress

Carmine Maraglio
Carmine Maraglio
29,762 Points

Developed a WordPress Theme - Looking for feedback

Hello!

I developed a WordPress theme and would love some feedback on the design, layout and what not.

site- http://carmine.io/wp-shelby

Thanks

The fixed navbar is a nice touch but without any background while scrolling down it decreases the readability (especially when you're in the post-container), furthermore it should be centred.

What I found odd about the header is that the logo is of a higher z-index and the two anchor tags are not and makes them hide underneath the header text for example.

And to finish off with the navbar, to make our life easier, you could add a fixed button to help us scroll to the top of the website while we're in mobile mode.

You image should be responsive so that it scales accordingly to the browsers size(it breaks past 1300px), I am on a 24' screen at this moment.

The footer text could be in the middle (that's just my preference), when your mobile nav is sliding down it's covering the header text and the picture.

Comments section looks fine when browsed on a mobile device, maybe put some background on certain elements to add a bit more readability ?

I am no wordpress expert by all means, but think about the desktop users as well :)

Carmine Maraglio
Carmine Maraglio
29,762 Points

Thanks for checking it out. I will make some modifications to adjust for the large monitoring sizes and fix the other items you mentioned. The background on scroll would make more sense for the header, thanks for the idea!