I've gone through all the CSS files looking for the light colored text and changing them to dark colors, but no matter what I do, most of the labels in the wizard are white on almost white, and near impossible to read. Can anyone direct me to the exact CSS to change this? I've attached a screenshot showing what I mean, the labels above the text fields are almost invisible.

I'm using the default theme for Asset, Bluelagoon was almost unusable due to placement of items being scattered all over the place.

CommentFileSizeAuthor
AssetSample.jpg26.09 KBSteel Rat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zroger’s picture

zroger CreditAttribution: zroger commented

most likely, this color is coming in from the stylesheet of your site's theme. to override, use a declaration like

#asset-popup .form-item label{
  color: #000000;
}
Steel Rat’s picture

Steel Rat CreditAttribution: Steel Rat commented

Thanks! That helped the form items, but there are other areas, mainly descriptive text.

One section after choosing Upload, at the bottom of the form which starts with the text "Select the permissions for this asset" are still coming up white. The page source says the div has a class of "description", but I don't find that class in any of the asset CSS files, either in the main asset folder or the themes folders.

nevets’s picture

nevets CreditAttribution: nevets commented

If you use firefox I suggest trying out the firebug extenstion. It has an inspect feature that speeds up finding the css rule(s) that are responsible for the styling you see.

wmostrey’s picture

wmostrey CreditAttribution: wmostrey commented
Component: User interface (Wizard) » Miscellaneous
Status: Active » Closed (fixed)
Steel Rat’s picture

Steel Rat CreditAttribution: Steel Rat commented

I think this misses the issue. The wizard is using CSS from the drupal theme, but only some of it, and therefore the display gets abused. Why should I have to go through and try and apply fixes for every potential theme I may want to use? The wizard should use its own distinct CSS and not inherit from Drupal.

1kenthomas’s picture

1kenthomas CreditAttribution: 1kenthomas commented

I suspect what is being looked for is more like:

.asset-tree-widget li.folder a.active{
  color: #ff0;
}

Without this asset picks up the li a.active (white) definition from system.css (not a theme!) and displays it on a white background (in most themes).

I'll second that this should perhaps be addressed at the module level.

This is also fairly hard to address as FFox web developer will not invoke in the wizard (no CTRL-Y), FireBug (F12) has problems, and IE7 web dev toolbar... well, does trace it correctly, but...

1kenthomas’s picture

1kenthomas CreditAttribution: 1kenthomas commented
Priority: Normal » Minor

Also

.asset-tree-widget ul{  background-color: #faf; }

Also, gently changing this to "minor, alpha3, UI" and "active..."

1kenthomas’s picture

1kenthomas CreditAttribution: 1kenthomas commented
Version: 5.x-1.x-dev » 5.x-1.0-alpha3
Component: Miscellaneous » User interface (Asset wizard)
Status: Closed (fixed) » Active

OK, next issue: the asset wizard popup in fact does not load any stylesheets from the current theme (aha!). Therefore it is not possible to theme from the theme folder at all (for those who give our themers access to that and not to the rest of our installs).

This is why the white-on-white is picked up from system.css; else the default theme would likely not define...

Hacking around it here now... upping priority to "normal..."

K

1kenthomas’s picture

1kenthomas CreditAttribution: 1kenthomas commented

I added 

  drupal_add_css(path_to_theme() ."/style.css");

to l.923 of asset_wizard.inc.

wmostrey’s picture

wmostrey CreditAttribution: wmostrey commented
Priority: Minor » Normal
Status: Active » Needs review

My preferred fix for this issue would just be to add drupal_add_css(path_to_theme() ."/style.css"); so people can overwrite css from within their theme. I'll check this with the default themes and come back with a small rapport. Thanks for your work Kenneth, it is highly appreciated.

wmostrey’s picture

wmostrey CreditAttribution: wmostrey commented
Status: Needs review » Closed (won't fix)

I won't add this. For instance in the Garland theme this completely screws the wizard's lay-out. If people want to do this, they can always plug in theme_asset_popup.

Steel Rat’s picture

Steel Rat CreditAttribution: Steel Rat commented

So, what's the fix apart from trying to fix all the possible problems manually?

Frankly I've stopped using Asset in favor of IMCE, for the reported reason and the ability to manage uploaded images (resize, delete, etc.) I would have liked to use Asset because it looked nice and was easier for non-techie users, but if the text isn't readable without jumping through hoops...