Pages: 1Website for Review
http://www.lease2u.co.uk
Additional Comments
Hi all,
Lease2u website was originally built by a guy who totally screwed us. he backlinked us to over 1000 porn websites in Poland! Obviously we sacked him and then I took over. I never knew what I was getting myself into at all. Although I have been online 20 years I had never ventured into web design at all. I inherited a very basic stand alone static site. I have since learned how to include manufacturer brochure, edit deals in xml, learn photoshop basics to design new logo etc.
I would appreciate all comments but mostly those that tell me what they like and offer suggestions rather than just criticism. The site is constantly being worked on but I am always open to new ways of improving it, after all i have been staring at it for months.
Pete
Administrator
From: Colorado, USA
Registered: 2006-02-15
Posts: 2169
I've been thanked 90 times.
Offline
Hello Pete, I started early 
Nice site, consistent layout and navigation, easy to find what I might be looking for and quick loading.
Just a few things to make it even better.
First thing I noticed is that the page is not centered and sits slightly to the right of the browser. When going down in browser size. the page does not move but adds a horizontal scroll bar. As browser is increased the off to the right gets worse. This should be corrected so that the pages are centered in the different browsers.
The lighter blue text is on the difficult side to read, not real bad but could use a little more contrast. Make the text a shade or two darker.
The font size is slightly on the small side. Maybe one or two steps bigger and maybe a little more spacing between lines. Actually if the size were bigger the color might be fine.
On your testimonials page the text for the testimonials runs right up against the left side content. Add some padding to the left side of the testimonials
On the technical side you have some coding issues that need to be taken care of. Setting aside some of the minor ones, I suggest you take care of the more serious ones.
You have multiple DTD declarations, head sections, body sections.
You should only have one of each.
At the very top of the page before the opening html tag is where the DTD goes. Then the opening html tag followed by the opening head tag.
All the head information such as the meta tags and script tags
Then close the head and open the body
Now all the page content
Close the body
close the html
Example;
Code: html
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html lang="en" xml:lang="en" xmlns="http://www.w3.org/1999/xhtml">
<head>
All the head information
</head>
<body>
All the page content
</body>
</html>
Attention designers and webmasters - "The Beauty of CSS"
Valid Web Designs tutorials on HTML, XHTML and CSS
Home Security Systems
Member
From: South America
Registered: 2008-01-21
Posts: 17
I've been thanked 1 times.
Offline
Hi, the white blue is a good colour for your service, itīs confortable and reflect what the people wants in a car. The information is complete and the web site is easy to navigate and fast. The product is always at sight. Thatīs the humble opinion of a simple man.
On your testimonials page the text for the testimonials runs right up against the left side content. Add some padding to the left side of the testimonials
Hi ColoEagle,
Thanks for taking the time to have a look. I have hit my head against the wall regarding the testimonials page. I cannot get the text to shift away from the left hand side.
The lighter blue text is on the difficult side to read, not real bad but could use a little more contrast. Make the text a shade or two darker.
I hear what you are saying here but the site is 100% acceible for all, even the visually impaired. I want to keep the font uniform.
I am thick, I don't know what a DTD tag is.
Thx
Pete
Administrator
From: Colorado, USA
Registered: 2006-02-15
Posts: 2169
I've been thanked 90 times.
Offline
Looking at your code you have the testimonials inside paragraphs with a class of testimonials. I was unable to find this class in your stylesheets. So you need to add this class to the stylesheet. Play around with the px setting to get the amount of padding that fits your taste.
Code: css
.testimonials {padding:10px;}
A DTD is a DocType Declaration. This is important as it tells the browser what to use to display the page. Keeps the browser from guessing and possibly displaying your pages incorrectly.
The DTD needs to be placed above everything on the page. As mentioned earlier above your opening html tag.
Keeping the font uniform is important, glad to hear this is a concern of yours. As mentioned it's not real bad but by making it just a shade or two darker will still let you keep the blue theme and make it even easier to read. Not affecting the accessibility. Again not something that you have to do, just a suggestion.
If you don't already have it get the FireFox browser Then get the html tidy extension.
Once you have these view your page source. Tidy will list the errors on the page and when possible give an explanation of how to correct it. This will also assist you in getting validating pages which helps to insure your pages are accessible to a larger number of people.
Attention designers and webmasters - "The Beauty of CSS"
Valid Web Designs tutorials on HTML, XHTML and CSS
Home Security Systems
Hi Guys and thanks for the advice. I have added the padding in the style sheet to testimonials, thx colo. This is strange though. When I open the editor the code is only showing one head and one body section, yet when I view source in a browser it shows what you guys have spotted.
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html lang="en" xml:lang="en" xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>Lease2u Car And Van Leasing</title>
<meta content="Evrsoft First Page" name="GENERATOR" />
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii" />
<meta content="car leasing, van leasing, contract hire. Lease2u the simplest way for all your vehicle leasing needs" name="description" />
<meta content="car,leasing,van,vehicle,business,contract hire,contract,cheap,cheapest," name="keywords" />
<link href="style.css" type="text/css" rel="stylesheet" />
<meta content="INDEX,FOLLOW" name="robots" />
<link href="/favicon.ico" type="image/x-icon" rel="icon" />
<link href="/favicon.ico" type="image/x-icon" rel="shortcut icon" />
</head>
<body>
<!-- caloSf -->
<div id="calosc">
<?php include('belka_top.php'); ?><?php include('top.php'); ?>
<div id="srodek">
<!-- czesc zmienna -->
Member
From: San Antonio, TX
Registered: 2006-08-07
Posts: 477
I've been thanked 23 times.
Offline
Just looking at the first page, here are a few problems that I see:
An 11 pixel high light-gray line just above your banner and below your menu. This looks out of place, because it shorter than the banner, and doesn't seem to fit with anything else there.
A shadow just to the left of the picture of the car, and aligned next to your side menu that looks out of place.
There are also some problems that are specific to Internet Explorer 6. Though I'm not an IE6 fan myself, there are just too many people out there that are still using it to ignore. Here are the problems:
The text "THE SIMPLEST WAY FORWARD FOR ALL YOUR VEHICLE LEASING NEEDS" takes up an extra row and seems to be in a larger font, causing your background image to repeat.
Your side menu is pushed down below your main text area.
If you don't have any computers with IE6 still on them, you can view it using http://browsershots.org/
Hi Steven,
Thanks for your comments. damn I never knew the site looked like that on IE6, is there anything I can do to fix how it looks?
Member
From: San Antonio, TX
Registered: 2006-08-07
Posts: 477
I've been thanked 23 times.
Offline
The first problem looks like an easy fix. Take out the <font size="5"> and it's end tag from around the text. Then in your style sheet change the font-size attribute in class .tg to something like 22px. I'd suggest, if you can, getting rid of all the <font> tags and controlling your font sizes through the style sheet. Different browsers act differently to the size attribute in the <font> tag.
The other problem might be a little more complicated. What's happening is IE6 has trouble sometimes with box sizes and margins. You're floating your main section in to the right, then on proper browsers the menu is able to slip into the leftover space. IE6 believes that leftover space is too small for the menu, so it puts it in lower on the page. what you may need to do is widen the division #srodek a little and see if the menu slips into place.
Thanks Steven,
Most of that was double dutch to me as i never designed the site. But alas am sure with a few beers in me I will be able to understand what you have said, like an immigrant speaking fluent swahili in 5 minutes! Thx bud.
| Never |
You are not logged in.
