I have succesfully installed this module,but for some strange reason it doesn't show anything in IE7,absolutely empty
But works fine on FF and Opera,
Did I mess something with my CSS?

Comments

jcfiala’s picture

jcfiala commented

I've certainly tested it on IE7, so that's an odd behavior. When you view the page source, do you find the UL?

greekman’s picture

greekman commented

Yes,UL tags are existing on the source,I tested on 3 diferent PC' with IE5/IE6/IE7.
I have reloaded site with diferent theme,the same.

Tested with IE7 with demos from Jcarousel site,and they worked for me.
By the way,I see this tags in the beging of the output HTML.

and some lines under,after the

function nc_load_carousel_blog_news() { --------------------- } }

I see this :

jcfiala’s picture

jcfiala commented

I hate to say it, but your comment got chopped, probably due to filter issues.

If you're not getting an error message, then some of the css is
probably hiding things.

The important part of the CSS is this:
1) The UL is presented as a horizontal or vertical 'strip' of cells,
with each cell being a LI. No dots or bullets, just each item as a
block, basically.
2) There's a jcarousel-clip div which must be set to be the size of
several of the cells, with overflow hidden.

Otherwise, it's just work on making things be the right size and look
pretty. I suggest going in and commenting out large parts of the CSS
until you see something, and then slowly uncomment sections until it
displays again.

greekman’s picture

greekman commented

Hunted down the ugly one ;-)
I followed your advice to comment blocks of CSS
I have a Garland theme with some modifications,but this are lines from style.css unmodified from Drupal core.

#sidebar-right .block {
padding: 0 0px 0 15px;
}

I found this description on CSS padding http://www.w3schools.com/css/css_padding.asp

Description is telling,that possible values of padding-top can be 4 or 1 W3C suggests 1 but not a 0 !

After I changed like that ,all is working in IE7.
#sidebar-right .block {
padding: 1 0px 0 15px;
}

I see now,that is definitely not a Nodecarousel or Jcarousel issue.
But it will affect anyone trying to use Nodecarousel with Garland theme. Will it ?
Otherwise,the is case is closed.Cheeres !

jcfiala’s picture

jcfiala commented
Status: Active » Closed (fixed)

Sounds good - I'm glad you were able to find the problem. That's interesting that it happened that way - I wonder if it's something specific to the right sidebar?

In any case, I'm making this closed, but if it comes up again from someone working in IE7, I'll bring it up.

Note that you're misreading that chart a little - they don't mean that the value can only be 4 or 1 - they're saying that in Netscape or IE 4.0 +, this attribute works, and in Firefox 1.0+ it works as well. I'm not sure why changing the padding caused this to work... but I"m glad things are working for you now.

encho’s picture

encho commented

Man, I was having problems with IE7 for a long time. Unfortunately, I'm on ZEN theme, so that did not work. And also not using the right column. Now, stripped down the code and finally found that problem was caused by css in nodecarousel.css
This line specifically:

div.jcarousel-last {
  float: right;
  clear: right;
  cursor: pointer;
}

float:right to be more precise. So I did float:none in my themes' css and it worked!

Hope this helps someone.