Need an external opinion on a new theme

by lukedidit

Posted: 12 years ago 5 replies

WEB DESIGN

Hello All,

I have been tweaking away with a new theme, and I don't trust myself as an indicator of what it will look like to an end user:

Pro Internet Marketing

Is there to much text based areas? Perhaps a cluttered layout?

P.s. Ignore the logo, that is due replacement very soon. I am interested in what you think of everything below the header?

Thanks:

#external #opinion #theme

zulfnore 12 years ago

I think the design and content are just fine.

The only problems I can see are in the css mark up...

You've disable the body { background: element with the /* but did not close it off and it therefore cancels out all the elements underneath it - the result is now you have an unwanted underline on the links.

A suggestion would be to remove the sidebar top padding (set it to 0 instead of 7px) to bring it in line with the content area> Gives it a better look aesthetically
- [ 1 ] Thanks
Signature

WP Strap Code - Cracking the WordPress + Bootrstrap Code
{{ DiscussionBoard.errors[5570310].message }}
xtrapunch 12 years ago

Remove the "underline" from hyperlinks. It will make the site look less cluttered. Leave some space between content and images.
- [ 1 ] Thanks
Signature

>> Web Design, Wordpress & SEO - XtraPunch.com <<
Web Design & SEO Agency | Serving World Wide from New Delhi, India
{{ DiscussionBoard.errors[5570354].message }}

lukedidit

12 years ago

oh wow - well spotted guys.

I must have fluffed up an edit - luckily its running as a child theme,

I will fix this now.

Other then that you think it looks ok?

zulfnore - can you enlarge up the sidebar padding suggestion, sounds very interesting.

This is the code:

/* sidebar
------------------------------------------ */
.widget {margin-bottom: 2em;}
.widget h3 {margin-top: 0; padding-top: 0;}
.widget select {width: 270px; margin: 5px 0;}
.widget ul {list-style: none; margin-top: -1px; padding: 0;}
.widget ul li {margin: 0; padding: 5px 0; border-top: 1px solid #c4d6e5; background-color: #fff; }
.widget ul li {padding-left: 15px; background: url("images/arrow1.gif") 0 10px no-repeat; }
.widget ul ul {margin: 5px 0 -5px;}
.widget ul ul li {background: url("images/arrow2.gif") 0 10px no-repeat; }
.widget ul ul ul li {background: url("images/arrow3.gif") 0 10px no-repeat; }
.widget ul li a:hover {color: #014e8f;}
.widget ul li a:visited, .widget .tagcloud a:visited {color: #014E8F;}
.widget ul>li.current-menu-item>a, .widget ul>li.current_page_item>a, .widget ul>li.current-cat>a {color: #000; font-weight: bold;}

.sidebar form {width: 290px; margin: 15px 0 0; padding: 0; border: none; background: none; }
.sidebar form.searchform input {width: 230px;}

Thanks

{{ DiscussionBoard.errors[5570454].message }}

zulfnore

12 years ago

Sure.

line 99 of style.css reads...
.sidebar {float: right; width: 293px; margin-bottom: 20px; padding: 7px 20px 0 0;}

change it so that the padding reads... padding: 0 20px 0 0
this should bring the sidebar up a notch and line it up with the content area.