Results 1 to 3 of 3

Thread: facebox1.1

  1. #1
    Join Date
    Nov 2008
    Posts
    8
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Default facebox1.1

    1) Script Title: facebox1.1

    2) Script URL (on DD): http://www.dynamicdrive.com/dynamici...ebox/index.htm

    3) Describe problem: I have used this script a lot and like it very much, the only problem I have is it not centering. I know it does cause when clicking on the demo links on your site it centers perfectly.

    http://vermontwholesalers.com/produc...roducts_id=144
    is a link to where the popup is triggered from.

    Thanks for your help and time.

  2. #2
    Join Date
    Mar 2005
    Location
    SE PA USA
    Posts
    30,495
    Thanks
    82
    Thanked 3,449 Times in 3,410 Posts
    Blog Entries
    12

    Default

    OK, that page is a bit of a mess from the point of view of standards and practices.

    Here's what you could do:

    1. Change the DOCTYPE to one that invokes standards. Replace (at the very top of the page's source code):


      Code:
      
      <!doctype html public "-//W3C//DTD HTML 4.01 Transitional//EN">
      with:

      Code:
      <!DOCTYPE html>
    2. Get rid of the second (highlighted) duplicate script call:

      Code:
      <link rel="stylesheet" type="text/css" href="stylesheet.css">
      
      <script src="facefiles/jquery-1.2.2.pack.js" type="text/javascript"></script>
      
      <link href="facefiles/facebox.css" media="screen" rel="stylesheet" type="text/css" />
      
      <script src="facefiles/facebox.js" type="text/javascript"></script>
      
      <script type="text/javascript">
          jQuery(document).ready(function($) {
            $('a[rel*=facebox]').facebox()
          })
      </script>
      
      <script type="text/javascript">
          jQuery(document).ready(function($) {
            $('a[rel*=facebox]').facebox()
          })
      </script>
      
      </head>


    Just that much will get it working. But except for IE 7 (and presumably less), it will also change the layout of the page because the boxes going down the left and right sides will each be a bit taller. This makes the entire page taller. You can address this in the rest of the markup/css if you like, or live with it. Look at the cellpadding of the tables. Some of those can be reduced. You may then need supplemental styles for IE 7 and less to keep them from being too short. There could be other ways. If you want help with that, open a new thread for it in the css section. The layout of the page in standards mode has nothing to do with this script other than being required by the script. A modern page (like one used for e-commerce like this one is) should be in standards compliance anyway. And the way it is now works equally poorly, though in a different way, in most other modern browsers.
    - John
    ________________________

    Show Additional Thanks: International Rescue Committee - Donate or: The Ocean Conservancy - Donate or: PayPal - Donate

  3. #3
    Join Date
    Nov 2008
    Posts
    8
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Default

    Thanks changing the top worked out good. I like the cart cause it has good features but can be improved a lot in the way it is displayed, (the theme setup) which there is none, have to go through each script to make changes. Eventually I'll have it converted to as much css as I can do in the future. Again thanks very much for your time and help.

Bookmarks

Posting Permissions

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