Website Review and Suggestions

**phb50530** · 02-11-2007, 08:23 AM

Review Needed For A New Webmaster Marketplace:

www.TrickFly.com

Thanks in advance!

**nman14** · 02-11-2007, 03:37 PM

looks great... i like the logo

**phb50530** · 04-03-2007, 10:24 AM

Anymore reviews ?

**Elderm1** · 04-03-2007, 10:55 AM

I like it, but the green on white not a good choice, hurts the eyes other than that i like it

**FindMyPages** · 04-03-2007, 01:30 PM

Are you trying to take over Webmaster-talk.com?

I like your home page, which is easy to read and clean. Don't know why you are doing redirect from your domain to forum pages.

In the actual forum post, it looks a little too busy. Most of the marketing and advertisement are on top which make me thinking that this is a spam site.

Otherwise, I think it's a good web site.

**GreatMaddrin** · 04-04-2007, 06:24 AM

Looks nice!

I don't mind the colours, in fact I quite like them. I have a bit of a loving of green on sites. My old site's primary colour was #b5e4af - mmm!

One thing that I don't like about it, is that it stretches across the entire page - I've never like that. It's more personal preference I suppose. Also, in the resolution of 800x600 the advertising gets cut off. Though I suppose it's quite a small majority who has 800x600, and the site does still work with it.

On the forum I think the topics are to close together and aren't seperated enough with those little white lines. I'd make a bit of a bigger divide and pad them all out a bit.

Nice site though!

**phb50530** · 04-04-2007, 08:53 AM

Thanks I have just added 4px padding - it makes a big difference thanks alot!

Anymore reviews would be welcomed.

**mudi** · 04-04-2007, 01:15 PM

not too much liking the colours my friend

**webcosmo** · 04-04-2007, 05:52 PM

great site. on forum lists you may include a short description telling whats its about.

**phb50530** · 04-04-2007, 08:22 PM

what do you mean forum lists ?

**travelagent** · 04-04-2007, 09:36 PM

Honestly, I don't care for your landing page (main site) - it looks gawdy and unprofessional.

You really should place all of your CSS in a separate file on your server and delete the CSS from within your HTML -- and, why code in tables, when you can do table-less CSS?

Lastly, you might want to fix the following 77 code warnings:

Code:

