...

View Full Version : Looking for criticism and praise



rvchuck
10-12-2007, 08:31 PM
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
10-13-2007, 01:16 AM
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
10-13-2007, 03:36 PM
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_
10-13-2007, 04:08 PM
With JS disabled your navigation background image doesn't show so the navigation is a bit hit and miss there.

rvchuck
10-14-2007, 07:41 PM
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
10-14-2007, 09:29 PM
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
10-14-2007, 11:58 PM
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
10-15-2007, 02:15 AM
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
10-15-2007, 05:44 AM
That did it. Thank you, thank you!
I appreciate your time.

twodayslate
10-16-2007, 12:01 AM
banner is much better.
a web babe would be much better too.

trigger_tre
10-16-2007, 03:32 AM
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
10-16-2007, 03:40 AM
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
10-16-2007, 05:17 AM
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
10-16-2007, 03:41 PM
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.

http://img181.imageshack.us/img181/4452/footergapmj5.jpg

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
10-16-2007, 09:10 PM
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.

trigger_tre
10-17-2007, 01:06 AM
I was confused when it didn't work. After looking at your code I noticed some errors so I tried to validate it. (Failed miserably). So I reworked the code removed the xtra divs and lack of others and cleaned it up a bit.

A few notes:

1.) Try not to leave empty spaces before you close a div tag, its just a waste of space and does nothing but increase file size.
2.) When you write html code always branch out each line of code accordingly. This makes it much easier to see whats going where, and easier to troubleshoot.
3.) ALWAYS, ALWAYS validate your code. This is a must. This will ensure that most people who view your page will view it in its correct form.
4.) When using special character symbols like "&" you can't just use "&", you have to use proper xhtml like "&amp;" which you can find here http://www.digitalmediaminute.com/reference/entity/index.php

This is your BAD html code:

<!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">
<head>
<META NAME=description CONTENT="Online Video Ads - Streaming Media Advertising - Offers Full range of Web Video Advertising Solutions, Online Video Advertising, Online Ads, Internet Video Advertising, Interactive Video Marketing, broadband video ads, advertising online video & other Web Media Advertising Solutions.">
<META NAME=keywords CONTENT="online video ads, online video, web video, internet video, walk on video, streaming media advertising, internet advertising, virtual web video, streaming advertising solutions, online video advertising, online streaming ads, internet video advertising, interactive video marketing, broadband video ads, advertising online video, streaming media advertising solutions.">
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Virtual Web Video Home Page</title>
<script type="text/javascript" src="ufo.js"></script>
<script type="text/javascript" src="AC_RunActiveContent.js"></script>
<!-- Advanced DHTML Popup Linking section start -->
<!-- Please place just after the <body> tag -->
<script language="JavaScript" src="virtualintro_adplib.js"></script>
<script language="JavaScript" src="virtualintro.js"></script>

<!-- Advanced DHTML Popup Linking Section End -->
<!-- compliance patch for microsoft browsers -->
<!--[if lt IE 7]><script src="ie7/ie7-standard-p.js" type="text/javascript"></script><![endif]-->

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<script type="text/javascript">
function MM_CheckFlashVersion(reqVerStr,msg){
with(navigator){
var isIE = (appVersion.indexOf("MSIE") != -1 && userAgent.indexOf("Opera") == -1);
var isWin = (appVersion.toLowerCase().indexOf("win") != -1);
if (!isIE || !isWin){
var flashVer = -1;
if (plugins && plugins.length > 0){
var desc = plugins["Shockwave Flash"] ? plugins["Shockwave Flash"].description : "";
desc = plugins["Shockwave Flash 2.0"] ? plugins["Shockwave Flash 2.0"].description : desc;
if (desc == "") flashVer = -1;
else{
var descArr = desc.split(" ");
var tempArrMajor = descArr[2].split(".");
var verMajor = tempArrMajor[0];
var tempArrMinor = (descArr[3] != "") ? descArr[3].split("r") : descArr[4].split("r");
var verMinor = (tempArrMinor[1] > 0) ? tempArrMinor[1] : 0;
flashVer = parseFloat(verMajor + "." + verMinor);
}
}
// WebTV has Flash Player 4 or lower -- too low for video
else if (userAgent.toLowerCase().indexOf("webtv") != -1) flashVer = 4.0;

var verArr = reqVerStr.split(",");
var reqVer = parseFloat(verArr[0] + "." + verArr[2]);

if (flashVer < reqVer){
if (confirm(msg))
window.location = "http://www.macromedia.com/shockwave/download/download.cgi?P1_Prod_Version=ShockwaveFlash";
}
}
}
}
</script>
<link rel="stylesheet" type="text/css" href="style.css"/>
</head>
<body>
<div class="wrapper">
<div class="navbg">
<div id="navcontainer">
<ul id="navlist">
<li><a href="index.html">Home</a></li><li><a href="types.html">Services </a>

