Input on clients website.

  • Thread starter Thread starter Der Alta
  • 12 comments
  • 1,269 views

Der Alta

Official GTP Bouncer
Staff Emeritus
Messages
9,209
Messages
DerAlta
Just hammering a website together for a client, on the cheap.

I'd like you guys to take a look and provide some feedback. I've not posted the site "live" as of yet, and only have it hosted on a private server.

Any feedback or thoughts you have would be greatly appreciated.

www.omandcorp.com/lelyhouse

Have at it.
 
Java for links is a huge no-no these days. For the effect you&#8217;re looking for, I would just throw in <a> elements, set them to display:block and a high negative text indent, and then use background images. I&#8217;d also use position:fixed instead of frames.
 
I got a load of java errors. Horrible. There is no need to use java on a website.
 
OK, so Java is bad and shouldn't be used. Input accepted. Can you tell me why?


It's likely I need a newer web page editor.
 
Java doesn&#8217;t load as fast (in fact, the :hover effect seems spurious in whether it loads or not), you can&#8217;t middle-click to open the link in a new tab, you can&#8217;t right-click to copy the link or open in a new tab, Google spiders don&#8217;t know what to do with them, they can&#8217;t be accessed with the keyboard using the tab key (this isn&#8217;t just for disabled people &#8211; I do it all the time), and there&#8217;s no fallback mechanism if the user doesn&#8217;t have Java or if their version is broken. On the other hand, there are zero drawbacks to using a bit of CSS on anchor tags, except maybe a half hour learning curve.
 
Java for links is a huge no-no these days. For the effect you&#8217;re looking for, I would just throw in <a> elements, set them to display:block and a high negative text indent, and then use background images. I&#8217;d also use position:fixed instead of frames.

Sage nailed it. Hate Java and frames.

Also, get an interior designer and retake the photos(no offense). :dopey:
 
Excellent, thanks!

And yes, the pictures are due to be retaken. Only hiccup, is I'm near Boston, MA, the house is in Naples, Florida.

so... the 1/2 hour learning curve will be undertaken.

Thanks again.
 
If it will help, I can offer a critique of your design.

The overall look of your design is good, though there are issues depending on which browser you look at it with. For instance, study the image below.



This image consists of a side-by-side comparisons of how your website looks in Safari and Google Chrome. Because you are using Frames, it is impossible for me to tell you why, but you can clearly see a white band to the right of the navigation links. Both Safari and Google Chrome use the WebKit rendering engine, so there may be some correlation in the way you have written the web page, and how it is displayed in those browsers.

Also, you will notice that there is a slight variation in the colours of the navigation frame, the header frame, and the main content frame. Within the header frame, the background behind the image is also a different colour to the background colour of the image itself.

This is true in Firefox and Safari though in Google Chrome, Opera and IE8, there 'seems' to be no difference. I can only surmise that you have used a 'named colour' ie 'red', instead of the CSS method of the colour 'red', #ff0000 in this example. It is always good practice to use the CSS method for declaring colours as it guarantees consistency between browsers.

As others have said, using frames and Java on web pages is not a good idea, for reasons of compatibility, usability and accessibility. Something else I noticed, that no one else pointed out, was that when you mouse over the links, there is a thin white line that appears first below the star on the 'Main' link, then above the star for the remaining links.

Looking at the source, for the page, you have chosen not to include a !DOCTYPE. A !DOCTYPE declaration is very important because it tells the browser in question how to render your website correctly. Failure to include a valid !DOCTYPE forces the browser to render your website in quirks mode. Quirks mode basically involves the browser guessing what !DOCTYPE you are using, though not all browsers guess in the same way, so you will see inconsistencies when viewed in different browsers.

For more information on !DOCTYPES read this: Fix Your Site With the Right DOCTYPE!

Moving on, looking at the Meta tags, I can see that you have used Microsoft Frontpage 5.0. Back in the day it might of been an acceptable tool for authoring web pages, but by today's standards, it is a very poor tool. Don't worry though, there are far better tools available, and for free!

Aptana IDE - An Excellent feature rich IDE based on Eclipse, and, I would say, on a par with Dreamweaver.

Netbeans - A superb feature rich IDE that is also on par with Dreamweaver.

Notpad++ - A superb Notepad replacement, excellent if you don't mind hand coding.

If you feel like adding fancy effects to your web pages, then I thoroughly recommend Jquery. It is a very well supported JavaScript framework that can help you create great 2.0 style effects within your web pages.

Further reading/inspiration:

Smashing Magazine
Ajax Rain
A list Apart
Verlee's Blog
(Adobe) Kuler

I hope this critique was helpful. I would of provided further assistance, but for the fact you are using frames. 👍
 
Taking a different perspective from the other guys, here are a few things that I noticed on the main.html page.

  • Remove the <strong> tags around the period in the first sentence.
  • It doesn't look like all of the &nbsp; are necessary.
  • Both &quot; and the standard " are used. For ease of future editing, I'd just pick one.
  • The first and second commas in the sentence "The links, to the left," should be removed.
  • In the next sentence, either query or question should be removed as they both mean the same thing.

Or, if you're so inclined, you could just use this:

Code:
<html>

<head>
<meta http-equiv="Content-Language" content="en-us">
<meta name="GENERATOR" content="Microsoft FrontPage 5.0">
<meta name="ProgId" content="FrontPage.Editor.Document">
<meta http-equiv="Content-Type" content="text/html; charset=windows-1252">
<title>This is the Main Page</title>
<base target="_self">
</head>

