zeeb44 — 2011-07-15T14:04:18-04:00 — #1
ralphm — 2011-07-15T23:42:43-04:00 — #2
Which is the new div, and why are you adding it?
Where's that? I don't see it.
zeeb44 — 2011-07-18T09:14:07-04:00 — #3
The new div it is in is .conatainer #homeBanner - This is how I have my site laid out.
ralphm — 2011-07-18T10:07:59-04:00 — #4
You have this in your JS:
But the div has an id, not a class, so try this:
zeeb44 — 2011-07-18T10:30:09-04:00 — #5
Nope. That didn't work...
ralphm — 2011-07-18T10:33:42-04:00 — #6
With things like this, I find that you need to install the code as received, and then make minor adjustments and test at each stage. Then if something goes wrong, you know where the trouble started. There are some styles in there hiding the content, but that doesn't seem to be the whole story.
zeeb44 — 2011-07-18T10:36:28-04:00 — #7
Yea I am going back through now and trying to test things one by one to narrow it down. Having no luck yet though. The test page that came with the gallery, when I edit it and put the gallery into another div it works fine. So it's only when I'm putting it into my new page..
ralphm — 2011-07-18T10:36:45-04:00 — #8
All the same, you are calling on a class that doesn't exist, so might as well make it an id.
You also call on the jQuery library twice, which is a mistake. Not sure if that messes anything up, but at least remove the second link to the library. It could cause problems to call it again—especially after you've cited it already. If nothing else, it's twice as much code to download.
zeeb44 — 2011-07-18T10:46:41-04:00 — #9
I changed it to a class. But I also changed the js just now back to .container because thats how it came when I downloaded it. Also where am I calling on the jquery library twice? Can't seem to find that...
ralphm — 2011-07-18T10:51:47-04:00 — #10
In the <head> you have this:
and then a little further down this:
<!-- jQuery Slideshow for Partners -->
So, two versions of jQuery. Just use one, and put it up above any other jQuery code.
zeeb44 — 2011-07-18T10:57:47-04:00 — #11
Oh ok I see that now. So I can use the latest one and they both would work? I was under the impression you needed to use the library that the script was written for. I have another slideshow as you can see in the third grey box. That's what the other one is for. So just delete it and use one?
zeeb44 — 2011-07-18T11:01:47-04:00 — #12
Also here is the gallery working in the page I got it from Codecanyon,
jQuery Banner Rotator
See it works here. I copied and pasted everything I needed.
ralphm — 2011-07-18T11:06:50-04:00 — #13
Generally yes, it should be fine to update to the latest script. Normally it's backward compatible—at least in my experience, although I've seen a minor break once.
zeeb44 — 2011-07-18T13:23:59-04:00 — #14
After going back through and trouble shooting it looks like I figured it out. Working fine, now just need to customize it. Thanks for your help.
ralphm — 2011-07-18T18:39:09-04:00 — #15
Great, glad it's on the way. Not sure if you've come to this yet, but the social buttons on the right are a bit messed up at this stage. See screen shot.
zeeb44 — 2011-07-18T18:42:08-04:00 — #16
What browser are you on? I was making changes just now. Could you try taking another look?
ralphm — 2011-07-18T18:47:55-04:00 — #17
On a Mac, and it's the same in all browsers. It was like that before—just didn't want to distract you. I'd prefer a CSS solution, but an easy fix is to add a <br>:
GET SOCIAL UPDATES FROM SIA<font color='"Red"'><br></font>
<img height="32" width="32" alt="Southern Insurance Agency Facebook" src="http://www.sitepoint.com/forums/images/social/facebook_32.png">
<img height="32" width="32" alt="Southern Insurance Agency Twitter" src="http://www.sitepoint.com/forums/images/social/twitter_32.png">
<img height="32" width="32" alt="Southern Insurance Agency LinkedIn" src="http://www.sitepoint.com/forums/images/social/linkedin_32.png">
EDIT: sorry, not true: not all browsers, but I first noticed in in Firefox.
zeeb44 — 2011-07-18T18:55:58-04:00 — #18
I was going to use that temp. What would be a proper css way to do that? If you don't mind when you have a chance could you show me? <br> will work for now. But it's weird I don't see that on a PC in any browser, IE, Firefox, Chrome...
ralphm — 2011-07-18T19:05:40-04:00 — #19
Browsers each render text a bit differently, so you can't rely on with width of the text to push those images down. I guess there's nothing really wrong with the <br>. I prefer not to have raw text in a div, though. I normally use a <p>, but others disagree with that. One thing you could do instead is just reduce the width of the div a bit (and maybe adjust its position a little also to make up for it not being 100% wide.
zeeb44 — 2011-07-18T19:07:44-04:00 — #20
Before you even mentioned it I did <p>. I don't like using <br> too much. Seem to work now?
next page →