</li><li><a href="why.html">Benefits</a></li><li><a href="walkon.html">Demos </a>
</li><li><a href="contact.html">Contact</a></li>
</ul>
</div>
</div>
<div class="banner">
<div class="bannerontop"></div>
<div class="content">
<div class="leftcolumn">
<h2>Web Video Services</h2>

<div class="navvy">
<ul class="navvylist">
<li><a href="why.html">Why Web Video Works</a></li>
<li><a href="who.html">Who Can Use Web Video </a></li>
<li><a href="types.html">Types of Web Video </a></li>
<li><a href="players.html">Web Site Video Players</a></li>
<li><a href="types.html">Floating Video Players</a></li>
<li><a href="virtualrealitysets.html">Virtual Reality Sets</a></li>
<li><a href="walkon.html">Walk On Video</a></li>

</ul>
</div>
<h2> Walk On Web Video Demo </h2>
<div class="navvy">
<ul class="navvylist">
<li><a href="vwv1.php">See Video On Your Web Site Now </a></li>
<li><a href="testimonials.html">Customer Testimonials </a></li>
<li><a href="contact.html">Contact Us </a></li>
</ul>
<p>&nbsp;</p><p>&nbsp;</p>
<!-- ADDFREESTATS.COM AUTOCODE V4.0 -->
<script type="text/javascript">
<!--
var AFS_Account="00627799";
var AFS_Tracker="auto";
var AFS_Server="www6";
var AFS_Page="Home";
var AFS_Url="http://www.virtualwebvideo.com/index.html";
// -->

</script>
<script type="text/javascript" src="http://www6.addfreestats.com/cgi-bin/afstrack.cgi?usr=00627799">
</script>
<noscript>
<a href="http://www.addfreestats.com" >
<img src="http://www6.addfreestats.com/cgi-bin/connect.cgi?usr=00627799Pauto" border=0 title="AddFreeStats.com Free Web Stats!"></a>
</noscript>
<!-- ENDADDFREESTATS.COM AUTOCODE V4.0 -->
</div>
</div>
<div class="rightcolumn">
<h3>Welcome To Virtual Web Video </h3>
<BR>
<div class="info">Full Service Web Video Solutions</div>

<div class="maincontevt">
<p> When it comes to your Web Video needs, no one compares to Virtual Web Video.<BR>
With our personalized one on one service we often spend more time on your consultation than the actual shooting of your video, assuring that your personal message comes across to your web site visitors in the most professional manner possible.</p>
<p>We have a wide variety of options available for your web video including simple image backdrops, a wide variety of virtual reality sets, and our high tech walk on video where you can become part of your web page.</p>
</div>
<h3>Complete Web Video Production </h3>

<BR>
<div class="info">Add Impact with Personalized Web Video</div>
<div class="maincontevt">
<p> Virtual Web Video will guide you through the entire production process. From tips on scripting to the finished product, ensuring your message gets delivered.<BR>
We also have our own servers to ensure smooth delivery of your video to your web site visitors.</p>
<p>We use state of the art technology for all our web video clients allowing us to create any web video style or format that exceeds our clients expectations.<BR>

With over 30 virtual reality sets to choose from we can easily put you in an environment that suits your style, message and website. Our advanced technology puts you in the set without ever leaving our studio.<BR>
Clients are welcome to come to our studio or we will travel to your location. You can also use our optional in house talent to star in your video, or deliver your web video message yourself.</p>
<p>&nbsp;</p></div>
</div>
</div>

<BR><BR><BR>

<div>

<div class="footer">&copy;2007 Virtual Web Video | A Division of The Chuck Morgan Group LLC</div></div>
</div>
</body>
</html>

