The Web Design Group

... Making the Web accessible to all.

Welcome Guest ( Log In | Register )

 
Reply to this topicStart new topic
> www.nobrainerdeals.net
html_guy
post Feb 3 2007, 08:14 PM
Post #1





Group: Members
Posts: 4
Joined: 3-February 07
Member No.: 1,772



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
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
Darin McGrew
post Feb 4 2007, 04:15 AM
Post #2


WDG Member
********

Group: Root Admin
Posts: 8,365
Joined: 4-August 06
From: Mountain View, CA
Member No.: 3



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.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
html_guy
post Feb 4 2007, 02:07 PM
Post #3





Group: Members
Posts: 4
Joined: 3-February 07
Member No.: 1,772



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?
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
Peter1968
post Feb 4 2007, 05:20 PM
Post #4


Serious Coder
*****

Group: Members
Posts: 448
Joined: 23-September 06
Member No.: 213



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.

User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
html_guy
post Feb 4 2007, 05:50 PM
Post #5





Group: Members
Posts: 4
Joined: 3-February 07
Member No.: 1,772



Wow. Thanks Peter. Definitely going to take your advice into account and apply some changes.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
CodeKing
post Feb 23 2007, 10:59 PM
Post #6


Advanced Member
****

Group: Members
Posts: 175
Joined: 12-September 06
Member No.: 118



The first thing I noticed are:
  • CSS file is hosted at another website.
  • Not a single image (Possibly one for the gradient on the logo, but I can't find it)
  • Very smart way to use affiliates! I'm impressed.
  • Boring looking (Spice it up a little bit)
  • Possibly some forms instead of emailing you.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
stjepan
post Feb 28 2007, 10:50 AM
Post #7


Serious Coder
*****

Group: Members
Posts: 329
Joined: 15-October 06
From: zagreb, croatia
Member No.: 445



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

CSS file is hosted at another website.


???????? huh.gif
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
Frederiek
post Feb 28 2007, 12:08 PM
Post #8


Programming Fanatic
********

Group: Members
Posts: 5,146
Joined: 23-August 06
From: Europe
Member No.: 9



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
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
stjepan
post Feb 28 2007, 01:41 PM
Post #9


Serious Coder
*****

Group: Members
Posts: 329
Joined: 15-October 06
From: zagreb, croatia
Member No.: 445



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.

This post has been edited by stjepan: Feb 28 2007, 01:43 PM
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
CodeKing
post Mar 9 2007, 11:30 PM
Post #10


Advanced Member
****

Group: Members
Posts: 175
Joined: 12-September 06
Member No.: 118



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.


This post has been edited by CodeKing: Mar 9 2007, 11:31 PM
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post

Reply to this topicStart new topic
1 User(s) are reading this topic (1 Guests and 0 Anonymous Users)
0 Members:

 



- Lo-Fi Version Time is now: 23rd April 2024 - 11:47 AM