Printable Version of Topic

Click here to view this topic in its original format

HTMLHelp Forums _ Site Review Requests _ www.nobrainerdeals.net

Posted by: html_guy Feb 3 2007, 08:14 PM

Hey everyone. This is my second website I've made. http://www.nobrainerdeals.net/

Constructive criticism welcome! It's basically a website that has special offers and sign-up bonuses. Do you feel that it offers a sense of trustworthiness? Anything you would change or add? (It's a pretty new website, hence the small amount of offers.)

Thanks! blush.gif

Posted by: Darin McGrew Feb 4 2007, 04:15 AM

Let's see... A fixed-width design that forces horizontal scrolling in my default browser windows, microfonts, <font>, a very amature-looking contact page... I'm unimpressed.

Posted by: html_guy Feb 4 2007, 02:07 PM

QUOTE(Darin McGrew @ Feb 4 2007, 04:15 AM) *

Let's see... A fixed-width design that forces horizontal scrolling in my default browser windows, microfonts, <font>, a very amature-looking contact page... I'm unimpressed.


Thanks for your input. How does it force horizontal scrolling in your default browser windows? Because your screen resolution is too low or because you open your browsers in smaller windows? And by microfonts you mean my fonts are too small?

Posted by: Peter1968 Feb 4 2007, 05:20 PM

You use a table for your main page layout. Why? It's fixed width.

That's why it forces horizontal scrolling at less than 700px screen width.

Google for fluid CSS layouts.

Yes, it could be argued that most people don't surf the web in a sized window or at 640x480 but having a fluid layout at least tries to accommodate for folks who do things that way. No, of course you don't have to, but you obviously are trying to make money with this site, right?

Use a doctype and code according to that doctype. Quirks mode is bad.

Specify fonts and sizes in CSS and consign the <font> tag to the WWW museum where it belongs. Set font sizes in percentages and not pixels or points.

Don't use =p and other smilie type stuff on a site that purports to be professional.

Avoid terms like "top deals" etc. First thing I thought of when I saw that, was scam. If you're going to use vogue terms, ensure that they are professional.

Use a proper contact form. There are hundreds of free PHP ones you can adapt to your needs.

Professional businesses rarely communicate via AIM.


Posted by: html_guy Feb 4 2007, 05:50 PM

Wow. Thanks Peter. Definitely going to take your advice into account and apply some changes.

Posted by: CodeKing Feb 23 2007, 10:59 PM

The first thing I noticed are:


Posted by: stjepan Feb 28 2007, 10:50 AM

QUOTE(CodeKing @ Feb 24 2007, 04:59 AM) *

CSS file is hosted at another website.


???????? huh.gif

Posted by: Frederiek Feb 28 2007, 12:08 PM

QUOTE(stjepan @ Feb 28 2007, 04:50 PM) *

QUOTE(CodeKing @ Feb 24 2007, 04:59 AM) *

CSS file is hosted at another website.


???????? huh.gif

http://www.asian-stuff.com/nobrainerdeals/other/mycss.css

Posted by: stjepan Feb 28 2007, 01:41 PM

Hey, html_guy, why's your CSS file on Asian stuff? I noticed that http://www.nobrainerdeals.net/ and http://www.asian-stuff.com/nobrainerdeals/ is the same thing.

Posted by: CodeKing Mar 9 2007, 11:30 PM

QUOTE
Not a single image (Possibly one for the gradient on the logo, but I can't find it)


I finally found it (How did I miss it?) Your CSS and divs are extremely inefficient. You might want to consider changing:

<div id="top">
<div align="center"><img src="http://www.asian-stuff.com/nobrainerdeals/images/main.gif"></div>
</div>

<div id="toptext">
<div align="center">
<span class="mainSiteTitle">No-Brainer Deals</span>
</div>
</div>
</div>

to:

<div id="logo">
No-Brainer Deals
</div>

And remove:

#top {
position : absolute;
top : 0px;
left : 0px;
width : 100%;
}

#toptext {
position : absolute;
left : 0px;
top : 10px;
width : 100%;
margin-left : 0px;
margin-top : 0px;
padding : 0px;
}

and

.mainSiteTitle{font-size:50;font-family:times new roman;color:#ffffff;}
from the CSS and add this:

#logo {
position: absolute;
top:0px;
left:0px;
width:100%;
height:100px; (?)
font-size:50;
font-family:times new roman;
color:#ffffff;
text-align: center;
background: x-repeat url('
http://www.asian-stuff.com/nobrainerdeals/images/main.gif');
}

And then crop main.gif so it's only 1px high.

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)