Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Page 1 of 2 12 LastLast
Results 1 to 15 of 18
  1. #1
    New Coder
    Join Date
    Jan 2007
    Location
    Left Coast
    Posts
    71
    Thanks
    3
    Thanked 1 Time in 1 Post

    Looking for criticism and praise

    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
    Last edited by rvchuck; 10-12-2007 at 08:33 PM.

  • #2
    Senior Coder twodayslate's Avatar
    Join Date
    Mar 2007
    Location
    VA
    Posts
    1,042
    Thanks
    67
    Thanked 39 Times in 39 Posts
    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.
    twitter | Quality Hosting - $5.95/mo*
    Feel free to PM me!

  • #3
    Regular Coder
    Join Date
    May 2006
    Posts
    290
    Thanks
    6
    Thanked 14 Times in 14 Posts
    Quote Originally Posted by twodayslate View Post
    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

  • #4
    Supreme Master coder! _Aerospace_Eng_'s Avatar
    Join Date
    Dec 2004
    Location
    In a place far, far away...
    Posts
    19,291
    Thanks
    2
    Thanked 1,043 Times in 1,019 Posts
    With JS disabled your navigation background image doesn't show so the navigation is a bit hit and miss there.
    ||||If you are getting paid to do a job, don't ask for help on it!||||

  • #5
    New Coder
    Join Date
    Jan 2007
    Location
    Left Coast
    Posts
    71
    Thanks
    3
    Thanked 1 Time in 1 Post

    Reworked

    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
    Last edited by rvchuck; 10-14-2007 at 07:44 PM. Reason: Forgot link

  • #6
    Regular Coder
    Join Date
    May 2006
    Posts
    290
    Thanks
    6
    Thanked 14 Times in 14 Posts
    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

  • #7
    New Coder
    Join Date
    Jan 2007
    Location
    Left Coast
    Posts
    71
    Thanks
    3
    Thanked 1 Time in 1 Post
    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>

  • #8
    Regular Coder
    Join Date
    May 2006
    Posts
    290
    Thanks
    6
    Thanked 14 Times in 14 Posts
    Quote Originally Posted by rvchuck View Post
    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

  • Users who have thanked trigger_tre for this post:

    rvchuck (10-15-2007)

  • #9
    New Coder
    Join Date
    Jan 2007
    Location
    Left Coast
    Posts
    71
    Thanks
    3
    Thanked 1 Time in 1 Post
    That did it. Thank you, thank you!
    I appreciate your time.

  • #10
    Senior Coder twodayslate's Avatar
    Join Date
    Mar 2007
    Location
    VA
    Posts
    1,042
    Thanks
    67
    Thanked 39 Times in 39 Posts
    banner is much better.
    a web babe would be much better too.
    twitter | Quality Hosting - $5.95/mo*
    Feel free to PM me!

  • #11
    Regular Coder
    Join Date
    May 2006
    Posts
    290
    Thanks
    6
    Thanked 14 Times in 14 Posts
    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

  • #12
    Regular Coder
    Join Date
    May 2006
    Posts
    290
    Thanks
    6
    Thanked 14 Times in 14 Posts
    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

  • #13
    New Coder
    Join Date
    Jan 2007
    Location
    Left Coast
    Posts
    71
    Thanks
    3
    Thanked 1 Time in 1 Post
    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

  • #14
    Regular Coder
    Join Date
    May 2006
    Posts
    290
    Thanks
    6
    Thanked 14 Times in 14 Posts
    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

  • #15
    New Coder
    Join Date
    Jan 2007
    Location
    Left Coast
    Posts
    71
    Thanks
    3
    Thanked 1 Time in 1 Post
    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.


  •  
    Page 1 of 2 12 LastLast

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •