Closed (fixed)
Project:
Drupal core
Version:
5.x-dev
Component:
javascript
Priority:
Critical
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
8 Jan 2007 at 00:11 UTC
Updated:
20 Jan 2007 at 00:22 UTC
Jump to comment: Most recent
JQuery has been recently updated to 1.0.4 in http://drupal.org/node/104833.
I think this needs thorough testing on IE 7, IE 6, Opera 9, Firefox 1.5 and 2.x, Safari and Konqueror. Uploads, collapsible fieldsets and autocomplete needs testing. Especially the edge conditions, eg pressing enter when autocomplete is searching.
Comments
Comment #1
BioALIEN commentedSpecs: FF2.0, Mac, Drupal5 RC1:
Auto complete works however, on removing text from a text field results in a blank area over the page. You're forced to press back and reload the entire page.
Replicate the bug: Create new page/story, uncollapse the "Author Information" block, click the "Authored by" field and slowly remove the characters.
Comment #2
mfer commented@BioALIEN: I tested in FF2, Windows, 5 RC1 and there was no problem with this.
Comment #3
webchickI cannot replicate the error BioALIEN refers to on the exact same setup using HEAD.
BioALIEN, would you be able to download http://ftp.osuosl.org/pub/drupal/files/projects/drupal-5.x-dev.tar.gz and see if you can duplicate it there? I believe there's been a fix since RC1 for the error you speak of.
Remember to clear your browser cache, as the .js files probably will have been updated.
Comment #4
webchickAnd actually, that goes for anyone who's not using CVS...
You should test the http://ftp.osuosl.org/pub/drupal/files/projects/drupal-5.x-dev.tar.gz file, not RC1. This change went in after RC1.
Comment #5
mfer commentedum, oh yeah. oops. I totally missed that this update wasn't in RC1.
Comment #6
heine commentedI've created the following issues:
http://drupal.org/node/107600 - progress bar widget moving when starting upload
http://drupal.org/node/107766 - attach button no longer clickable after 3rd upload
Not sure if those are 1.0.4 related.
Comment #7
Will White commentedNot sure if this is due to the JQuery update but...
"Pressing enter while auto-complete is searching causes strange behavior"
http://drupal.org/node/107611
Comment #8
BioALIEN commentedUpdate: having downloaded the latest Drupal5-dev and testing the bug in #1 seems to be resolved. Well done to everybody - another bug bites the dust.
@diggersf: as far as I am aware, it's always had this characteristic from the first time I laid eyes on it many months ago.
Comment #9
BioALIEN commentedUpdate2: while my original bug is now fixed, I've continued to try breaking the auto complete to test diggersf's bug. Here are some results:
Same test as #1 but this time we remove the entire "Authored By" name and add "anon" (for anonymous user)... as soon as autocomplete starts doing its magic press ENTER quickly.
Here are the outcomes:
1) The white/empty layer appears on the page (as per #1).
2) The "Authored By" text field in question becomes read-only and I am not able to change it unless the page is reloaded.
3) The page is submitted (this could be normal, it means you have out paced the auto complete) which usually results in an error "anon doesn't exist".
4) Clicking away from the "Authored By" field replaces the text/empty field with the string "undefined".
Most of this could be extreme cases, but I thought it would be good to put it under all sorts of abuse.
Comment #10
Steven commentedI've verified all core JavaScript to be working on Safari, Firefox 2 and Opera 9.1 OS X.
Comment #11
chx commentedI tested Konqueror 3.5.5 on Drupal.org and found that grippie, collapsible fieldsets, upload work but autocomplete does not seem to work: the throbber does not display and there is no autocomplete popup either.
Comment #12
Anonymous (not verified) commentedI fully tested in IE 6.. only issue I found.. http://drupal.org/node/108215
Comment #13
Steven commentedThe konqueror issue is not critical... it degrades to the non-JS version just for that feature. We'll have to figure that out later.
I've fixed the IE issue that steve reported.
Still need testing on IE7.
Comment #14
Steven commentedI just tested IE7 on Vista over VNC... everything worked. That gives us all the major browsers.
Comment #15
Caleb G2 commentedI'm not positive but I have a feeling this issue (which I posted before I knew about this thread) is related, as is possibly this one.
Also, I'm not sure how the newly released jquery 1.1 fits into this matter.
Marking as active, because clearly there are some javascript errors in the 5.0 release which people are noticing.
Comment #16
Steven commentedBoth issues already have tickets of their own, and the latter looks like a plain and simple widgEditor bug.