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.
* Removed the jumpy fade effect. I was thinking of doing something fancy but decided it was better doing it simple.
* Clear the OpenID field when clicking cancel, sort of a bug (test by filling in the field, go back and login normally - the OpenID login is fired)
* Removed the icon from the Cancel link.
* Clear out form_error message when going between login methods.
* Give the Cancel link a weight to have it under the login button on /user, just as in the block.
* Put focus on the textfields when going between login methods.
* Shortened some of the original jQuery code.
Comment | File | Size | Author |
---|---|---|---|
#1 | openid_login_form_0.patch | 4.57 KB | anders.fajerson |
openid_login_form.patch | 3.98 KB | anders.fajerson |
Comments
Comment #1
anders.fajerson CreditAttribution: anders.fajerson commentedAs a bonus, here is a Konqueror friendly version that uses .css("display", "block") instead of .show(). Otherwise the same as above. The Konqueror problem was reported here and I could confirm it using Konqueror 3.5.8. I think it fits into this patch (and requeries the removal of the fade effect) so keeping it in the same issue for now.
Comment #2
BioALIEN CreditAttribution: BioALIEN commented+1 to the suggestions here. If Konqueror requires the removal of the fade to work with OpenID, then I'm all for it.
Comment #3
samirnassar CreditAttribution: samirnassar commentedWell, this patch is Drupal 6 specific. I think in terms of Konqueror 3.5.8 and Drupal 5 this issue is a moot point. In terms of Drupal 6, KDE4 will be coming out fairly soon in 2008 which solves the issue.
I will try the patch against D6 this week and let you know how it behaves with Konqueror 4 betas as well as Konqueror 3.5.8
Actually feel free to send any Konqueror related patches my way for testing and such.
Comment #4
anders.fajerson CreditAttribution: anders.fajerson commentedStill applies. Desperately needs testing. Bumping up to "bug" due to "Clear the OpenID field when clicking cancel" mentioned above and the Konqueror fix.
Comment #5
Crell CreditAttribution: Crell commentedI just tested the patch in #1 against Konqueror 3.5.8, and was able to log in successfully. No flickry animation, just a successful login. Yay. The other changes appear to work as well, and in Firefox too. For the sake of all 3 Konqueror users, I'll RTBC this. :-)
Comment #6
Gábor HojtsyOK, committed, thanks.
Comment #7
anders.fajerson CreditAttribution: anders.fajerson commentedI'm moving this to the OpendID project issue queue for backport to the Drupal 5 module (hopefully this is correct use of the issue queue, please correct me if I'm wrong).
Comment #8
walkah CreditAttribution: walkah commentedTotally acceptable use, fajerstarter. I'll make sure this gets backported!