I think the grid overlay would make for a good module in the Accelerator package.

Pros:

  • Expandability: additional grid tools, such as labels for regions (à la omega), would be a good fit for accelerator. It'd be more consistant to have these tools in one place.
  • Reduces fusion_core footprint: production sites could disable the grid overlay from the modules page and its corresponding CSS/JS would never be loaded.

Cons:

  • Instead of going to theme settings to turn on/off the grid overlay tool, user would presumably visit admin/appearance/fusion. Alternatively, perhaps accelerator could hook_form_alter() the theme settings from the module.
  • The module would have an implicit rather than explicit requirement for a Fusion theme. It's possible that some users might enable the grid tool for a non-fusion theme and not understand why things aren't working.

Comments

esmerel’s picture

I think the cons are fairly minor and definitely documentable; this seems like a reasonable move to me. Tools are not necessarily something that belong in the core and if it's in accelerator, the possibility exists that someone could make it more compatible with other themes in drupal so that grid overlay would work on more of them.

stephthegeek’s picture

Thumbs up from me!

Poieo’s picture

Issue summary: View changes
Status: Active » Closed (fixed)

This was moved to Accelerator some time ago.