new site - opinions?

[Jonathan]

Well after working for bout 2 weeks (good pace, not urgent though),
then the current PHP-Nuke site getting hacked I speed it up.

Anyways ~ xcept for the order page (working on it as we speak),
I've basicly got it all setup and ready to go.

l a b s . g a m e r d e s i g n s . c o m / l p h c

For ClientCP, user and password are both 'test'
( please do not send out the support requests )

I think its very basic, yet excellent;
sober and professional, yet stylish

O and its all 100% XHTML STRICT valid
You guys know how hard that is with PHP!?!?!?!

[Buddha]

I like it. Lot quicker response than that PHP-Nuke thing. I like the simple layout and color scheme too.

[Jonathan]

thanks

[Duane]

page not found

l a b s . g a m e r d e s i g n s . c o m / l p h c
page not found

[Jonathan]

Try this--

G a m e r d e s i g n s . c o m / p o r t f o l i o . p h p
Click the far right side one

[hmp]

Try this--

G a m e r d e s i g n s . c o m / p o r t f o l i o . p h p
Click the far right side one

Hi Jonathan,

I'm new here (just signed up for an account today). I was browsing the forums and this one caught my eye as I'm very much into web design.

I looked at your site, and wanted to give my comments. Please bear in mind that I'm only trying to help, and am not at all trying to criticize you in a bad manner.

I'm only commenting on the actual underlying design, not the visual aesthetics.

A few notes:
1) You have a many "inline styles" in your code, along with an external stylesheet. When I say "inline styles", I mean this:
<div style="width: 100%; text-align: center;">

This can be avoided by giving that <div> a class or id name that you can then assign attributes to in the stylesheet. This will make the code cleaner, and further separate your content and structure from presentation.

2) You are using tables for your navigation list. The same effect that you are trying to achieve can easily be done with an unordered list (<ul>)

Tables are best used for their intended purpose: tabular data. Although you could theoretically keep your navigation as a table and be fine, it is much cleaner (and semantically correct) to use a list. Additionally, it will look better across a variety of platforms (such as devices with small screens)

I'd recommend the following tutorial to achieve this effect:

http://css.maxdesign.com.au/listamatic/horizontal01.htm

3) I would also recommend using headings and other html elements that are not present in your code. For example,
<div class="header">Welcome to...</div>

could be done as
<h1>Welcome to...</h1>

To take this further, let me show you, what, I believe would be a cleaner and more optimized version of some of your text:

before:
<div class="normal"><a href="URL" title="SITE" class="crlink">
SITE</a>, description <br />
dedicated to low cost, high quality webhosting for everyone.

<br /><br />

<a href="URL" title="SITE" class="crlink">
SITE</a>, or <a href="URL" title="SITEs" class="crlink">
LPH</a>, believes in excellent <br />

service, with the tools that all sites require to grow -- <br />
PHP, mySQL, Apache, excellent support, and low cost.

<br /><br />

So, please take your time to browse our site; <br />
we truly hope you enjoy it and find what you need.</div>



<br /><br />
<br /><br />

new version

<h1>Welcome to...</h1>
<p><a href="URL" title="TITLE" class="crlink">SITE</a>, a web hosting service dedicated to low cost, high quality webhosting for everyone.</p>

<p><a href="URL" title="TITLE" class="crlink">SITE</a>, or <a href="URL" title="TITLE" class="crlink">SITE</a>
LPH</a>, believes in excellent service, with the tools that all sites require to grow -- PHP, mySQL, Apache, excellent support, and low cost.</p>

<p>So, please take your time to browse our site;
we truly hope you enjoy it and find what you need.</p>

You are using <br />'s to make the paragraph a certain width. Why not just put something like the following in your css file:

p {
width: 200px;
}

or whatever your desired with may be. And to make more space between paragraphs, you can use:

p {
margin-top: 10px;
margin-bottom: 10px;
}
or something to that effect.


There are some other minor issues as well that I would be happy to help you with if you'd like to PM me. I see that you have helped alot of people here and I'd be more then happy to help someone that has done alot for this community that I have just recently joined.

Looking forward to your comments.

[Jonathan]

1. For the love of bob please don't post my URL or site name

2. On the navigations issue, I perfer tables; not sure why though,
I used to hate them with such a passion...o well..
Anyways ~ I'm using XHTML Strict, which does not support UL (or types of it):

http://www.w3schools.com/tags/tag_ul.asp

Plus, even if I were to convert to XHTML transitional, it'd invalid my code

3. True ~ I have many inline styles; this is because (and it could've been FireFox),
when I tried to put that in the stylesheet ~ it wouldn't work as I wanted it too

4. the <h1> or simular <h#> tags are not supported in XHTML Strict,
and again ~ if I did Transitional, it'd invalid my code:

http://www.w3schools.com/tags/tag_hn.asp

5. Actually the <br /> for the height issue, but I think
you put width instead of height, as I see a margin-buttom tag


<EDIT>Btw thanks for that comment ~ I thought the only thing
I did was get people to get angry @me (I guess I was wrong lol)</EDIT>

[kieran]

You may want to re-read those pages.... <h1> through <h6> and <ul> tags are not only supported, but encouraged as proper document structure in xhtml 1.0 transitional, strict and xhtml 1.1 alike.

Cheers,

Kieran

[Jonathan]

Many thanks for pointing this out!

[waltonics]

Hey there, Im new, don't mean to start with negatives, just got led into this forum.

I like the site. One point I have to reply to, Outmost is a word, yes, but the word you are looking for is actually 'utmost'.

And, just cos my modem is slow today, you should perhaps change the div cell background-color behind the 'grungey creme' background to a creme as well... That way you don't have to wait for the background to load before being able to read page text.. forms the 'structure' of the site quicker to the viewer I think too.

just two cents.

[Jonathan]

Believe me, I'd start off with negatives for my site..
I've been planning a redesign for awhile now, just never
got to it. But I do like feedback so I know where I went
wrong the first time