My First Webpage!

**Defuze** · 12-05-2010, 12:20 PM

I am not advertising just want to share so please don't lock or remove the link.

Here it is. I have spent like all day from 12:00pm from UK. I think it turned out okay I guess. Anyways here it is.

Dave-Ford

Please post your thoughts.

**Downfall** · 12-05-2010, 01:04 PM

all you did was a grey background??

**Nevad** · 12-05-2010, 03:56 PM

Its white for me with some text.

But im guessing you wanna show off that awesome loading intro :P

**Skaryface** · 12-05-2010, 07:28 PM

i got tired of the intro and closed the site. lol

**Dark Method** · 12-05-2010, 07:48 PM

Well..I couldn't do that, but my advice is that if you are going to some sort of loading/flash screen of any sort, make it look decent. I'm not really a fan of them but that's just me. Big corporations sure, other than that, I would skip it generally speaking.

***Pet*** · 12-05-2010, 07:55 PM

The intro takes way too long, and is to... gimmicky to be used for a website. Most loading screens, for flash websites, are a simple loading bar stylized to be attractive, not full coding style text, which could have been executed a lot better than all comment tags without proper opening, or closing. And as for the website itself, is so basic, that I am unclear as to why there was an intro to begin with. Simple background, with a small white square, and text doesn't take too long to load, and it was not flash either, to make full use of a flash intro/loading screen.

If you really want to learn HTML and CSS, focus on the website basics before adding things like intros.

**Monroe** · 12-05-2010, 09:57 PM

Everyone else already hated on the intro, so I'll spare you my hate of it as well. As for the web site itself, there's not much to talk about as far as design goes, so I'll talk about your code since that's what you're trying to learn.

HTML Code:

<html>
<head>
<link rel="stylesheet" type="text/css" href="css\main.css" />
<title>Dave Ford</title>
</head>

<body>
<div class="whitebox">
        
            <h1 class="title1" align="center" ><i>Dave-Ford.co.cc</i></h1>
                <div class="content">
                    <hr>
                    <br />
                    <p class=text1>Hello and welcome to my website, 
                     I am learning html & css. I like to use Photoshop and Illustator. This is 
                     my very first design I have made using NotePad ++, I would like to 
                     thank Jordan for pointing me to some webpages with lots of 
                     information about html & css and Ryan van Zeben for giving 
                     me some tips and just helping me along the way.</p>
                    <br /> Special Thanks to Billy Fielder for the cool intro for my site.
                    </div>
</div>    
</body>
</html>

Why do you have <div class="whitebox">? Is this supposed to be the container div? If so, then you should change the class to an id.

You should also change your <h1> to an id because the h1 element shouldn't be used again. Plus the name of the class is "title1" so I doubt you would use that same class for something else. Save the align attribute for CSS. I would also remove the <i> tags since they are deprecated and use CSS to do that. All you would need to put in your external style sheet for that is:

h1 {font-style: italic;}

I don't recommend the use of <hr> tags, but if you insist on using them then they need to be closed just like the <br /> tags.

Why do you have a class on the first paragraph?

And your text "Special Thanks to Billy Fielder for the cool intro for my site." needs to be in <p></p> tags as well.

I think you need to change "css\main.css" to "css/main.css" as well. Maybe that's why your site looks so plain, because it's not properly linked to the style sheet.

***Chris*** · 12-06-2010, 10:09 AM

I couldn't get the site to even load for me, but I did have one thing I wanted to add.

Originally Posted by Monroe

Why do you have <div class="whitebox">? Is this supposed to be the container div? If so, then you should change the class to an id.

I wouldn't necessarily advise this, even though the page is simple. Reason being, he may want to use the same selector for multiple divs on the same page, should he add more, in which case the id would not be appropriate.

Originally Posted by Monroe

You should also change your <h1> to an id because the h1 element shouldn't be used again. Plus the name of the class is "title1" so I doubt you would use that same class for something else.

I also disagree here. The h1 element can be used multiple times within the same page. It's not an element that really needs to be assigned an id. Header elements in general are only used to assign importance in terms of markup. Sure you could argue that you should have one H1 and then multiple h2's and h3's as subheaders, but the same could be said for having multiple h1's. All it really does is add semantic value to your content. Search engines can index your site easily by seeing what content belongs to what.

Originally Posted by Monroe

I don't recommend the use of <hr> tags, but if you insist on using them then they need to be closed just like the <br /> tags.

How come? HR tags may not be used by many, but it still serves a purpose. On self-closing, it really depends on if you're doing XHTML or HTML4. But the hr element in general is also used in HTML5, and can be styled in CSS, so it's certainly not deprecated. Note, however that it would be self-closing in HTML5.

But as everyone else said, the site itself does need a bit more work done to it. Minimalism is great, but it in itself has some form of style.

**Monroe** · 12-06-2010, 10:36 AM

Originally Posted by Solaris

I wouldn't necessarily advise this, even though the page is simple. Reason being, he may want to use the same selector for multiple divs on the same page, should he add more, in which case the id would not be appropriate.

Seeing as how the class name is "whitebox" I suppose it is possible that he would reuse it, not sure for what. But I was only suggesting he use ID instead if it is meant to be his "container" or "wrapper" div as some call it. For the most part, that div encloses everything else making it one of a kind, so I always give it an ID. Either way does work though.

I also disagree here. The h1 element can be used multiple times within the same page. It's not an element that really needs to be assigned an id. Header elements in general are only used to assign importance in terms of markup. Sure you could argue that you should have one H1 and then multiple h2's and h3's as subheaders, but the same could be said for having multiple h1's. All it really does is add semantic value to your content. Search engines can index your site easily by seeing what content belongs to what.

Generally the title of your page is the h1 element, and all of your other headers are h2's, and their subheaders are h3's and down. That's how I do it at least, definitely not the one absolute approach. The reason I suggested he give that h1 element an ID instead of class is because he called it "title1". That tells me that he means for it to be the title residing in the header of his page which is only used once. It's kind of hard to give solid advice though when there's only two lines of content on the entire page.

How come? HR tags may not be used by many, but it still serves a purpose. On self-closing, it really depends on if you're doing XHTML or HTML4. But the hr element in general is also used in HTML5, and can be styled in CSS, so it's certainly not deprecated. Note, however that it would be self-closing in HTML5.

I suppose this is more of a personal decision, depending on how much you advocate the separation of structure and presentation. To me, the <hr> tag is a presentational element, maybe not for others, but because it is to me, I leave it to CSS. I like adding border-bottom to headers/titles rather than having an <hr> element underneath in the HTML document. As far as it self-closing, I always go with XHTML so I always close every tag. Plus it's a good habit to have whether your using XHTML or not. Plus he self-closed the <br> tags so I figured why not the <hr> ones. It's obvious he's very new to coding, so when giving advice I think it's more beneficial to say what you should do and not just what you have to do.

But as everyone else said, the site itself does need a bit more work done to it. Minimalism is great, but it in itself has some form of style.

.....

***Chris*** · 12-06-2010, 10:38 AM

I understand where you're coming from. I suppose it's important to note the many ways of going about doing something. But I wholeheartedly agree with your comment about there only being so much you can do with 2 sentences. :P

I think a couple good books about one specific language (either HTML5 which is picking up speed) or XHTML might be advantageous.

» Search Forums

» Online Users: 5,424

» Site Navigation

» Stats

Thread: My First Webpage!

Thread Tools

Search Thread

Display

My First Webpage!

Similar Threads

A couple of webpage/layout designs....

My webpage layout design

First complete webpage

Webpage

Posting Permissions