First identified here http://drupal.org/node/732914#comment-3455956

I'm having a very hard time figuring out how to reproduce this - but it just seems that *sometimes* locale.css loads for this path.

Right now in Drupal this has no discernible affect on the exposed filters at admin/content because other CSS overrides locale.css, however if we apply the patch from #732914: Improve the markup/CSS for content and user filter forms those overrides are removed and if locale.css loads it messes up the filters.

From what I can tell locale.css should not be loading on this page so I is this a bug?

I'm attaching a screenshot - this taken before the #732914 patch, you can clearly see locale.css is loading but being overriden by other CSS (node and user css actually).

Comments

sun’s picture

jacine’s picture

File issue aside, I think this CSS is potentially problematic because the field names are very generic:

.form-item-language,
.form-item-translation,
.form-item-group {
  ...
}

This seems safer, and actually more efficient, minus the fact that it's using an ID as opposed to a class:

#locale-translation-filter-form .form-item {
  ...
}
jacine’s picture

Looking at the rest of the CSS in that file, it also makes more sense in the context of the existing code to use the ID: http://drupalcode.org/viewvc/drupal/drupal/modules/locale/locale.css?&vi...

mertskeli’s picture

[edit]
I do not have locale.css at admin/content. (I have enabled Locale module but have not touched anything in it since that.)

sun’s picture

Title: locale.css loaded on admin/content, sometimes » locale.css loaded and applied on other pages and elements
Status: Active » Needs review
StatusFileSize
new1.06 KB
new10.53 KB

Indeed a problem:

locale.css_.png

Attached patch fixes the problem.

Usage of a HTML ID is indeed unfortunate, but the other existing styles for the same form are using it too already.

jacine’s picture

Status: Needs review » Closed (fixed)
Jeff Burnz’s picture

Awesome, at last :)