Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
When using an autocomplete field, the throbber icon does not dissapear after HTTP errors (e.g. 404 if the autocomplete url is not found).
Attached patch fixes this and also makes it work in one of my modules where the same autocomplete javascript object (ACDB) is shared by many controls (they're all recipe ingredient fields, so they share the same cache).
Comment | File | Size | Author |
---|---|---|---|
#32 | throbbers.png | 19.18 KB | soxofaan |
#26 | throbber.gif | 1.3 KB | mfer |
#26 | ajax-loader.gif | 1.77 KB | mfer |
#21 | throbber_highlight.gif | 2.73 KB | soxofaan |
#21 | throbber_nohighlight.gif | 2.73 KB | soxofaan |
Comments
Comment #1
Thox CreditAttribution: Thox commentedChanged the name of the issue because it's not just HTTP errors that make the throbber start/stop at the right time.
When typing something in the autocomplete box, there's a 300ms delay before the HTTP request is made. The throbber should start when the HTTP request is made and end when the HTTP request finishes.
Comment #2
Dries CreditAttribution: Dries commentedWe forgot to commit the throbber image. Can't we cool up the throbber a bit? (Make it a rotating Druplicon or something. ;))
Comment #3
adrian CreditAttribution: adrian commentedNO NO!. make it a throbbing druplicon =)
Comment #4
Steven CreditAttribution: Steven commentedI didn't commit the throbber because I wasn't sure it should go into core. In any case, it was just a 5 minute try out to show off the idea.
Comment #5
Thox CreditAttribution: Thox commentedThe throbber should go in, but I don't think it should be anything like druplicon as default. A throbbing / spinning / bouncing druplicon could go in the bluemarine theme, but I think the throbber.gif in /misc/ should be something plain.
Having a throbber makes the control much more usable:
http://formassembly.com/blog/ajax-not-all-about-xmlhttprequest/
Comment #6
Dries CreditAttribution: Dries commentedI find the throbber handy -- it tells me up front how the textfield will work (I know what behavior to expect), and shows me when it is busy getting data. Last but not least, it is cool. It are cool but useful details like this which makes an application stand out. So I'm all for includding throbber.
Steven: can you commit whatever is meant to be the final throbber image? It's not clear whether you have plans to refine it.
Comment #7
Steven CreditAttribution: Steven commentedCommitted this patch and an improved throbber to HEAD.
http://cvs.drupal.org/viewcvs/*checkout*/drupal/drupal/misc/throbber.gif...
Comment #8
(not verified) CreditAttribution: commentedComment #9
sign CreditAttribution: sign commentedJust a quick try, throbbing druplicon :)
Comment #10
kourge CreditAttribution: kourge commentedAwesome, sign! Now this is coolness.
Comment #11
xmacinfoReopening. I like the proposed throbber attached to comment 9.
Comment #12
chx CreditAttribution: chx commentedWe should add this to Garland. Let's keep the default simple -- but as Garland gives Drupal a Drupal face, why not use our icon?
Comment #13
Steven CreditAttribution: Steven commentedUsing the Druplicon as throbber feels tacky and out of place and is certainly going to annoy a lot of developers and their clients. The image's function is to denote activity, not advertise software.
The current throbber's only link to Drupal is the use of 'drupal blue', which is already quite garish and likely to clash with other themes. If anything, it should be made more neutral.
Even if this is only for Garland... you're talking about the ultimate chameleon theme. Don't ruin that by plastering Druplicons all over. Take a page from Apple's book: they replaced the -symbol with ⌘ in their menus. For their loyal customers, their brand is only marginally associated with the logo, and relies instead on the elegant designs and technology applied in their products.
Comment #14
mfer CreditAttribution: mfer commentedWhile this may quickly become a color of the bike shed thread, here is my 2 cents.
I like Stevens idea of having something more neutral in drupal core. Something that looks good across most themes. This seems more professional and useful for all those sites that don't use drupal blue. If we did that, we could use the blue throbber in garland and, maybe, a druplicon throbber on d.o (as long as it looks good).
Comment #15
xmacinfoAlthough I like this new throbber, I agree, it should be configurable at a theme level like we do for the Druplicon logo and the Drupal favicon.
Using a druplicon in the autocomplete area is another way to advertize Drupal.
Comment #16
Susurrus CreditAttribution: Susurrus commentedIf we had a way to easily override icons such as in #138489: named and themed icons that would definitely make this less of an issue. I do agree that a more neutral throbber would be great. What Garland could do, however, is override the standard throbber with a Druplicon throbber. I think that would be a nice compromise there.
Comment #17
yoroy CreditAttribution: yoroy commentedIf we want to increase usability and overall appearance of Drupal, this is not the way to go. Fully agree with Steven's comment that this is tacky, likely to become annoying real quick and an inappropriate place for branding. Even for Garland this is not a good idea.
Please, just don't do this.
Comment #18
kourge CreditAttribution: kourge commentedDespite how cool I'd admit a Druplicon throbber would be, I'd also agree that the throbber is not exactly the best place to put branding.
Applying Jobs' quote to this situation: "We're taking the Druplicon in vain!"
Comment #19
soxofaan CreditAttribution: soxofaan commentedI also agree than a Druplicon throbber as default is a bit too aggressive branding-wise
but it doesn't hurt having one around
so I couldn't prevent myself making this one ;)
(edit: typo)
Comment #20
mfer CreditAttribution: mfer commented@soxofaan Check out the format of the existing image (linked above). What is the part of the image for the case when we don't have a moving throbber?
Comment #21
soxofaan CreditAttribution: soxofaan commented#19 was just a proof of concept
but here you are
(the nohighlight version has no specular highlight in the upper left part of the drop)
Comment #22
xmacinfoWell, I think that everyone agrees that branding through the autocomplete throbber is way too agressive.
So we will need a patch that would let the admin load his own throbber, a new druplicon throbber or use the current default one through the theme administration GUI.
We may also decide to simply modify Garland and have it use a Druplicon autocomplete throbber, leaving intact the rest of Drupal core.
Comment #23
mfer CreditAttribution: mfer commentedWhy would we do this through an admin interface? The throbber is handled via css. To override the throbber and put your own in it's a simple css override. I think it would look something like:
in your style.css file.
Comment #24
m3avrck CreditAttribution: m3avrck commentedI agree with @Steven's comments.
I think there should be a default, neutral throbber that works with any theme. I would be in favor of using the 2nd throbber in this list (the one most commonly used around the internet) : http://www.ajaxload.info/
Example : http://www.ajaxload.info/cache/FF/FF/FF/00/00/00/2-1.gif
Comment #25
Dries CreditAttribution: Dries commentedI think the proposed throbber is funny (especially the latest ones), but as indicated by others, it isn't necessarily very professional. It would be a step backward compared to a neutral throbber.
Comment #26
mfer CreditAttribution: mfer commentedHere are a couple throbbers to take a look at. One is the classic blue one from Steven converted to grayscale. The other is the most widely used on the web with white space when the throbber isn't active. Thoughts?
Comment #27
cburschkaThis makes me wonder: Perhaps the color module could recolor the throbbing icon as well?
Comment #28
catchA recolourable throbber (and possible other icons) sounds like a very nice feature.
I agree the rotating/throbbing druplicons are tacky, yet also fun. We already have a few choices of drupal powered buttons in /misc - so could it go in there without modifying any themes? Or maybe just modify bluemarine, which is ultra-Drupal and less ubiquitous. Or somewhere in the handbook if not?
Also, it might've been mentioned, but there's an Icons module in the works which'd help with some of these issues: http://groups.drupal.org/node/11066
Comment #29
Gurpartap Singh CreditAttribution: Gurpartap Singh commented+1 for re-colourable throbber(unless overridden maybe), preferably within global theme settings.
Comment #30
mfer CreditAttribution: mfer commentedCreating animated gifs in php with GD takes a little more effort than just modifying an image. From what I have read we would have to have each frame as an individual image, modify the color scheme of each frame, and then build an animated gif from the individual frame images.
For an example check out http://www.phpclasses.org/browse/package/3163.html.
How about we open a separate issue to add color module support for the throbber and decided on core throbber here.
Comment #31
rickvug CreditAttribution: rickvug commentedI think there is general agreement here that a throbbing druplicon is not the correct way to do. In #26 mfer posted ajax-loader.gif. Seems like a clean, neutral option that works with any type of theme.
Comment #32
soxofaan CreditAttribution: soxofaan commentedNote that the throbber in ajax-loader.gif from #26 is 16x16 px, while the original throbber (#7) or the desaturated version (throbber.gif in #26) is only 13x13 px (I'm talking about the size of the throbbers themselves, not the image dimensions).
In the default Garland text box, ajax-loader looks a bit cramped (see attached screenshot).
I still prefer the desaturated version of the original throbber (throbber.gif in #26) because it has a disabled state, it is simpler/less intrusive and will consequently be less annoying in the long term, I think.
It is also a bit different than the usual web2.0 throbbers, which sets Drupal apart from the rest (in a positive, branding kind of way), while it is still clear what it means because it looks like a clock.
Comment #33
cburschkaAs far as I understand, the Druplicon throbber won't be used as it is "too tacky" for a default theme (which I agree with).
So the original issue is kind of "won't fix", but we would like a recolorable throbber if it is possible. So I'm changing the issue title.
Comment #34
cburschkaMoved, too.
Comment #35
betamos CreditAttribution: betamos commented+1
Comment #36
webchickDidn't make feature freeze. Bumping to 8.x.
Comment #37
q0rban CreditAttribution: q0rban commentedI think recoloring it could actually be pretty easy (and done without color module) if your color is set via CSS and behind the throbber image (ie using negative space). The problem with this is that throbber.gif would have to be a pretty big image so that there would be no color bleed (It wouldn't be big in byte size, but in dimensions).
The down side to this is that it would have to be something simpler than the current throbber, since gif aliasing would be ugly without knowing the underlying color.
Comment #38
q0rban CreditAttribution: q0rban commentedJust noticed that Firefox and Safari both have support for aPNGs (animated pngs). Now that would be awesome! What kind of browser support do we need to see before we can use something like that? (Please don't say IE, Please don't say IE)
Comment #39
Open Social CreditAttribution: Open Social commentedThis is a very cool tool for spicing up your ajax loader or throbber: http://www.ajaxload.info/
Comment #40
markhalliwellDidn't make feature freeze. Bumping to 9.x (that is if we still want this). Otherwise someone should just close the issue. FWIW, I actually had planned on making the throbber icon based (webfont/CSS3 rotations) and replacing it via the Icon API.
Comment #41
catchComment #55
xmacinfoNot sure what is happening with this issue (and the commits).
Comment #56
apadernoIt is happening with other issues too: An automatic comment is added for old commits (in this case, one done on 2005), and it's being reported as done for the 9.1.x branch. I guess that some changes on Drupal.org are confusing the module adding those system comments.
Comment #62
quietone CreditAttribution: quietone at PreviousNext commentedColor has been removed from core, #3270899: Remove Color module from core.