Hi,

Here are some issues i found while modyfying your theme:

in blue.css:

div#middlecontainer {
  background: #ffffff url(../images/blue/body-bg.png) 0 0 repeat-x;
}

This section have no influence on the body look. the image is not loaded so the bg is white

in suckerfish_blue.css:
while separating:

#suckerfishmenu	 a:active,
#suckerfishmenu	 a:hover {
  background: #818181 url(../images/blue/menu-bg-over.png) repeat-x;
  text-decoration: none;
  border-right: #999 solid 1px;
}

to

#suckerfishmenu	 a:active{
    background: #818181 url(../images/myimage/a.png) repeat-x;
}
#suckerfishmenu	 a:hover {
  background: #818181 url(../images/blue/menu-bg-over.png) repeat-x;
  text-decoration: none;
  border-right: #999 solid 1px;
}

Whaen active (after click & pageload) a menu item is not using a.png as background.

Possibly those are not intentious but mayby I am wrong. If so please explain the way how those can be changed

Comments

roopletheme’s picture

roopletheme commented
Assigned: Unassigned » roopletheme

Looks like I might have some remnants of previous experiments with background images. I'll remove these definitions from the CSS file in HEAD during my next set of changes.

Bazin’s picture

Bazin commented

Please let me know in this bug that this has been fixed when You'll upload its correct version to the drupal download site

roopletheme’s picture

roopletheme commented

I'll update this issue once I've updated the theme.

Bazin’s picture

Bazin commented

Current version is 6.x-1.7 so the next 6.x-1.8 will have the css fixed?

mrtoner’s picture

mrtoner commented

This issue hasn't been identified correctly:

div#middlecontainer {
  background: #ffffff url(../images/blue/body-bg.png) 0 0 repeat-x;
}

The issue is not that the image isn't loaded -- it's that divs inside that container (#main and #sidebar-left, for example) have a background that covers up the background of the container.

kisugiai’s picture

kisugiai commented
Status: Active » Closed (fixed)

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

mrtoner’s picture

mrtoner commented
Title: CSS issues » Background images not showing
Assigned: roopletheme » Unassigned
Status: Closed (fixed) » Active

I appreciate your desire to close issues that have had no activity and start fresh, Alyx, but this issue certainly wasn't fixed. This particular problem isn't noticeable at first, but once you realize that there's supposed to be a gradient background in the #middlecontainer you miss the fact that it's not there.

kisugiai’s picture

kisugiai commented
Status: Active » Needs review

OK fixed in the dev if the dev not ready in zip then take it from git

well, i don't even have noticed, I've been wondering about the problem.
thanks.

and i have all issues closed they was more then few weeks old.
if some still actual then they will post again or reopen.

mrtoner’s picture

mrtoner commented
Version: 6.x-1.7 » 6.x-1.x-dev
Status: Needs review » Fixed

Interesting. Theming is my weakness; I would not have thought that adding that class was the fix. I'll file that away for further use.

Status: Fixed » Closed (fixed)

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