It's not a big deal, but the second time I ended up adding a class to my theme to let me hide the colorbox div, I thought why not have a class in the module called colorbox-hidden that could just be applied without having to define it each time. Patch attached.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggmarshall’s picture

frjo’s picture

Drupal core already has a element-hidden class definition in system.base.css.

/**
 * Hide elements from all users.
 *
 * Used for elements which should not be immediately displayed to any user. An
 * example would be a collapsible fieldset that will be expanded with a click
 * from a user. The effect of this class can be toggled with the jQuery show()
 * and hide() functions.
 */
.element-hidden {
  display: none;
}
greggmarshall’s picture

Good point, I think I'll add it to the documentation page.

frjo’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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