Jump to content


Photo

my first website


  • This topic is locked This topic is locked
14 replies to this topic

#1 mamba

mamba

    Young Padawan

  • Members
  • Pip
  • 11 posts

Posted 14 April 2006 - 10:21 AM

Hi peeps,

Well i recently finished a website for my driving instructor for an assignment in my college course. Its my first proper attempt at a website and i wana progress and do better ones.

Right i will get to the point its at: http://mini-adventures.net .

Any feedback given would be greatly featured and i could not of done the php form without pixel2life

Thanks :)

#2 Jacorre

Jacorre

    P2L Jedi

  • Members
  • PipPipPip
  • 824 posts
  • Gender:Male
  • Location:USA
  • Interests:Computers, Technology, Internet, Graphic/Web Design, Music, Soccer

Posted 14 April 2006 - 03:05 PM

It's a good start for your first. The link at the bottom for "Top of Page" usually bumps you to the top of the page. Right now it's reloading the page. Try a href=#top since you do have an anchor at the top called top.

Try reading up on CSS too it can help you alot instead of using font tags within the pages. If you had to change the color in the future, you'd have to go to every page and change it. If you were using a CSS file you could change it in one file and the change would reflect in all pages.

#3 makavelimx

makavelimx

    Young Padawan

  • Members
  • Pip
  • 69 posts
  • Interests:Events Manager of MD5Encryption.com

Posted 14 April 2006 - 06:32 PM

Don't compress the picture of the car in the banner as much, there's too much grain. Other than that, its really good for a first webpage.

#4 DanWilliamson

DanWilliamson

    P2L Jedi

  • Members
  • PipPipPip
  • 651 posts

Posted 15 April 2006 - 05:41 AM

It's awesome for a first but I think the rollover is too 'in your face' for me. The mini on the header seems bad quality so I would try and solve that =]

#5 mamba

mamba

    Young Padawan

  • Members
  • Pip
  • 11 posts

Posted 15 April 2006 - 06:01 AM

thx guys that was helpful! :D

#6 ICT Helpers

ICT Helpers

    Young Padawan

  • Members
  • Pip
  • 56 posts
  • Location:England

Posted 16 April 2006 - 07:20 AM

For a first website you should be really pleased with yourself :) You could improve it by using CSS more and getting your code to validate giving you cleaner code and something to be extra pleased with :P

Well done!

Edited by ICT Helpers, 16 April 2006 - 07:20 AM.


#7 mamba

mamba

    Young Padawan

  • Members
  • Pip
  • 11 posts

Posted 17 April 2006 - 05:10 AM

thx for your comment.
I had a look at the validator and i got 17 errors and i am strugling to find out how to correct them, any advice?

#8 ICT Helpers

ICT Helpers

    Young Padawan

  • Members
  • Pip
  • 56 posts
  • Location:England

Posted 17 April 2006 - 07:51 AM

I have re coded your first page which you can use as a template. It validates as XHTML 1.0 Transitional and CSS. I can paste the code here if you want?

#9 mamba

mamba

    Young Padawan

  • Members
  • Pip
  • 11 posts

Posted 17 April 2006 - 08:04 AM