line 316 column 11 - Warning: <style> isn't allowed in <body> elements
line 375 column 56 - Warning: <img> element not empty or not closed
line 377 column 4 - Warning: <img> element not empty or not closed
line 379 column 35 - Warning: <img> element not empty or not closed
line 381 column 4 - Warning: <img> element not empty or not closed
line 386 column 5 - Warning: missing </a> before </td>
line 388 column 7 - Warning: missing </a> before </td>
line 390 column 6 - Warning: missing </a> before </td>
line 395 column 3 - Warning: discarding unexpected </tr>
line 397 column 34 - Warning: <img> element not empty or not closed
line 316 column 11 - Warning: <tr> isn't allowed in <body> elements
line 401 column 2 - Warning: inserting implicit <table>
line 468 column 8 - Warning: <input> attribute name "/style" (value=" font-size: 130%; margin-top: 2px; padding: 3px 10px; font-weight:bold") is invalid
line 468 column 8 - Warning: <input> element not empty or not closed
line 556 column 41 - Warning: <br> element not empty or not closed
line 559 column 58 - Warning: <br> element not empty or not closed
line 560 column 5 - Warning: <br> element not empty or not closed
line 580 column 1 - Warning: <br> element not empty or not closed
line 601 column 15 - Warning: missing </font> before <p>
line 603 column 4 - Warning: inserting implicit <font>
line 603 column 116 - Warning: <img> element not empty or not closed
line 607 column 48 - Warning: <br> element not empty or not closed
line 621 column 15 - Warning: missing </font> before </p>
line 628 column 15 - Warning: discarding unexpected </font>
line 629 column 15 - Warning: discarding unexpected </font>
line 633 column 11 - Warning: <br> element not empty or not closed
line 639 column 13 - Warning: <br> element not empty or not closed
line 643 column 15 - Warning: <img> element not empty or not closed
line 646 column 172 - Warning: <br> element not empty or not closed
line 654 column 36 - Warning: missing </font> before <div>
line 654 column 15 - Warning: missing </font> before <div>
line 657 column 1 - Warning: inserting implicit <font>
line 657 column 1 - Warning: inserting implicit <font>
line 657 column 1 - Warning: missing </font> before <p>
line 657 column 1 - Warning: missing </font> before <p>
line 658 column 1 - Warning: inserting implicit <font>
line 658 column 1 - Warning: inserting implicit <font>
line 660 column 57 - Warning: discarding unexpected </font>
line 664 column 11 - Warning: <br> element not empty or not closed
line 695 column 15 - Warning: <img> element not empty or not closed
line 698 column 75 - Warning: <br> element not empty or not closed
line 702 column 15 - Warning: <br> element not empty or not closed
line 703 column 15 - Warning: missing </font> before </p>
line 704 column 15 - Warning: replacing <p> by <br>
line 704 column 15 - Warning: inserting implicit <br>
line 704 column 15 - Warning: <br> element not empty or not closed
line 709 column 70 - Warning: discarding unexpected </font>
line 730 column 24 - Warning: <br> element not empty or not closed
line 406 column 3 - Warning: missing </div>
line 405 column 2 - Warning: missing </div>
line 404 column 1 - Warning: missing </div>
line 373 column 1 - Warning: <table> proprietary attribute "height"
line 379 column 35 - Warning: <img> lacks "alt" attribute
line 386 column 5 - Warning: <a> proprietary attribute "a"
line 388 column 7 - Warning: <a> proprietary attribute "a"
line 390 column 6 - Warning: <a> proprietary attribute "a"
line 392 column 5 - Warning: <a> proprietary attribute "a"
line 432 column 1 - Warning: <table> proprietary attribute "bordercolor"
line 459 column 3 - Warning: <table> proprietary attribute "bordercolor"
line 574 column 1 - Warning: <table> proprietary attribute "bordercolor"
line 576 column 5 - Warning: <td> proprietary attribute "bordercolor"
line 582 column 1 - Warning: <table> proprietary attribute "bordercolor"
line 603 column 116 - Warning: <img> lacks "alt" attribute
line 634 column 11 - Warning: <table> proprietary attribute "height"
line 670 column 1 - Warning: <table> proprietary attribute "bordercolor"
line 672 column 5 - Warning: <td> proprietary attribute "bordercolor"
line 674 column 1 - Warning: <table> proprietary attribute "bordercolor"
line 695 column 15 - Warning: <img> lacks "alt" attribute
line 566 column 52 - Warning: trimming empty <p>
line 601 column 15 - Warning: trimming empty <font>
line 604 column 1 - Warning: trimming empty <p>
line 609 column 15 - Warning: trimming empty <p>
line 646 column 74 - Warning: trimming empty <font>
line 654 column 36 - Warning: trimming empty <font>
line 654 column 15 - Warning: trimming empty <font>
line 657 column 1 - Warning: trimming empty <font>
line 657 column 1 - Warning: trimming empty <font>

**phb50530** · 04-05-2007, 08:28 AM

But why should I put my CSS in an external file rather than keeping it on the same page ?

**travelagent** · 04-05-2007, 12:13 PM

Quote:

Originally Posted by phb50530

But why should I put my CSS in an external file rather than keeping it on the same page ?

So that your site loads faster, and your HTML is trimmed down -- it's not mandatory; but most professional designers utilize this method.

**phb50530** · 04-05-2007, 12:51 PM

how do I call my external CSS file ? what code is it ?

**codeofheaven** · 04-05-2007, 01:58 PM

nice site ..

**travelagent** · 04-05-2007, 04:38 PM

Quote:

Originally Posted by phb50530

how do I call my external CSS file ? what code is it ?

Create your CSS file in "Notepad" and save it as something like trick.css -- copy / paste all of your CSS into that file; save it again; and upload to your server.

After you do that, simply enter a line of code similar to this in your HTML:

<link href="http://www.trickfly.com/trick.css" type="text/css" media="screen" rel="stylesheet" />