Fake News

Anxiety35's picture

He has: 10 posts

Joined: Feb 2005

I just finished working on a new site of mine. It's my first major attempt at php... by that I mean that I've only done includes before this.
I'd just like some input on if everything is working fine and you like the very basic look.
Also, I realize that the script will let you leave the forms empty... and I plan to add an error soon so that it requires every form to be filled out.

Fake News

Renegade's picture

He has: 3,022 posts

Joined: Oct 2002

Fix the spelling errors and add in another option to specify the gender of the person.

Anxiety35's picture

He has: 10 posts

Joined: Feb 2005

thanks... I had never check that pumpkin story for errors because a friend of mine wrote it for me... and I didn't think to spellcheck it for some reason...
Thanks for the input... I like the gender idea.

Megan's picture

She has: 11,421 posts

Joined: Jun 1999

Hello, Anxiety Smiling Sorry it took me so long to get to reviewing your site. To be honest, I think the design needs a lot of work. I think what you should do is get a template and customize it for your needs. That would be the fastest and easiest way to get a decent looking design so you can concentrate on showing off your programming skills.

Once you get a good template the only hard part will be to put in your site title. Grab a unique and fun looking font. Don't bother trying to put special effects on it or anything, although colouring some letters or playing with sizes might be effective.

One other piece of advice is to avoid centering blocks of text. It looks really unprofessional and makes the text hard to read. Since the lines are start at different spots the eye has to jump back and forth to read from line to line. It's uncomfortable for people.

I hope that's helpful Smiling

They have: 33 posts

Joined: Jan 2005

Hey have you seen your website on a 1400 X 1050 screen resolution. Your banner/header on top is 592 pixels but the main content table looks to be 75%. Therefore your table is bigger then your banner making your site look very wierd. Put everything in a 750 pixel table and change your header imager to 750 pixel width. Other then that you I agree with Megan you site needs alot of work in design.

dojo's picture

She has: 87 posts

Joined: Dec 2004

The banner is from a site that make banners instatly ... used the same style 4 years ago Smiling
You need some work .. a lot of work. The idea is nice, but use a nicer package Smiling

Good luck

Anxiety35's picture

He has: 10 posts

Joined: Feb 2005

alright, thats for the input... I'll work on a new design to make everything look nicer and I'll make my own banner too... I'll post here again once I've re-done it and see what you guys think.

Thanks a bunch

CptAwesome's picture

He has: 370 posts

Joined: Dec 2004

You set the background colour of the form inputs, without setting the font colour so, for people who don't use black on white, it's invisible.

Anxiety35's picture

He has: 10 posts

Joined: Feb 2005

thanks, I never would have thought of that.

CptAwesome's picture

He has: 370 posts

Joined: Dec 2004

I decided to send myself a copy of the fake news, and noticed you just put all the variables in the address line as Get variables. This is of course the easiest method, but it's also the elast effective, since most people will instantly spot their name along the top, amongst all the other details.

I don't know what permissions you have on the server, but you'd be better served to have a database with IDs, or just textfiles, the it could be "article=1244" instead of all the get variables.

Anxiety35's picture

He has: 10 posts

Joined: Feb 2005

I'm not very good with php yet, but I'll work on doing that. That would look a lot more real and professional. Thanks for all your advice, it's helping out a lot.

Want to join the discussion? Create an account or log in if you already have one. Joining is fast, free and painless! We’ll even whisk you back here when you’ve finished.