wow thats great, yeh if you dont mind could you post it on here :( :D

#10 ICT Helpers

ICT Helpers

    Young Padawan

  • Members
  • Pip
  • 56 posts
  • Location:England

Posted 17 April 2006 - 08:31 AM

No problem :D

Your XHTML page is
<?xml version="1.0" encoding="UTF-8"?>
<!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" xml:lang="en">

<!-- Created by Andy at ICT Helpers http://www.icthelpers.com -->

  <!-- This is the head of my document -->
  <head>

	<!-- This is my title -->
	<title>Welcome to mini-adventures.net </title>

	<!-- meta content -->
	<meta name="keywords" content="mini driving lessons, mini, mini driving lessons harrogate, driving lessons harrogate, driving lessons knaresborough, driving lessons leeds, driving lessons knaresborough mini, driving lessons leeds mini, learn to drive in a mini, cheap driving lessons in harrogate, cheap driving lessons in leeds, pass plus, pass plus harrogate, pass plus in harrogate, pass plus in leeds, pass plus in knaresborough, intensive driving lesssons harrogate, intensive driving lessons leeds, intensive driving lessons knaresborough, refreshers lessons harrogate, refreshers lessons leeds, refreshers lessons knaresborough." />
	<meta name="description" content="mini Adventures offer a wide range of services for learning to drive or if you have passed your test there are still many services to offer" />

	<!-- load external stylesheet for screen use -->
	<link rel="stylesheet" type="text/css" media="screen" href="/screen.css" />	

	<!-- load external javascript for rollover navigation buttons -->
	<script type="text/javascript" src="javascript.js"></script>

  </head>

  <!-- body -->
  <body>

	<!-- main container -->
	<div id="container">

	  <!-- flash banner -->
	  <div class="banner">
		<object type="application/x-shockwave-flash" data="flashbar/mini.swf" width="745" height="110">
		  <param name="movie" value="flashbar/mini.swf" />
		</object>
	  </div>

	  <!-- navigation buttons -->
	  <div id="navigation">
		<img src="images/home-nav_01-over.gif" alt="Home" title="Home" id="home" /><a href="about.php" onmouseover="rollover_on('about');" onmouseout="rollover_off('about')"><img src="images/about-nav_01.gif" alt="About" title="About" id="about" /></a><a href="contact.htm" onmouseover="rollover_on('contact');" onmouseout="rollover_off('contact')"><img src="images/contact-nav_01.gif" alt="Contact" title="Contact" id="contact" /></a><a href="services.htm" onmouseover="rollover_on('services');" onmouseout="rollover_off('services')"><img src="images/services-nav_01.gif" alt="Services" title="Services" id="services" /></a><a href="links.htm" onmouseover="rollover_on('links');" onmouseout="rollover_off('links')"><img src="images/links-nav_01.gif" alt="Links" title="Links" id="links" /></a>
	  </div>

	  <!-- title section -->
	  <div class="title">
		<h1>Welcome to Mini Adventures Driving School</h1>
		<h2>Very Patient Professional Instructors. Competitive Prices</h2>
	  </div>

	  <!-- main content -->
	  <div id="content">
		<h3>Why choose mini Adventures?</h3>
		<p><span style="float: right"><img src="images/mini/mini(side)1.jpg" width="325" height="188" alt="Welcome to Mini Adventures Driving School" title="Welcome to Mini Adventures Driving School" /></span>We offer a wide range of services which are available in:</p>
		<p><strong>Knaresborough</strong>, <strong>Harrogate</strong>, <strong>Wetherby</strong></p>
		<p>Also in <strong>Barnsley</strong> &amp; <strong>surrounding Villages</strong></p>
		<p>These services include: </p>

		<ul>
		  <li>Driving Lessons,</li>
		  <li>Refresher Lessons,</li>
		  <li>Intensive Lessons,</li>
		  <li>Pass Plus Scheme</li>
		</ul>
	  </div>

	  <!-- includes text -->
	  <div class="includes">
		<strong>Which includes <span style="background-color: inherit; color: #FF0000;">Free</span> help and advice with Theory &amp; Hazard Perception<br />whenever you may need it.</strong> For more information visit the <a href="services.htm">services page</a>.
	  </div>

	  <!-- logos -->
	  <div class="logos">
		<span style="float: left;"><img src="images/mini/minilogo.jpg" width="136" height="69" alt="Mini" title="Mini" /></span><span style="float: right;"><img src="images/mini/pp.jpg" width="88" height="76" alt="Pass Plus" title="Pass Plus" /></span>
		<div style="clear: both; height: 1em;"></div>
	  </div>

	  <!-- navigation text links, Top of Page and XHTML and CSS validation links -->
	  <div class="navigation-bottom">
		<p>:: Home :: :: <a href="about.htm">About</a> :: :: <a href="contact.php">Contact</a> :: :: <a href="services.htm">Services</a> :: :: <a href="links.htm">Links</a> ::</p>
		<p><span style="padding-right: 200px;"><a href="http://validator.w3.org/check?uri=referer"><img style="width:88px;height:31px" src="http://www.w3.org/Icons/valid-xhtml10" alt="Valid XHTML 1.0 Transitional" title="Valid XHTML 1.0 Transitional" /></a></span><a href="java script:scroll(0,0);" title="Top of Page">Top of Page</a><span style="padding-left: 200px;"><a href="http://jigsaw.w3.org/css-validator/check/referer"><img style="width:88px;height:31px" src="http://jigsaw.w3.org/css-validator/images/vcss" alt="Valid CSS!" title="Valid CSS!" /></a></span></p>
	  </div>

	</div>

  </body>

</html>
Your CSS page is
/*

Screen CSS Styles for use throughout my website

Created by Andy at ICT Helpers http://www.icthelpers.com

*/

/* container */
#container
{
  margin-left: auto;							/* Sets the left margin so the content is centered horizontally */
  margin-right: auto;						   /* Sets the right margin so the content is centered horizontally */
  padding: 0px;								 /* 0px padding on the inside of the container */
  text-align: left;							 /* Aligns the text to the left */
  width: 743px;								 /* Sets the width to 743px */
}

/* navigation buttons */
#navigation
{
  padding: 0px;								 /* 0px padding on the inside of the navigation */
}

/* navigation links at the bottom of the page */
.navigation-bottom a
{
  font-weight: bold;							/* Sets the font weight to bold */
}

/* Title, main content, includes text and navigation links at the bottom of the page */
.title, #content, .includes, .logos, .navigation-bottom
{
  background-color: #DFF0FD;					/* Sets the background colour to a pale blue */
  color: inherit;							   /* Font colour is inherited from the body tag */
  padding: 10px;								/* 10px padding on the inside of the navigation */
  width: 723px;								 /* Sets the width to 723px */
}

/* Title, includes text and navigation links at the bottom of the page */
.title, .includes, .logos, .navigation-bottom
{
  text-align: center;						   /* Aligns the text to the center */
}

/* Body XHTML tag */
body
{
  background-color: #A3CFEA;					/* Sets the background colour to a pale blue */
  color: #000;								  /* Font colour is set to black */
  font-family: Arial, Helvetica, sans-serif;	/* Sets font globally */
  margin: 0px;								  /* 0px margin */
  padding: 10px;								/* 10px padding on the inside */
  text-align: center;						   /* centers board in Internet Explorer */
}

/* links */
a:link, a:visited, a:active
{
  background-color: inherit;					/* Background colour is inherited from the body tag */
  color: #993399;							   /* Font colour is set to purple */
  text-decoration: none;						/* no text decoration for links */
}

/* links when mouse over */
a:hover
{
  background-color: inherit;					/* Background colour is inherited from the body tag */
  color: #33CCFF;							   /* Font colour is set to cyan */
  text-decoration: none;						/* no text decoration for links */
}

/* img XHTML tags */
img { border: 0px; }							/* Set a 0px border around the img XHTML tag so no border is displayed */

/* h1 XHTML tags */
h1
{
  font-size: 1.5em;							 /* Sets font size */
  font-weight: bold;							/* Sets the font weight to bold */
}

/* h2,h3 XHTML tags */
h2, h3
{
  background-color: inherit;					/* Background colour is inherited from the body tag */
  color: #000066;							   /* Font colour is set to dark blue */
  font-size: 1em;							   /* Sets font size */
}

/* li XHTML tags */
li
{
  background-color: inherit;					/* Background colour is inherited from the body tag */
  color: #0000FF;							   /* Font colour is set to blue */
  font-weight: bold;							/* Sets the font weight to bold */
}
Your javascript page is
/*

Javascript for navigation rollover buttons for use throughout my website 

Created by Andy at ICT Helpers http://www.icthelpers.com

*/

var imgPath="images/"

// Preload

if (document.images)
{
  var imageUrls = new Array("home-nav_01.gif","about-nav_01.gif","contact-nav_01.gif","services-nav_01.gif","links-nav_01.gif");
  var pic = new Array(imageUrls.length);

  for (i=0;i<imageUrls.length;i++)
  {
	pic[i] = new Image();
	pic[i].src = imgPath + imageUrls[i];
  }
}

function rollover_on(img)
{

  if (document.images)
  {
	document.images[img].src = imgPath + img + "-nav_01-over.gif";
  }
}

function rollover_off(img)
{

  if (document.images)
  {
	document.images[img].src = imgPath + img + "-nav_01.gif";
  }
}
If you need anything explaining let me know :(

If you want, you could also get this to validat to XHTML 1.1 Strict by changing this line
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
to this
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

I have just commented out the code I created for you for easy understanding :D

Edited by ICT Helpers, 17 April 2006 - 12:42 PM.


#11 mamba

mamba

    Young Padawan

  • Members
  • Pip
  • 11 posts

Posted 17 April 2006 - 12:19 PM

kool cheers m8, when i get the time i will use that code :D

#12 ICT Helpers

ICT Helpers

    Young Padawan

  • Members
  • Pip
  • 56 posts
  • Location:England

Posted 17 April 2006 - 12:21 PM

removed as I think it may have been classed as plugging my site - sorry.

Edited by ICT Helpers, 17 April 2006 - 02:17 PM.


#13 mamba

mamba

    Young Padawan

  • Members
  • Pip
  • 11 posts

Posted 17 April 2006 - 12:51 PM

Glad to help and if you would like to add a link to my website on your links page, then that would be appreciated but only if you want to :)


i cant m8 coz its not very professional and its not my site tbh. but i tell ya what in july i am doing a website which will be related and i will like u then :D

#14 ICT Helpers

ICT Helpers

    Young Padawan

  • Members
  • Pip
  • 56 posts
  • Location:England

Posted 17 April 2006 - 01:07 PM

No problem at all, just asking :D

#15 Morrigan

Morrigan

    Young Padawan

  • Members
  • Pip
  • 6 posts

Posted 19 April 2006 - 12:38 AM

It's okay, I guess. I don't really like the bevel/embossed rollovers, however. Well, I don't really like anything beveled/embossed ;3. It's pretty plain, but you can improve on it by far.




0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users