Review: Computerbuddy.org

[amxcorey]

Please take the time to review computerbuddy.org

All of your help is appreciated. Be Nice

Thanks
Amxcorey

[OnyxDesign]

I like the lay-out. It is nice and clean.
But it seems like the tabs are not exactly the same blue as the blue under the tabs. It's a slight difference but it's there.

The site isn't valid though. You should make it valid or remove the link.

[amxcorey]

you mean remove the link thats powered by phpld? the sponsored, designed, and powered?

Ill change the tab color.

Thanks

[OnyxDesign]

No I mean this image:
Your site is not valid, so you shouldn't say that it is by placing that icon

[amxcorey]

Actually i just checked the color codes of both the main blue and the active part of the button and they are both the same.

Do you think it would look better if i made the color of the button after you click it the same as the hover color?

Ill chage it now so you can see.

Thanks
Amxcorey

[amxcorey]

oh ok....hmmmm it used to be valid i wonder whatws wrong. any way ill get on it.

[OnyxDesign]

No I don't think that looks better..

Photoshop says that the color of the tab is : #007ab8
And the color underneath is: #007ebc

You should make them both #007ebc or #007ab8

[amxcorey]

ok i changed it how is it now?

[OnyxDesign]

It's fixed now. The two colors are the same .
Also.. maybe lower the search form a bit, I think that will look better.

[amxcorey]

yea i know about the search form. Are you using firefox?

I changed it to be lower in internet explorer but when i see it in firefox its right back at the top.

Do you know how to make it so it stays in the same place for both?

[OnyxDesign]

Yes, I am using firefox...
Can you post the html and css code that you used for the search form?

[amxcorey]

CSS:

/* Search Box */
.searchtext {
font-size: 10pt;
margin: 0;
color:#333333;
font-family: Verdana, Tahoma, Arial, sans-serif;
}
.btn-search {
background-image:url(images/btn-search.gif);
width:75px;
height:22px;
border : 0;
border-style : none;
cursor:pointer;
}

HTML CODE:

<p>
<div align="center" style="margin-top:10px;">
<p>&nbsp;</p>
{php}
if ( isset( $_GET['q'] ) ) {
$sQ = $_GET['q'];
} else {
$sQ = '';
}
{/php}
<p>{* Search form *}
</p>
<form action="{$smarty.const.DOC_ROOT}/index.php" method="get">
<input type="text" value="{$smarty.get.q|default:""}" name="q" size="30" class="searchtext" /> <input type="submit" value="" class="btn-search"/>

</form>
</div>
</p>

[OnyxDesign]

Well you could use this to lower it 10px:

position:relative;
top:10px;

But then it becomes to low in Internet Explorer. So you'll need to hide this rule from internet explorer with a css hack. I used this one several times and it's one of my favourites:

html>/**/body .searchtext {
position:relative;
top:10px; /* .searchtext is lowered 10px but IE doesn't see this.*/
}

This CSS code should do the trick . Don't forgot to also lower the button