architecturalinstallationteam.com

[Revlet]

After several redesigns, recodes, and porting to PHP, I think the site is finally complete.

 

http://www.architecturalinstallationteam.com/

 

I would like some input by the phpfreaks community as you all helped me to get the contact preview system up and running.

 

The only glitch I still have is that the form validation only works in IE for some reason but that is another post in another section 

 

 

Anyways, any input, suggestions, comments, or errors found are highly appreciated!

 

Thanks,

Revlet

[Axeia]

I'd make the entire height of the mainmenu links between the separators clickable, most people prefer big clickable things (you already did the width).

Links prolly look better with font-weight: bolder; as well and you got the room for it, so why not.

 

Are you sure want the alt text of the image on the top to be blank? Google wont know Bob Balsiger or Scott Bryant like that (rest of the text seems to be make a reoccurence in the rest of the page)

If you intended the alt text on the image on the right to be a tooltip (going by the text I think you might have) you should use title instead of alt as only IE shows that as a tooltip.

 

Is the text in the footer suppossed to be THAT small? I can usually take pride in my eyesight.. but 8px Georgia is pushing it. Shouldn't the "Designed by JMedia" be a link btw?

 

http://www.architecturalinstallationteam.com/index.php?p=major_projects Where'd the images go when I disabled javascript? I'd also use cursor: pointer; on the thumbs on the right as an indication they're clickable.

 

http://www.architecturalinstallationteam.com/index.php?p=contact_us

That page is done really odd.. you oughta group related elements together. And you should use <label>'s to assign text to a form element <p><label for="name"></labe><input id="name" name="name"></p>, like that.

In case you're wondering why..

that's what it looks like in some other browsers (okay granted all 3 of them combined will make up for the last 0.5% of your traffic..) but it gets the point accross I hope, related elements should be related somehow. (and you could consider http://recaptcha.net/ if you want a friendlier captcha, you did however provide email addresses so I don't see too much of a problem)

 

This page has 7 external Javascript scripts. Try combining them into one.

Grade C on Put JavaScript at bottom

 

There are 6 Javacript scriptss found in the head of the document

 

    * http://www.architecturalinstallationteam.com/greybox/AJS.js

    * http://www.architecturalinstallationteam.com/greybox/AJS_fx.js

    * http://www.architecturalinstallationteam.com/greybox/gb_scripts.js

    * http://www.architecturalinstallationteam.com/js/divswap.js

    * http://www.architecturalinstallationteam.com/js/swfobject.js

    * http://www.architecturalinstallationteam.com/js/validate.js

Smushed 38.27% or 23.90 KB from the size of your image(s).

http://smushit.zenfs.com/results/2d2a74a4/smush/head_bg.jpg <- What's up with that one anyway? doesn't seem like the most useful I've ever seen.

 

Overall I must say good job! You use tags where they made sense and <strong> instead of <span style="font-weight: bolder;">, will do your SEO rating good. You should look into either mod rewrite or using .htaccess to make the URLs search engine friendly though.

 

 

[Revlet]

I'd make the entire height of the mainmenu links between the separators clickable, most people prefer big clickable things (you already did the width).

Links prolly look better with font-weight: bolder; as well and you got the room for it, so why not.

 

Thanks for the suggestion, I'll try implementing it and seeing how it looks.

 

Are you sure want the alt text of the image on the top to be blank? Google wont know Bob Balsiger or Scott Bryant like that (rest of the text seems to be make a reoccurence in the rest of the page)

If you intended the alt text on the image on the right to be a tooltip (going by the text I think you might have) you should use title instead of alt as only IE shows that as a tooltip.

 

Good point I overlooked that...its intended as alt text but as a title that could be effective (seeing as how the major projects are one of the selling points for AIT)

 

Is the text in the footer suppossed to be THAT small? I can usually take pride in my eyesight.. but 8px Georgia is pushing it. Shouldn't the "Designed by JMedia" be a link btw?

 

At clients request it is that small, and I just started up JMedia - this is the first build under the new name so I haven't gotten around to building a JMedia site.

 

http://www.architecturalinstallationteam.com/index.php?p=major_projects Where'd the images go when I disabled javascript? I'd also use cursor: pointer; on the thumbs on the right as an indication they're clickable.

 

In order to avoid flash problems I used the swfobject javascript.

 

http://www.architecturalinstallationteam.com/index.php?p=contact_us

That page is done really odd.. you oughta group related elements together. And you should use <label>'s to assign text to a form element <p><label for="name"></labe><input id="name" name="name"></p>, like that.

In case you're wondering why..

that's what it looks like in some other browsers (okay granted all 3 of them combined will make up for the last 0.5% of your traffic..) but it gets the point accross I hope, related elements should be related somehow. (and you could consider http://recaptcha.net/ if you want a friendlier captcha, you did however provide email addresses so I don't see too much of a problem)

 

Well the contact us page is still a work in progress...but I started with labels and was having problems with them in the preview script for some reason. so I decided to go with adjustable div's floating left/right.

 

This page has 7 external Javascript scripts. Try combining them into one.

Grade C on Put JavaScript at bottom

 

There are 6 Javacript scriptss found in the head of the document

 

    * http://www.architecturalinstallationteam.com/greybox/AJS.js

    * http://www.architecturalinstallationteam.com/greybox/AJS_fx.js

    * http://www.architecturalinstallationteam.com/greybox/gb_scripts.js

    * http://www.architecturalinstallationteam.com/js/divswap.js

    * http://www.architecturalinstallationteam.com/js/swfobject.js

    * http://www.architecturalinstallationteam.com/js/validate.js

