Need some feedback on this website

wackoblacko

New Member
Messages
7
Reaction score
0
Points
0
Im kind of new to this web design game. Someone look at my site and tell me the flaws please. Feed back would be great:
lapietra.x10hosting.com
 
Last edited:

Zenax

Active Member
Messages
1,377
Reaction score
4
Points
38
First things first, is that the header image doesnt display.

Code:
<HTML>
<HEAD>
    <TITLE>Title</TITLE>
</HEAD>
<BODY bgcolor="138f6a">
<div align="center">
<IMG SRC="[B]Images\title5.jpg[/B]" USEMAP="#title3" WIDTH=780 HEIGHT=138 BORDER="0">
[I]<MAP NAME="title3">
     <AREA SHAPE="RECT" HREF="body.html" TARGET="body" COORDS="468,106, 503,120">
     <AREA SHAPE="RECT" HREF="contactform.html" TARGET="body" ALT="Schedule an Estimate!" COORDS="375,105, 453,122">
     <AREA SHAPE="RECT" HREF="mycalendar.html" TARGET="body" ALT="See our schedule!" COORDS="283,105, 360,121">
     <AREA SHAPE="RECT" HREF="commercial.html" TARGET="body" ALT="Commercial/Industrial" COORDS="191,105, 271,122">

     <AREA SHAPE="RECT" HREF="residential.html" TARGET="body" ALT="Residential Projects" COORDS="97,105, 177,124">
     <AREA SHAPE="RECT" HREF="about.html" TARGET="body" ALT="About Us!" COORDS="5,105, 88,123">
</MAP>[/I]
</div>
</BODY>
</HTML>

First you should remove the map and replace with Hyperlinks.

Code:
<a href="body.html" target="body">Main</a>

Second of all make sure the text in bold actually exists in an Images Folder.

I feel that the BBB logo part should be included in the header frame. This will make it look more professional.

I would also like to see the removal of frames. I recommend using DIVS or Tables for your layout. DIvs are the Web 2.0 standard which basically means newish.

I hope this helps you out!
 

Chris73

New Member
Messages
5,356
Reaction score
1
Points
0
The reason your header image isn't displaying because you used a back slash \ instead of a forward slash / .

/Images\title5.jpg needs to be this Images/title5.jpg
 
Last edited:

wackoblacko

New Member
Messages
7
Reaction score
0
Points
0
why do you not like frames? I use them because you dont have to reload the whole page all the time. but i see that not alot of sites are using them now. so there must be something to that. thanks for the good advice.
 

Chris73

New Member
Messages
5,356
Reaction score
1
Points
0
I have to agree. I tend to use tables. They hold the content plus you can either make them with a border or without. Keep everything lined up. They can be set to expand according to the content or can be set to be a static size and make the content fit inside.

<divs> work well also if you are fairly good with page location or you can adjust it by trial and error.
 

Dazz

New Member
Messages
371
Reaction score
0
Points
0
Nice site all looks good now, just had a look and logo etc displays ok.

The reasons not to use frames are that some browsers have trouble loading them depending on how they are set up and also its bad for the search engines, apparently it confuses the spiders and they dont know which page to index and so your ranking either goes down or drops off altogether.
 

legendphil

New Member
Messages
152
Reaction score
0
Points
0
hao!

hmm.. seems fine, not much of a design (<-lessens design flaws), but it's working. the content's fine too. I suggest that you use a CMS for professional stuff.

I'll be in the construction biz some 2 years from now (I think), I'm a Civil Engineering student... but that's off the topic.

nice site!
 

chewett

New Member
Messages
137
Reaction score
0
Points
0
you have a really good site as it seem very good

but

if you have the frames you can really mess it up with the wrong code

you could end up with a looking frame

constantly making a new one inside itself
 

mildlyhotpeppers

New Member
Messages
16
Reaction score
0
Points
0
You have good content so far...but as people have said, try to avoid frames. You can easily have the banner across every page by using PHP includes.

If you don't know what PHP includes are, basically you use PHP (it's a language) to copy and paste the same segment of code. You would save the code needed for the top (it appears you named it title4.html already). Then, you would make a file with the extension .php. At the top, you would begin it with
<?php include("title4.html"); ?>
and the rest of the page would just be normal html. Be careful, though, because it will include EVERYTHING in title4.html, including the <head> tags. What I do is keep the file (title4.html, for example) with only the <html>, <head> tags and a starting <body> tag:
<html><head>whatevergoeshere</head><body>
and in my main file I would have
<?php include("title4.html"); ?>Content<?php include("footer.html");?>
footer.html would have the closing </body> and </html> tags, and when the user visits that php page the script would generate
<head>whatevergoeshere</head><body>Content</body></html>
which is a basic but complete HTML file (generated from PHP).
Using this method, if I were to change the title4.html file, the header on every page that included it would change as well. This is extremely convenient for updating and maintaining a website, and very useful for maintaining a uniform look. (Which replace the need for frames.)
On my website, my header and footer are much more elaborate, but the idea is the same. Every page has only content in it, and the layout is achieved by including a header.html and a footer.html file.

Just a though. :biggrin:
 

wh00p101

New Member
Messages
241
Reaction score
0
Points
0
it might even be easier if you generate the header and footer in php instead of html. It would allow them to slot in a lot easier.

site looks good although I found the images weren't up for long enough before they changed.
 

rickle42

New Member
Messages
59
Reaction score
0
Points
0
Looks great! I like that the "frames" dont have to reload unless they need to. The only other way to get this seemless effect is with ajax. If your new it is kind of tuff. Not really hard to pick up for a programmer though.
Also, your email form appears to have no form validation, They should not be able to enter 44 for name and ddd for email right?

Good job!!
 
Top