The default Drupal throbber clashes with Slate and assumes a white background color. This patch updates the autocomplete CSS to match the Slate color scheme and provides a new throbber.gif file. Try setting the author for a node to try it out.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jwolf’s picture

Status: Needs review » Fixed

Thank you!

Committed.

Status: Fixed » Closed (fixed)

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

WorldFallz’s picture

Version: 6.x-1.4 » 6.x-3.x-dev
Assigned: quicksketch » Unassigned
Status: Closed (fixed) » Needs work

This doesn't seem to have made it into the fusion version-- i'll try to post back with a patch.

WorldFallz’s picture

FileSize
102.97 KB

I still had some problems with ghosting on the original gif, so I made a new one. Feel free to use it if you like.

The css for it is:

html.js input.form-autocomplete {
  background-position: 100% 2px;
  background-image: url(../images/throbber-small.gif);
  max-width: 96%;
}
 
html.js input.throbbing {
  background-position: 100% -16px;
}
jeremycaldwell’s picture

Status: Needs work » Reviewed & tested by the community

Thanks WorlfFallz, this will be included in the next release of the theme.

asb’s picture

Title: New Trobber, Auto-complete CSS » New Throbber, Auto-complete CSS

That thing in /misc/ is named "throbber.gif". Fixing title for better searchability.

asb’s picture

Status: Reviewed & tested by the community » Active

For the current dev release, watchdog logs:

type: page not found
message: sites/all/themes/acquia_slate/images/throbber-small.gif
severity: warning

and

type: page not found
message: sites/all/themes/acquia_slate/js/jquery.equalheights.js
severity: warning

Greetings, -asb

jeremycaldwell’s picture

Status: Active » Postponed (maintainer needs more info)

The issue with the throbber image is related to this one: http://drupal.org/node/856842

So that fix will be included in the next release of the theme with the correct file name and path etc.

The equal heights script should actually be loading from Fusion Core and not Acquia Slate so not sure what's going on with that one. Does it throw that error anytime you visit a page where the equal heights script is being used? Does the equal heights script work on your site as intended? And if not what happens if you copy it from Fusion Core to the path it is looking for it within your Acquia Slate theme?

asb’s picture

Status: Postponed (maintainer needs more info) » Closed (fixed)

Hi,

Manually downloading the missing image (#3 from issue 856842) resolves the missing throbber-small.gif icon. If the upcoming dev release includes it, let's consider this as fixed.

Am I the only one encountering the missing jquery.equalheights.js error messages? That's suspicious and needs more investigation on my end, I'm afraid (potential incompatibilities with contrib modules like 'CSS Gzip', 'JavaScript Aggregator' or 'Boost'; no, 'Panels' is not running on this site ;). Since this issue is about throbber-small.gif, I'll open a new issue as soon as I can provide more details about this.

Thanks & greetings, -asb

(P.S. Regarding you questions: The script appears to be working somehow on the site where watchdog logs the missing file error under certain conditions ("Equal heights" checked; "Dark transparent background" checked; "Rounded corners with gray background" checked; it does not work with "Equal heights" checked; "Acquia Slate - Background Styles" none; "Rounded corners with gradient background and border" checked). However, on a second site, where two blocks are embedded ino a panel, the equalize script does not either, but watchdog nevertheless does not log any errors; on this second site, I'm running Panels, but neither 'CSS Gzip', nor 'JavaScript Aggregator' or 'Boost'. In this case, the panels display settings conflict with the skinr styles in multiple ways; that's definitely a different issue, but I'm not sure if it'd belong to Panels, Ctools, Fusion, Skinr or whatever else might be involved)

asb’s picture

Status: Closed (fixed) » Active
FileSize
1.65 KB

Since there has been no new dev release for a couple of moths I'm reopening this issue as a reminder.

Also, the CSS from #4 doesn't fix the shifted throbber icon for me. Screenshot attached (Opera 11.00, Build
1156, Linux 64 Bit/Ubuntu 10.10).

This might be a issue of Fusion as I'm experiencing this in Magazeen 3.x as well. Related issue: #835694: Throbber icons on autocomplete fields are broken in Opera

grapher’s picture

102Kb gif?? :)