Smushed 38.27% or 23.90 KB from the size of your image(s).

 

Yea since the site is still under construction I decided to keep them independent for the momment. Once I have the finished product I will merge them.

 

http://smushit.zenfs.com/results/2d2a74a4/smush/head_bg.jpg <- What's up with that one anyway? doesn't seem like the most useful I've ever seen.

 

I was having cross-browser compatability problems using a blank white space to make the top so that bg is the white space on top of page.

 

Overall I must say good job! You use tags where they made sense and <strong> instead of <span style="font-weight: bolder;">, will do your SEO rating good. You should look into either mod rewrite or using .htaccess to make the URLs search engine friendly though.

 

Thanks and yea I need to fix that but since I am still working on the site for the sake of ease I decided to hold off until I have the complete product.

 

 

Thank you so much for the input I will fix/improve on the issues addressed asap!

-Revlet

[ILMV]

Just one point from me at the moment, and that is that your URL is insanely long, I don't think this will be very memorable and because of it's complexity you are more likely to getting spelling mistakes.

 

 

ILMV

[Axeia]

I don't know what you changed on the "major projects" page, but it worked for me before and now it's telling me

JavaScript and the Flash Player are required to view this gallery. Get Flash.

 

And I can assure you both are installed, enabled and functional (listening to a youtube video atm). So whatever you're using to detect that is giving the wrong result for at least the combination of the operating system:

• openSUSE 11.1 (linux)
  

And any of the following browsers

• Firefox 3.0.10
  
• Konqueror 4.2.3
  
• Opera 9.64
   
  

[nrg_alpha]

Over all, I like the look. Very clean and simple. Nicely done.

 

Here are my thoughts:

 

- My personal preference is using sans serif type fonts. It's not that what you have chosen is horrible.. I just think sans serif would work better IMO.

- Sans serif issues not withstanding, I find the footer text of the wrong size (too small, I would enlarge that a tad).

- As far as 'Major Projects' is concerned, as already mentioned there are issues with this. One should not develop any crucial elements requiring javascript or the like. This could kill off prospective client interest if stuff like javascript is disabled. Always have core functionality available without the need for such.

- Looking at your SEO stuff (home page for example), I am seeing an <h1> tag.. fair enough.. but then the only other heading tag is a single <h4>, and seems blank? I would make use of h2's and h3's before jumping from h1 to h4. If you chose h4 for it's size, you can use css to drive the size of heading tags..Also note that the meta tag keywords is pretty much rendered useless these days. Safe to remove that.

- Also related to SEO, the title tag should be a short descriptive sentence (I read the recommendation of less than 12 words) that is impactful with relevant keywords, and is natural sounding. You simply have 'AIT - Home'; not very SEO friendly.

- 1 markup error. not bad I would fix it.

- While your css validates, you have 35 warnings. I would consider looking into those.

 

I do like the over all look and feel of the site though. Again, nicely done.

[Revlet]

Over all, I like the look. Very clean and simple. Nicely done.

 

Here are my thoughts:

 

- My personal preference is using sans serif type fonts. It's not that what you have chosen is horrible.. I just think sans serif would work better IMO.

- Sans serif issues not withstanding, I find the footer text of the wrong size (too small, I would enlarge that a tad).

- As far as 'Major Projects' is concerned, as already mentioned there are issues with this. One should not develop any crucial elements requiring javascript or the like. This could kill off prospective client interest if stuff like javascript is disabled. Always have core functionality available without the need for such.

- Looking at your SEO stuff (home page for example), I am seeing an <h1> tag.. fair enough.. but then the only other heading tag is a single <h4>, and seems blank? I would make use of h2's and h3's before jumping from h1 to h4. If you chose h4 for it's size, you can use css to drive the size of heading tags..Also note that the meta tag keywords is pretty much rendered useless these days. Safe to remove that.

- Also related to SEO, the title tag should be a short descriptive sentence (I read the recommendation of less than 12 words) that is impactful with relevant keywords, and is natural sounding. You simply have 'AIT - Home'; not very SEO friendly.

- 1 markup error. not bad I would fix it.

- While your css validates, you have 35 warnings. I would consider looking into those.

 

I do like the over all look and feel of the site though. Again, nicely done.

 

The footer is that small at clients request. I appreciate the input but I am going to stick with the font since the client likes it.

 

The sitename is long, but www.aiteampro.com forwards to it and that is the domain that they give out

[Revlet]

I don't know what you changed on the "major projects" page, but it worked for me before and now it's telling me

JavaScript and the Flash Player are required to view this gallery. Get Flash.

 

And I can assure you both are installed, enabled and functional (listening to a youtube video atm). So whatever you're using to detect that is giving the wrong result for at least the combination of the operating system:

• openSUSE 11.1 (linux)
  

And any of the following browsers

• Firefox 3.0.10
  
• Konqueror 4.2.3
  
• Opera 9.64
   
  

 

I made an SWFObject javascript change that caused a problem but its all fixed now - thanks 

[Revlet]

Just one point from me at the moment, and that is that your URL is insanely long, I don't think this will be very memorable and because of it's complexity you are more likely to getting spelling mistakes.

 

 

ILMV

 

www.aiteampro.com forwards there and they give out aiteampro.com on the business cards and to contacts.

 

they wanted the long company name spelled out as primary hosting domain.

and I also SEO the url's so all is well now in terms of the ?p=wduiahdwuah which was unsightly and confusing