First off, location.css is being added from within hook_menu() rather than being called specifically when it's needed (when there's location information on the page.

Second off, the one CSS declaration in this file is WAAAAAY too general.

.container-inline input {
display:inline;
width:auto;
}

The effect of this is that every input element -- including hidden elements -- within a .container-inline are displayed as inline elements. The effect of this is that hidden elements (usually display: none;) get displayed on the page, horribly breaking things like the search form on most Drupal sites.

I'm not sure exactly what this declaration is supposed to be doing, so I won't venture a fix. But I'm guessing that there needs to be a lot more specificity with this.

CommentFileSizeAuthor
#3 124314.patch1.08 KBdouggreen

Comments

Nick Brown’s picture

Although the location.css file has now been removed from releases of this module it is still being referenced in location.module;

else {
drupal_add_css(drupal_get_path('module', 'location') .'/location.css');
}

which results in drupal logging page not found warnings.

I think the reference to to file also needs to be removed.

seanr’s picture

Confirmed. The reference needs to be removed ASAP - this is crapping up everyones logs.

douggreen’s picture

StatusFileSize
new1.08 KB

minor patch to remove reference to location.css.

ankur’s picture

Status: Active » Fixed

The reference has been removed. Thanks for the patch.

Anonymous’s picture

Status: Fixed » Closed (fixed)