Here is the code I reworked for you: (Please note its only the body of your html page.

<body>
<div class="wrapper">
<div class="navbg">
<div id="navcontainer">
<ul id="navlist">
<li><a href="index.html">Home</a></li>
<li><a href="types.html">Services</a></li>
<li><a href="why.html">Benefits</a></li>
<li><a href="walkon.html">Demos</a></li>
<li><a href="contact.html">Contact</a></li>
</ul>
</div>
</div>



<div class="banner">
<div class="bannerontop"></div>
</div>



<div class="content">
<div class="leftcolumn">
<h2>Web Video Services</h2>
<div class="navvy">
<ul class="navvylist">
<li><a href="why.html">Why Web Video Works</a></li>
<li><a href="who.html">Who Can Use Web Video</a></li>
<li><a href="types.html">Types of Web Video</a></li>
<li><a href="players.html">Web Site Video Players</a></li>
<li><a href="types.html">Floating Video Players</a></li>
<li><a href="virtualrealitysets.html">Virtual Reality Sets</a></li>
<li><a href="walkon.html">Walk On Video</a></li>
</ul>
</div>
<h2> Walk On Web Video Demo </h2>
<div class="navvy">
<ul class="navvylist">
<li><a href="vwv1.php">See Video On Your Web Site Now</a></li>
<li><a href="testimonials.html">Customer Testimonials</a></li>
<li><a href="contact.html">Contact Us</a></li>
</ul>

</div>
<div>
<p>&nbsp;</p>
<p>&nbsp;</p>
<!-- ADDFREESTATS.COM AUTOCODE V4.0 -->
<script type="text/javascript">
<!--
var AFS_Account="00627799";
var AFS_Tracker="auto";
var AFS_Server="www6";
var AFS_Page="Home";
var AFS_Url="http://www.virtualwebvideo.com/index.html";
// -->
</script>
<script type="text/javascript" src="http://www6.addfreestats.com/cgi-bin/afstrack.cgi?usr=00627799">
</script><a href="http://top.addfreestats.com/cgi-bin/main.cgi?usr=00627799"><img src="http://www6.addfreestats.com/cgi-bin/connect.cgi?usr=00627799Pauto&amp;js=1&amp;title=Home&amp;url=http%3A//www.virtualwebvideo.com/index.html&amp;refer=http%3A//www.codingforums.com/showthread.php%3Ft%3D125608&amp;resolution=1280x1024&amp;color=32&amp;Tips=0.35647199330213986" title="AddFreeStats.com Free Web Stats" border="0"></a><a href="http://www.addfreestats.com"></a>
<noscript>
<a href="http://www.addfreestats.com" >
<img src="http://www6.addfreestats.com/cgi-bin/connect.cgi?usr=00627799Pauto" border=0 title="AddFreeStats.com Free Web Stats!"></a>
</noscript>
<!-- ENDADDFREESTATS.COM AUTOCODE V4.0 -->
</div>
</div>



<div class="rightcolumn">
<h3>Welcome To Virtual Web Video</h3>
<br>
<div class="info">Full Service Web Video Solutions</div>
<div class="maincontevt">
<p>When it comes to your Web Video needs, no one compares to Virtual Web Video.<br>With our personalized one on one service we often spend more time on your consultation than the actual shooting of your video, assuring that your personal message comes across to your web site visitors in the most professional manner possible.</p>
<p>We have a wide variety of options available for your web video including simple image backdrops, a wide variety of virtual reality sets, and our high tech walk on video where you can become part of your web page.</p>
</div>
<h3>Complete Web Video Production </h3>
<br>
<div class="info">Add Impact with Personalized Web Video</div>
<div class="maincontevt">
<p>Virtual Web Video will guide you through the entire production process. From tips on scripting to the finished product, ensuring your message gets delivered.<br>We also have our own servers to ensure smooth delivery of your video to your web site visitors.</p>
<p>We use state of the art technology for all our web video clients allowing us to create any web video style or format that exceeds our clients expectations.<br>With over 30 virtual reality sets to choose from we can easily put you in an environment that suits your style, message and website. Our advanced technology puts you in the set without ever leaving our studio.<br>Clients are welcome to come to our studio or we will travel to your location. You can also use our optional in house talent to star in your video, or deliver your web video message yourself.</p>
<p>&nbsp;</p>
</div>
</div>
</div>
<br><br><br>


<div class="footer">2007 Virtual Web Video | A Division of The Chuck Morgan Group LLC</div>
</div>
</div>

There are still a lot of errors but better to take it step by step.

After you replace your html then go back and make the change to your css that I stated in the post before this one.

-trigger

rvchuck
10-24-2007, 12:56 AM
Made the corrections. Footer looks correct at finer resolutions now.
Thank you for taking the extra time. You are a big help and I greatly appreciated it. I did have to leave the color tag in the upper section to get the background to work properly but all looks good.
Thanks again.:thumbsup:

trigger_tre
10-24-2007, 01:42 AM
Looks much better now. Glad to help.

I still HIGHLY recommend taking the time to validate your code so that its compliant. I ran it through W3Schools Validator and there are still a lot of errors.

You can view your results here Markup Validation (http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.virtualwebvideo.com%2F).

You could fix half the errors by using proper XHTML code.
1.) remember to use lower case and to close ALL tags.

For example:



When using XHTML
<BR> is WRONG because its capitalized and not closed properly
<br> is WRONG because its not closed properly
<br /> is CORRECT because its lowercase and closed properly

Actually now that I look at code just correcting/closing those "break" tags should correct a lot of your other parsing errors your getting.

-trigger



EZ Archive Ads Plugin for vBulletin Copyright 2006 Computer Help Forum