Jump to content

My website need criticism

Minimeallolla

By Minimeallolla
in Website Critique

 Share

Recommended Posts

nano Member

nano

Posted
Posted
This website is best viewed in: Mozilla Firefox

 

Awesome  8)

 

For a first website, it's not too bad but there is a lot to learn. Though my first site was one static image, so you're doing well ;)

 

You need to think about structuring your code, make sure you separate style from content.

 

You need to think about optimisation, don't use a lot of large images and if images are required, use a tool like smushit to optimise them: http://www.smushit.com/ysmush.it/

 

Check you code is valid and that means both the HTML and CSS.

 

The background is a little off putting so maybe use a natural colour (though I am no designer), and make sure your site advertises your brand and online purpose.

 

And finally - make sure it works with multiple browsers. Test, test and test again with something like Spoon:

 

http://www.spoon.net/browsers

 

Good work though!

 

Link to comment
Share on other sites
ManiacDan Regular Member

ManiacDan

Posted
Posted

The first thing that jumped to my mind was usability (entering a bad password doesn't allow you to try again) and copyright concerns.  Even though freeOnlineGames.com is free for you to play, that doesn't necessarily mean the games are free for you to take and put on your own site.  With the DMCA and ACTA out there, a company can have your internet connection disabled just for looking at them sideways.

 

-Dan

Link to comment
Share on other sites
ManiacDan Regular Member

ManiacDan

Posted
Posted

The lime green "you are logged in" page did permanent damage to my eyes.

 

Also, you don't verify email addresses, as you can probably tell by the testy@mctesterson.com that's now in your database.

 

I'm unclear what the whole "comments" thing does, I posted comments and nothing happened.  Also, you cannot comment on individual games.

 

The whole thing is kind of...ugly.  A graphics designer or a cleaner plan for the site's flow would help.  The over-wrought graphics (which are nearly impossible to read), the poor background colors, and the low contrast text make the whole thing difficult to use.

 

Also, your navigation is screwy.  No reason to have "logout" when you're not logged in, or "login" and "register" when you're already logged in.

 

The left nav for the games should be a true left nav and not just tacked to the left side of the bottom of the page.

 

I tried a couple of script and injection attacks on the site, you seem safe from the basics.

 

You set two cookies, one of which is my username and the other is an md5 of my password.  Don't do that, it's obvious to anyone with a modicum of skill what it is.  If you're using those cookies for authentication, DON'T. 

 

-Dan

Link to comment
Share on other sites
Minimeallolla Regular Member

Minimeallolla

Posted
  • Author
Posted

Dan:

you must have got me when I was just experimenting with random stuff.

I can assure you the comment thing does work and the background is set as one throughout the whole site. I do need advice one what background to have. I'm no good with backgrounds >.<

I also didn't want to add a verification email because it would annoy people and from experience I haven't gone through with some sites because I simply could not be bothered to check my emails.

Link to comment
Share on other sites
ManiacDan Regular Member

ManiacDan

Posted
Posted

http://www.mysite.giacjr.dino-hosting.net/profiles/abc123.php?u=num

 

Warning: include(1) [function.include]: failed to open stream: No such file or directory in /home/giacjrdi/public_html/mysite/profiles/write.php on line 129

 

Warning: include(1) [function.include]: failed to open stream: No such file or directory in /home/giacjrdi/public_html/mysite/profiles/write.php on line 129

 

Warning: include() [function.include]: Failed opening '1' for inclusion (include_path='.:/usr/lib/php:/usr/local/lib/php') in /home/giacjrdi/public_html/mysite/profiles/write.php on line 129

Link to comment
Share on other sites
wigpip Newbie

wigpip

Posted
Posted

the HTML is not quite valid, use a structure like this with no HTML above the DOCTYPE:

 

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head>

<title>Untitled Document</title>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />

</head>

<body>

 

 

 

</body>

</html>

Link to comment
Share on other sites
Minimeallolla Regular Member

Minimeallolla

Posted
  • Author
Posted

wigpip: Is this write, or does the doc type have to be dead first? cause if i dont put the include ("background.php"): at the very top it, for some reason gets a tiny breakline and looks different to what i want it to. well here is what i changed it to.

 

<center>
<a href="/index.php"><img border="0" src="/1.png"></a><a href="/login.php"><img border="0" src="/2.png"></a><a href="/logout.php"><img border="0" src="/3.png"></a><a href="register.php"><img border="0" src="/4.png"></a><a href="/games.php"><img border="0" src="/6.png"></a><style type="text/css">
<!--
body
{
background-image:url('homebackground.jpg');
background-repeat:no-repeat;
background-attachment:fixed;
}
-->
</style>	
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
"http://www.w3.org/TR/html4/strict.dtd">
<HTML>
<HEAD>
<link rel="favicon.ico" href="http://www.rohan.giacjr.dino-hosting.net/favicon.ico">
<title>
MySite
</title>

<style type="text/css">
<!--
A:link { COLOR: black; TEXT-DECORATION: none; font-weight: normal }
A:visited { COLOR: black; TEXT-DECORATION: none; font-weight: normal }
A:active { COLOR: green; TEXT-DECORATION: none }
A:hover { COLOR: blue; TEXT-DECORATION: none; font-weight: none }
-->
</style>
</HEAD>
<BODY>

	<h4><font color="yellow"><DIV ALIGN=CENTER>

	Number of website views:</font><font color="white">
	010029</h4>

Link to comment
Share on other sites
This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
 Share
Go to topic listing
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.