<body background="image/background.jpg" bgcolor="#DEC2A9">

<div dir="ltr" align="left">
  <span class="989181619-10032010">
  <p><font size="2" face="Arial"><span style="color: black; font-size: 9pt"> Nestled between the Gulf of Mexico and the Everglades, Naples combines breathtaking natural beauty with relaxed, sophisticated living in one of the world’s most desirable tropical locations. </span>When you combine the weather with fantastic beaches, warm sea's, incredible restaurants and great shopping, anyone can see why our home in Naples is so special to us.</font></p>
  <p><font size="2" face="Arial">Our home is in the Masters Reserve community of the beautiful Lely Resort which boasts more than 3,000 acres of wonderful architecture, green space and plenty of outdoor activities for the entire family. Lely Resort was awarded "Community of the Year" in 2008 and 2009.
  </font><span style="FONT-FAMILY: Tahoma; COLOR: black; FONT-SIZE: 9pt">
  <font size="2" face="Arial"><font size="2" face="Arial">The Lely Golf &amp; Country Club is home to 36 holes of spectacular golf with two distinct open-to-the-public Championship Courses. We invite you to take a look at the list of amenities available throughout Lely Resort and additional services offered to our guests in the exclusive members only “Players Club and Spa”.</font>
  </font></span></p>
  <p><span style="FONT-FAMILY: Tahoma; COLOR: black; FONT-SIZE: 9pt">
  <font size="2" face="Arial">Lely Resort is conveniently located just 10 minutes from Marco Island, 10 minutes from downtown Naples and 15 minutes from the Everglades. For those of you who like to explore, this central location offers multiple possibilities for entertainment, dining, shopping, attractions and outdoor activities. Easy day trips can be made to the Naples Botanical Garden, Zoo or to the Everglades. </font></span></p>
  <div>
    <font size="2" face="Arial">The links to the left provide more information on our home, the Naples area, Lely Resort, and information on how to book. If you have any questions then please do not hesitate to contact us. </font></span>
  </div>
  <p><font size="2" face="Arial"><span class="989181619-10032010">Enjoy!</span></font></div>
&nbsp;<p>

<a href="https://www.vrbo.com/286859?cid=L_OwnerBadge">
<img border="0" src="https://www.vrbo.com/VRBO-vacation-rentals-by-owner-listing/286859?t=1" alt="VRBO - vacation rentals by owner - listing #286859" align="right"></img></a>
</p>

<p>&nbsp;</p>

</body>

</html>

I think. :p
 
@TB:

Where did you get that from? I guess I didn't look deep enough into the other pages. In my defence though, it was very late when I posted my initial critique. :sly: 👍

Looking at the code you have posted, I would say that it is going to be a nightmare to update, if for what ever reason, the client decides he wants a different font or font size for instance.

As an example, look at the <p> tag. It's style is declared identically several times over. This not only makes it infinitely harder to change the style of the page, but it also adds weight to the page, making for a larger download.

If you added some CSS in the head section, you could remove some of the 'tag soup' and have a cleaner, and lighter page.

For instance, just looking at the <p> tag, if you added only this style block in the head section:

<style type="text/css">
p {
font-family: arial, helvetica, sans serif;
font-size: your font size here;
}
</style>

you would remove the vast majority of the repeated style statements and <font> tags. You would then only have to write:

<p>Your text here</p>

The style declaration in the head would sort it out, though you would still need to alter the style in the head section of each page, which could be time consuming depending on how many pages you website has.

If you add the following in the head section though, you would only need to edit one single file for the changes to cascade through your whole website:

<link rel="stylesheet" type="text/css" href="link to your css file" />

CSS files are easy as pie to create too. All you have to do is open up any text editor and enter your style declarations and save it as CSS, ie. your_css_file.css.

Depending on what text editor you use, you may or may not have the option to save the file directly as a css file. If you have no option to, just hard code the file extension. In Notepad for instance, you just enter the .css after the file name, save it, and you have your CSS file!

So, going back to the style declaration in the head (I posted above), you would type in the text editor:

p {
font-family: arial, helvetica, sans serif;
font-size: your font size here;
}

Then, save it as a CSS file and link it to your document for those rules to apply to every <p> tag in your document.

If you would like to learn more about HTML, or CSS, the W3C School (Full Web Building Tutorials) is an invaluable resource for those eager to learn more. 👍

One more tip, I would strongly suggest that you learn XHTML (the link tag I posted above was written in XHTML). It is the standard now on the web, and ensures maximum compatibility with a wide rage of devices, because not everyone surfs the web from a desktop PC.

There is a lot more I could discuss about the code posted by TB, so if I can be of further assistance, please feel free to ask. 👍
 
Wow! Thanks guys for all the input. I'm still struggling with it as I reread all of this. In addition to trying to learn Dreamweaver. I picked up CS4 and having been trying to sort out the coding and redoing the page.

Until I scrap the entire thing and rebuild it with Dreamweaver, I'll have a tough time sorting all the info and incorporating it all.

What's really a sticking point at this time, is the calendar (Booking tab). I just don't know enough about the scripting to properly put it together. If anyone can point me in the right direction for the calendar, I'd sincerely appreciate it.
 
OK, I've made a whole slew of changes and updates.

Can you guys take another look and give me a run down? BTW, thanks Sage for your help!

I've posted you each a PM, with the website link.

Thanks!
 
👍

From an aesthetic standpoint, I&#8217;d probably make the URLs the same blue-green as the :hover color on the navigation links. Otherwise, nice!
 
Back