Looking for criticism and praise

[rvchuck]

Hoping to get feedback on my new site. Appearance, functionality, browser-resolution issues etc. I have noted 1 particular problem in firefox that the close button for the transparent players doesn't appear. I don't know why. Any thoughts and all comments are appreciated.
I am self taught and not a web builder by trade.
http://www.virtualwebvideo.com

[twodayslate]

the nav background is broken in FF. it goes 1/2 the way up.
The guy is freaky and should be killed.
The banner does not feel as professional as the rest of the layout.
thing under the footer takes up too much space.

[trigger_tre]

Originally Posted by twodayslate

The guy is freaky and should be killed.

Funny. Seriously though, the guy is fine, professional looking and speaks well.

The banner does not feel as professional as the rest of the layout.

Much agreed on this one. The background image works for the site, but the text and video graphics you placed on top of it are VERY pixelated. If you have flash then try using vector graphics. For a professional (money earning) site the banner is very important as its the first thing people look at when they visit. Trust me if they see that the text doesn't look good then I bet it will leave the impression that your video skills are at the same level. (even though they are probably not)

The footer needs to be wait its meant to be, a footer. Either drop it all the way down to the bottom of the page or cut off the rest of the white background and just show the blue behind it.

The blue background needs some depth to it. I suggest putting a repeating 20x20 pixel image. Either diagonals or hatch.

-trigger

[_Aerospace_Eng_]

With JS disabled your navigation background image doesn't show so the navigation is a bit hit and miss there.

[rvchuck]

Ok
I can't kill the talent as he writes the checks. He will however be replaced by an infobabe that we are currently interviewing.
I made a completly new banner and put a grey border to match better.
Put the footer at well, the footer, I think? I was trying to space down the hit counter so it wouldn't intrude on the page. That's why the footer moved up.
Any suggestions as to where to put the counter image and how? (It's free and has to appear).

The navigation is not javascript based so I'm unclear as to why it wouldn't operate with javascript off? The videos are controled by javascript however.

The only difference I see in Firefox is the top navigation moves over a bit but I can't find any major issues layout wise on ie or firefox at 3 different resolutions, at least on the 3 systems here I have tested it on.
I have placed a session cookie on every page that has a video (except "see video on your website") so they will each only play once. There also is a close button and a stop button on all but one.
Your continued observations are appreciated.
http://virtualwebvideo.com

[trigger_tre]

The footer still isn't a footer.

Simply because the background still continues below the footer, shouldn't be this way.

As far as the banner = much improved.

-trigger

[rvchuck]

Thank you
My css for the footer reads:

.footer{
width:100%;
background:url(images/navbg_02.gif) repeat-x;
padding-top:20px;
position:relative;
padding-bottom:20px;
clear:both;
text-align:center;
color:#fff;
}

Can this be modified to make it work right or is it in the html?
<div class="footer">blahblahblah</div>

[trigger_tre]

Originally Posted by rvchuck

Thank you
My css for the footer reads:

.footer{
width:100%;
background:url(images/navbg_02.gif) repeat-x;
padding-top:20px;
position:relative;
padding-bottom:20px;
clear:both;
text-align:center;
color:#fff;
}

Can this be modified to make it work right or is it in the html?
<div class="footer">blahblahblah</div>

Its not the code for the footer.

Try to place your gbyrepeat.jpg background in your .wrapper not in your .body

This way your gbyrepeat.jpg will stop after the footer ends and not continue to the bottom of the page.

Here is an example of my css. I put the background in the wrapper that way if the page is too short it will stop at the footer.

html, body {
margin:0;
padding:0;
background:#036;
color:#666;
font-family:Arial, Helvetica, sans-serif;
font-size:10px;
}
body {
padding:0px 0px 10px;
}
#wrapper {
width:965px;
margin:auto;
background:#FFF url(image/bg_body.png);
}
#header {
height:130px;
padding:0px 60px 0px;
background:#FFF url(image/bg_header.png);
}

Please note that your current image is 806pixels and your wrapper is 806pixels (if you include the borders) so you will either have to adjust the background image or adjust the coding for your wrapper.

-trigger

[rvchuck]

That did it. Thank you, thank you!
I appreciate your time.

[twodayslate]

banner is much better.
a web babe would be much better too.

[trigger_tre]

Its still not working as it should you missed a step, which is very noticeable when viewing in firefox.

Indicated below in blue is the correct change you made by adding the gbyrepeat.jpg to the wrapper.

BUT......

Indicated below in red you failed to remove the gbyrepeat.jpg from the body.

http://www.virtualwebvideo.com/style.css

body {

margin-left: 0px;

margin-top: 0px;

margin-right: 0px;

margin-bottom: 0px;

text-align:center;

background:#ccc url(images/gbyrepeat.jpg) repeat-y center;

}

.wrapper{

width:800px;

margin:0 auto;

background:#fff;

border-right:3px solid #ededed;

border-left:3px solid #ededed;

background:#ccc url(images/gbyrepeat.jpg) repeat-y center;

}

Just go into your CSS and remove the line in red

-trigger

[trigger_tre]

Forgot something. You might want to clean up your code just slightly.

For instance here is what your body code will look like after you make the above gbyrepeat.jpg fix.....

body {

margin-left: 0px;

margin-top: 0px;

margin-right: 0px;

margin-bottom: 0px;

text-align:center;

}

That same code can be written like this.....

body {
margin: 0px;
text-align:center;
}

The only change is that your combining all the values for the same property into one line of code instead of 4. This makes is easier to read as well as reduces file size.

Read more http://www.w3schools.com/css/css_margin.asp

-trigger

[rvchuck]

ok understood. I did that but the page now does not display correctly.
When I removed the line from the body area I lost the grey border on each side I had before, and wanted. It looked ok to me in firefox. I checked on 3 different computers at 3 different resolutions in ie and firefox. What is it that is showing up incorrectly in firefox and is there a way to keep my grey border and have the coding correct?
Thanks
The page is a template from freecss.info

[trigger_tre]

Here is what I see when I view your page in Firefox on my screen. My guess is that you dont see it because you are using a low screen resolution. I am using 1600x1200.



To fix this simply add the background property back to the body but just for the color. The code below should be right.

body {

margin-left:0px;

text-align:center;

background:#ccc;
}

.wrapper{

width:800px;

margin:0 auto;

border-right:3px solid #ededed;

border-left:3px solid #ededed;

background:#fff url(images/gbyrepeat.jpg) repeat-y center;

}

Just replace your css code for your body and wrapper with the above code. This should solve the problem.

-trigger

[rvchuck]

ok, thanks for your effort and extra time helping.
I replaced code exactly as shown and I lose the white background on the right center column and all my text on the right is on the grey background.
I will leave it as is and deal with the footer issue later.
thanks again. I don't want to take anymore of your time.
Let me know if you have any video questions in the future.
I'm better at that.