WordPress.org

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3123 closed enhancement (fixed)

Login Form Improvements

Reported by: Viper007Bond Owned by:
Milestone: 2.1 Priority: normal
Severity: minor Version: 2.1
Component: Administration Keywords: login form has-patch commit
Focuses: Cc:

Description

As mentioned on wp-hackers. Please note that I just went over this with a fine tooth comb, but it still needs a little testing and double checking just to be sure. ;)

Demo: http://test.viper007bond.com/wordpress-latest/wp-login.php

What's new (oh where to begin?!):

  • wp-register.php moved to wp-login.php?action=register and redirect added in wp-register.php
  • Registration page is currently only like half styled in the SVN. This has been fixed.
  • Multiple errors can now be displayed for each form
  • Bottom links made to look pretty (they looked bad if registration was enabled)
  • Fixed some display issues in IE
  • Made the WP logo much easier to click in FF (it was all funky before)
  • Removal of all references to wp_die(). All errors and messages (nearly) are now reported via the standard error area.

And the most important:

  • Tons of new hooks and filters

You can now restyle the login pages however you want due to the addition of a hook into the <head>.

Each form also now has a hook in it so that you add new input fields (say their sex or birthday or even a CAPTCHA). There's also of course a new hook for each that allows you to handle the $_POST and then set an error if your field is left blank, not filled in correctly, etc. This error will be displayed like any other error.

For example, check out this example which was created via a very simple plugin:

http://img215.imageshack.us/img215/5851/wpregistercolorjt7.png

Attachments (4)

improvedlogin.patch (31.3 KB) - added by Viper007Bond 8 years ago.
improvedlogin2.patch (31.3 KB) - added by Viper007Bond 8 years ago.
WP version number added to .css call
improvedlogin3.patch (31.2 KB) - added by Viper007Bond 8 years ago.
This should be commit worthy
revert_login_post_vars.diff (1.4 KB) - added by markjaquith 8 years ago.
Revert login post vars to "pwd" and "log", per Matt's request

Download all attachments as: .zip

Change History (14)

comment:1 Viper007Bond8 years ago

Oh, and the translation people are probably going to hate me for this patch, lol. I tried to keep as many of the existing strings as possible, but I did have to make some new strings. Sorry guys.

Viper007Bond8 years ago

comment:2 shorty1148 years ago

I like it. Especially now that the registration pages have been styled... looked a little weird before. Nice job.

Viper007Bond8 years ago

WP version number added to .css call

comment:3 Nazgul8 years ago

  • Keywords has-patch added

I've been playing with this for some time now and I like it.

comment:4 Sewar8 years ago

  • Keywords login form commit added
  • Priority changed from low to normal

+1 for this, Why wait more?

comment:5 Viper007Bond8 years ago

A few commits have broken the styling of this a bit. Updated patch coming soon.

Viper007Bond8 years ago

This should be commit worthy

comment:6 ryan8 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4339]) Improved login from Viper007Bond. fixes #3123

comment:7 matt8 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This harmfully changes the POST variable names used from log and pwd to something else.

Please revert that part.

markjaquith8 years ago

Revert login post vars to "pwd" and "log", per Matt's request

comment:8 markjaquith8 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [4354]) revert login $_POST vars change from [4339]. fixes #3123

comment:9 Viper007Bond8 years ago

Whoops, was trying to improve the names to better match what they were and I didn't realize it'd affect anything. My apologies.

comment:10 matt8 years ago

No worries, I agree your names are better it's just the old ones are built into anything that tried to tie into the login system.

Note: See TracTickets for help on using tickets.