Review the Code

[Tree]

http://projects.neoncube.net/Adam

I don't really care about how the design looks, it's for a client, and that's how he wanted it. All of the experienced developers, take a look at the code, see if it all looks good to you.

http://projects.neoncube.net/Adam/index.php is the page
http://projects.neoncube.net/Adam/style.css is the CSS

Thanks.

 

[iBrightDev]

it all seems to be in order, but since you cant actually view the php source sice once a page is displayed in html, it does not show the code as typed originally. but from what i can tell, it looks ok. i cant believe the client would actually want a site that looks like that. bad tast. but oh well. please him i guess.

 

[Tree]

In that page there's no actual PHP. There will be soon, though.

 

[Bryson]

I didnt look at the code, but I can tell you there isnt much to it I bet...

Why would they want to do worry about the code, when you can manipulate just about anything with a style sheet anyway? You can open it up even more, and make it look cleaner.

I just dont think people understand what they are up against when it comes to websites anymore, thier not an instant penthouse in a market of run down shacks. Content, functionality, and looks are all very important.

Sorry, you were asking about the code though, but on the surface, I wouldn't want to look at the code.

 

[Bryson]

line 31 column 1 - Warning: missing </table> before </body>

 

[Tree]

Well, it's exactly what a client wants. He drew me a paint drawing, and I followed it to the T. It's not the main site page, it's a backend ad administration tool.

http://projects.neoncube.net/Adam/pages/Index/verybadpaintdrawing.bmp

Thanks Bryson. He wants it to be HTML and CSS compliant, so I am. I wanted people to look at the code just to see if it's standard HTML with CSS, since I haven't done too many CSS designs.

 

[Bryson]

is this some sort of indexing site or something? ie, quotes, stocks, or is it something to do with exchange rates?

Nevermind, something to do with ages by country...

Perhaps a demographic of profit vs gender vs country vs age?

Not a bad idea if thats something close to what it is.

 

[Tree]

It's a targeted ad management program. The admin sets up bids and the advertisers can see the current bids and the demographic to whom they apply. I've set up the login now, username and password are both "foo".

 

[_AndY]

lol incorrect un/pw returns a mysql lookup error XD