Review my new site :D

Jesse

Jesse

Active Member
Messages
1,360
Reaction score
0
Points
36
Hello Guys and Girls...

I have opened up a Audio Search Engine and a Lyrics Search Engine...

Hmm... Im improving on the layout... any suggestions would be appreciated...

URL is:
Code: 
http://imusicz.info/
 
J

jjpeacha

New Member
Messages
125
Reaction score
0
Points
0
I really like the "dock" effect at the top of the site, although it could be a little more pronouced cus at first I didn't know what it was there for. Also, maybe the site could be centered cus at the moment its sitting slightly to the left of my browser window.

Appart from that its really quick to load, and the graphics are totally sweet!
 
S

sonicsshadow

Member
Messages
200
Reaction score
0
Points
16
Wow, I really like the menu at the top and the logo. It looks really great.
The site is also very useful.


Also, the search button and search text field could use some new styles to make them look more stylish.

Other than that, great site.
 
Last edited:
Jesse

Jesse

Active Member
Messages
1,360
Reaction score
0
Points
36
Hmm yeah im improving on the layout... (im trying even though im not good on CSS)

I also know that the site does not look good on IE... im finding a way to fix it... if anyone here knows, can u tell me? :D
 
rockee

rockee

New Member
Messages
120
Reaction score
0
Points
0
The header and body is very mixed up and does not conform to much at all really.

XHTML = all tags are lowercase even those JavaScript onload tags.

Too many head and body tags through out the index page.

No & in the title tag - use full words and capitalize the title - I have dropped the 's as it's meaning is not as you have intended - you are using a plural of MP3 (MP3s) not something belonging to MP3 as in MP3's what (sound perhaps).

Here is a modified and XHTML 1.0 compliant header:
Code: 
<!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>iMusicz.info - Download And Listen To MP3 Music</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<link href="dockstyle.css" rel="stylesheet" type="text/css" />
<link href="style.css" rel="stylesheet" type="text/css" />
<script type="text/javascript" src="js/interface.js"></script>

<script type="text/javascript">
<!--
function popitup(url) {
	newwindow=window.open(url,'name','height=300,width=400');
	if (window.focus) {newwindow.focus()}
	return false;
}

// -->
</script>

<style type="text/css">
<!--
.style6 {font-size: 7pt}
.style7 {font-size: 7px}
.style8 {font-size: 8px}
.style9 {font-size: 9px}
.style11 {font-size: 11px}
.style12 {font-size: 11pt; }
-->
</style>

<!--[if lt IE 7]>
 <style type="text/css">
 div, img { behavior: url(iepngfix.htc) }
 </style>
<![endif]-->

</head>

<body onload="init()">

[I]Followed here by all the body code minus any more of those head and body tags.

If you need to add more styles and scripts to the head section then cut and paste following
the format I have included for you - keeps it nice and tidy and so easy to follow and edit.[/I]
The above header now passes W3C standards for your declaration.

Remove any more <head> and <body> tags from the body of your html code.

Check the completed cleaned up index page at the W3C site link above - it is very good for learning such things.

Here is how your header looked before any changes:
Code: 
<html>
    <head>
	<title>iMusicz.info - Download & Listen MP3's</title>
	<LINK href="style.css" rel="stylesheet" type="text/css">

<SCRIPT LANGUAGE="JavaScript">
<!--
function popitup(url) {
	newwindow=window.open(url,'name','height=300,width=400');
	if (window.focus) {newwindow.focus()}
	return false;
}

// -->
</script>

    </head>



   <body onLoad="init()">

	<p align="center"><span class="sidebar">
<!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>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />

<title>Untitled Document</title>
	<LINK href="dockstyle.css" rel="stylesheet" type="text/css">
	<script type="text/javascript" src="js/jquery.js"></script>

<script type="text/javascript" src="js/interface.js"></script>

<!--[if lt IE 7]>
 <style type="text/css">
 div, img { behavior: url(iepngfix.htc) }
 </style>
<![endif]-->

<link href="dockstyle.css" rel="stylesheet" type="text/css" />
<style type="text/css">
<!--
.style6 {font-size: 7pt}
.style7 {font-size: 7px}
.style8 {font-size: 8px}
.style9 {font-size: 9px}
.style11 {font-size: 11px}
.style12 {font-size: 11pt; }
-->
</style>
Plus you have more <head> and <body> tags in your main page code.
Very confusing for me and so would be even more confusing for the browsers.


You can also have your CSS checked and validated at this link and this will also help with your mastering of CSS:
http://jigsaw.w3.org/css-validator/


If I have missed anything it is because my time on your code was limited.

If you correct your coding then I will have another look at your index page again.

The site has promise and no doubt you will be continuously tweaking it to perfection.

Good luck with it.

Regards,
Rocky
 
Jesse

Jesse

Active Member
Messages
1,360
Reaction score
0
Points
36
Thanks Rocky... But it seems error when i applied the fix you showed above... any help?
 
rockee

rockee

New Member
Messages
120
Reaction score
0
Points
0
Jesse said:
Thanks Rocky... But it seems error when i applied the fix you showed above... any help?
Click to expand...
Only if you provide more info about the error or else I can only hazard a guess - and it was not intended as a fix more of a pointer for you to address the many HTML tag issues that will fail your declared document type, which affects how a standards compliant browsers sees your page.

Only you can address these issues at source if the pages are generated by a script you use.

If you coded the pages yourself then you have done so incorrectly and if you want me to help then please read on...

...attach the actual html page source code please, as I used my browser's source code view which may not be accurate if the page is generated by a script - I will then take a closer look at why there are multiple and incorrect html tags throughout the index page.

Regards,
Rocky
 
kkenny

kkenny

Active Member
Messages
1,950
Reaction score
0
Points
36
Pretty good bar on the top, it was pretty cool how you did that effect. Overall the page is pretty nice, it's clean simple and uncluttered. It makes it pleasurable to be viewing the site :) Nice job!
 
rockee

rockee

New Member
Messages
120
Reaction score
0
Points
0
Just to add to my previous post, if you want to check out how good your html code is then visit http://validator.w3.org and type in your site address.

I see Failed validation, 74 Errors, which is way far too many.

Regards,
Rocky
 
You must log in or register to reply here.
Top