Hello,

firstly thanks for this gracious theme! One of the best for free themes for Drupal.

I just encountered a problem occuring with the Galleria module which seems to be related to Corolla for it doesn't occur with other Drupal themes.

I get the following error "Could not extract width/height from image: http://www.example.com/sites/example.com/files/styles/galleria_thumb/pub.... Traced measures: width:0px, height: 0px."

The issue was closed in Galleria, because it seems not to be related to the module itself: http://drupal.org/node/1308912

It works also fine under Safari & Chrome, just not in Firefox.

I applied the following fix in the .js itself but it still doesn't work out:

http://getsatisfaction.com/galleria/topics/could_not_extract_width_heigh...

Any help would be gladly appreciated!

Comments

Jeff Burnz’s picture

Jeff Burnz commented

Any chance you have a test site setup somewhere I can view and play around with this, I am way busy this week I'm afraid. If you haven't, and you don't hear from me by the end of next week bump this issue and I'll take a look (I might forget about it, big week coming up, travelling a lot etc).

Frankly I have no clue, someone in the Galleria forum said something about max-width: 100% which this theme uses for responsive images, could possibly be part of the issue, not sure.

ergophobe’s picture

ergophobe commented

This is not a Corolla issue. It is true with any theme based on Adaptive Theme, but it is also true for the Wordpress Galleria plugin and several other contexts.

For me setting
.galleria-image img {max-width: none;}

fixes the issue in FF, Chrome, Opera and Safari, but not IE.

See #1308912

ergophobe’s picture

ergophobe commented

fixed in AdaptiveTheme -- #1393534: IE8 disappearing images bug - cannot use max-width: 100% and declare width attribute on image elements

I found that the fix I posted above worked in everything except IE (all versions, not just 6-8), and now the fix to AT fixes this, but you still need to override it in Corolla

./corolla/css/html-elements.css lines 30-33.

Jeff Burnz’s picture

Jeff Burnz commented

./corolla/css/html-elements.css lines 30-33.

Yeah, that stupid line in Corolla, should remove it.

ergophobe’s picture

ergophobe commented

although I actually came back here to point out that my comment is wrong

The new rule in AT is more specific than this one, so it prevails and even without removing the lines mentioned above, everything seems to work (don't have access to IE 9 on my laptop, but IE8 is good).