Hi

one little thing about storm bugs me

function storm_init() {
  drupal_add_js(drupal_get_path('module', 'storm') .'/storm.js', 'module', 'header', FALSE);
  drupal_add_css(drupal_get_path('module', 'storm') .'/storm.css', 'module');
}

You load those files on every page request. Please try to load them only if really needed!

Best regards,

Karsten "Kars-T" Frohwein

Comments

Magnity’s picture

Magnity commented
Component: Code » Storm.module

I've made a few commits on the CSS front to gradually move css from storm.css to usage specific css files (the Drupal aggregator can stick them back together on page load so that performance doesn't suffer). Also, am adding comments in the css files so that it is possible to understand where each item of css is used.

I'll be expecting any further patches on other to follow suit with comments and by putting relevant css in an appropriate file.

kars-t’s picture

kars-t commented
Status: Active » Closed (won't fix)