...

View Full Version : simplicity..



select sires
12-04-2002, 07:08 PM
ok, i went with tables and i frames. I'm just going to have a disclaimer saying what needs to be used to view the site (i think its the higher range IE NS and Opera, let me know if i'm wrong).
Anyway, I have a few breaks in the page, and i'm not sure where they came from. I basically threw border=0 everywhere i could, tried to kill margins and cellpadding, and they are still there. This is probably terribly simple and I just don't know, any help is appreciated. Feel free to give any feedback.

SS

beetle
12-04-2002, 07:12 PM
page doesn't load for me

select sires
12-04-2002, 07:15 PM
oh man, one sec, my ftp is being screwy

new link:

problem (http://filebox.vt.edu/users/lkornhau/main.html)

beetle
12-04-2002, 08:28 PM
I don't mean to be harsh, but you have GOT to clean up your HTML. It's some of the worst code I've seen in years. Like this
<!-- BEGIN DATE MESSAGE -->
<font size=1>
<span class="font8"><span class="white">Three inline elements (including the deprecated <font>) that can be accomplished with one. Besides the fact that you don't have the classes font8 or white in your stylesheet. Or this
<BODY text=#2F5D93 bgColor="#E5AD72" leftMargin=0 background="2.1angelsplash.gif" topMargin=0 marginwidth="0" marginheight="0">
<body leftmargin="0" marginwidth="0" topmargin="0" marginheight="0" bgcolor="#E9BA87">Two body tags? Each with different attributes? Or this
<img src="bottom.gif" border=0 cellpadding=0 cellspacing=0 topMargin=0 leftmargin="0" marginwidth="0" marginheight="0">An <img> tag does not have cellpadding, cellspacing, or any margin attributes like that. (In fact, it seems like you cut'n'paste these attributes onto almost EVERY tag in your page). Also, there is no reason to put each IFRAME in it's own table. Just use the <iframe> tag. Other things that are wrong (plus some recaps) Deprecated HTML (<font>, <center>) being used
CSS class attributes with no classes defined
improper nesting of tables (between </tr> and <tr> or </table>)
Invalid attributes on img, table, iframe (and maybe other) tags
Attribute values missing double quotes
Two <style> blocks in the <head>
Two <body> tags
Missing closing tags for spans and tds (maybe others)
Uppercase HTML tags
No title
superfluous use of &amp;nbsp;sAnd to be honest, maybe more. I'd suggest getting a beginner book on writing proper HTML first so that you can recognize the errors here as I have recognized them.

Oh, and follow this link (http://validator.w3.org/check?uri=http%3A%2F%2Ffilebox.vt.edu%2Fusers%2Flkornhau%2Fmain.html&charset=iso-8859-1+%28Western+Europe%29&doctype=HTML+4.01+Strict) to see what the W3C validator thinks of your HTML...

Roy Sinclair
12-04-2002, 08:32 PM
Your html is messed up,

You have:


<table>
<tr>
<td>somecontent</td>
</tr>

<table>
...
</table>

...


That is illegal, any content embedded inside a table MUST be inside of a <td> which is inside of a <tr>.

Since the embedded tables are single row and single element tables the whole table declaration is unneeded, at worst you only need place the content inside a <td> which spans all the columns of your table instead.

Use a validator service to validate
your page (http://validator.w3.org/check?uri=http%3A%2F%2Ffilebox.vt.edu%2Fusers%2Flkornhau%2Fmain.html;verbose=1;doctype=HTML%204.01%2 0Transitional;charset=utf-8%20(Unicode%2C%20worldwide))

Roy Sinclair
12-04-2002, 08:38 PM
Hah! Looks like Beetle had the same ideas (but was much more verbose and also beat me to the punch).

BTW visually it's a nice enough page, it just needs the html code cleaned up and it should come together for you.

Vladdy
12-04-2002, 08:43 PM
One more suggestion: load background AFTER the page has loaded using body onload event.
I'm on DSL and still was losing patience. (I foresee the replies "change sevice" :D and "take it easy on coffee" :D so try to be original :D )

If you really want to do it right lose frames and tables and go with css

cg9com
12-04-2002, 08:50 PM
http://www.cg9com.web1000.com/smilies/whoa.gifdont get to overwhelmed, you can learn correct markup here:
http://www.w3schools.com/html/default.asp

i always take steps in my html

start with your BASIC setup

<html>
<head>
<title>title</title>
</head>
<body>

</body>
</html>

work from there, and read that link
good luck

select sires
12-05-2002, 12:20 AM
haha oh man,
i appreciate all the comments, and i'll look into all the comments and links, hopefully cleaning up to Beetle's standards.:o

thanks again, i'll probably be back with an update.

one extra:
Vladdy: I'm a little confused how that works. If I add that, it would make the page wait until everything is loaded?

SS

select sires
12-05-2002, 05:28 AM
aight..attempted.

updated (http://filebox.vt.edu/users/lkornhau/main.html)

cg9com
12-05-2002, 08:07 AM
looking better, you wrote this:

<style type="text/css">
body {
font-family: "verdana";
color:"#A78969";
font: 10pt;
background: url("2.1angelsplash.gif");
}
body { scrollbar-arrow-color: #9C9C9C;
scrollbar-3dlight-color: #E6E6E6;
scrollbar-highlight-color: #FBF8E8;
scrollbar-face-color: #E5AD73;
scrollbar-shadow-color: #E6CD94;
scrollbar-track-color: #E5AD72;
scrollbar-darkshadow-color: #D3C198; }
</style>


this has 2 body sections, just combine all the info inside the second {} and place them inside the first one like this:

<style type="text/css">
body {
font-family: "verdana";
color:"#A78969";
font: 10pt;
background: url("2.1angelsplash.gif");
scrollbar-arrow-color: #9C9C9C;
scrollbar-3dlight-color: #E6E6E6;
scrollbar-highlight-color: #FBF8E8;
scrollbar-face-color: #E5AD73;
scrollbar-shadow-color: #E6CD94;
scrollbar-track-color: #E5AD72;
scrollbar-darkshadow-color: #D3C198; }
</style>


the sections in red, you have font family, but no actual family, just one font, and the second part you have font: this is shorthand, you can just combine the two red sections as well like this:

font:10pt verdana


also you have written your time script inside the <table> and <tr> place it above the <table> tag.

those are the few things i spotted at a first glance, it looks alot better though, good job http://www.cg9com.web1000.com/smilies/smile.gif

select sires
12-05-2002, 11:31 PM
gotcha, that makes a lot of sense. I think i've started learning how to incorporate css mainly by seeing how it is used on other sites, so I often overdo it, or misuse out of ignorance. Thanks for heading me in the right direction.

One question: I have those breaks in the main section of the page, right above the links, and below the main iframe, I'm not sure what needs to be done to get rid of them, any advice is appreciated.

SS

cg9com
12-06-2002, 04:59 AM
thats how i started learning 2 , from other webpages :)

as far as the spacing, what exactly do you mean?
i cant see what your talking about, maybe because its hidden with the colors or something(?) ....

beetle
12-06-2002, 05:22 AM
Spacing like that usually occurs because of linebreaks between an inline and a block level element....such as

<img /> <-- This space right here
<table
...
</table>

The only solution is to put all the code on one line

<img /><table>
...
</table>

Or wrap your inline elements in a block element

<div><img /></div>
<table>
...
</table>

If you can't find the error, let me know, but I'll let you have first crack at it ;)

select sires
12-06-2002, 07:34 PM
haha amazing, i tried putting those elements on the same line and it worked perfect.
i'm going to keep working on the site layout, but i'll keep you guys updated with my problems.:)
but hopefully with something worth looking at eventually.

SS

cg9com
12-06-2002, 08:31 PM
when are you going to add pictures instead of those [X] boxes? ;)



EZ Archive Ads Plugin for vBulletin Copyright 2006 Computer Help Forum