(Drupal 5.7, PHP 5.2.4, MySQL 5.0.45)

I really hate to say anything against such a fantastic module, but I've stumbled across a strange IE problem. I need to display exactly nineteen individual tables on a page but the nineteenth causes IE, I think, to ignore my theme (background colors gone, fonts wrong, etc.). It can be nineteen different tables or nineteen of the same table, doesn't matter. Each table has 42 rows counting the header, except for table #19 which has seventeen rows.

Thinking maybe I hit some sort of cap on maximum number of stylesheets in IE, I removed one stylesheet import line but this did not fix the problem.

I also noticed that TableManager loads its own CSS redundantly, when it is invoked more than once ion a page. This seems inefficient to me, although I don't actually know if it's a real problem.

My only workaround is to consolidate some of these tables, but that will take some time and also violates other design and data organization rules on the site in question.

Comments

pobster’s picture

pobster commented

I'm afraid I have literally NO idea, perhaps try changing this function right at the bottom of tablemanager.module;

/**
 * Imports css file
 */
function tablemanager_css() {
  $path = url(drupal_get_path('module', 'tablemanager'));
  drupal_set_html_head('<style type="text/css" media="all">@import "'.$path.'/misc/tablemanager0.css";</style>');
  return;
}

To;

/**
 * Imports css file
 */
function tablemanager_css() {
  drupal_add_css(drupal_get_path('module', 'tablemanager') .'/misc/tablemanager0.css');
}

It'll still be called for every table (so still inefficient) but looking at the code on api.drupal.org it builds it as an array - so perhaps it'll see that it's already set and just apply the stylesheet the once? Not sure... Not really something I know anything about I'm afraid.

Pobster
PS. Thanks for your positive comments btw, hopefully you'll like tablemanager v2 even more! ;o)

silurius’s picture

silurius commented

Wow, that corrected the problem and reminded me that you happen to be one of the more responsive developers on this site. You just saved me a ton of work. Thank you very, very much.

pobster’s picture

pobster commented
Status: Active » Fixed

No problem! Glad it worked!

Pobster

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.