Add body classes and 'login_body_class' filter to wp-login.php

[husobj]

The body class for the login page is hardcoded and simply outputs 'login' no matter what the context - ie login, register, forgotten password etc.

It would make more sense to add additional classes for the context and also make it filterable so that styling can be tweaked for register/login/forgotten password etc - I wrote this patch in response to requirements I had to style the register layout differently to the login.

My attached patch adds additional classes for the context.
(It retains the 'login' class throughout - this is required anyway for backwards compatibility)

It also adds a new filter 'login_body_class' in a similar way to 'body_class' and 'admin_body_class'

[SergeyBiryukov]

To reduce code duplication, 21133.2.patch​ introduces _get_login_action() and mirrors admin_body_class filter a bit closer.

[husobj]

Yep, that's definitely more tidy

[kobenland]

Talking about tidiness, I like the body_class() implementation a lot more than how it's done in the admin.
21133.3.diff​ picks up Sergey's _get_login_action(), too.

[SergeyBiryukov]

21133.4.patch​ adds a missing !is_array( $class ) check before preg_split().

[kobenland]

Uh, good eye! Wonder where that went :)

[nacin]

I think we can keep this much simpler. admin_body_class is weird because it doesn't deal with an array of classes, but we could make it deal with an array here. How about 21133.diff​?

[husobj]

I definitely prefer the array approach like the body_class filter.

Is using the global $action variable preferable to the _get_login_action() in previous patches? It's certainly more tidy

Maybe just needs an 'rtl' class adding too using is_rtl() like previous patches.

Other than that I like nacin's patch

[nacin]

Replying to husobj:

Is using the global $action variable preferable to the _get_login_action() in previous patches? It's certainly more tidy

wp-login.php is a pretty messy file as it is. Checking the global seemed the least intrusive way to do this.

Maybe just needs an 'rtl' class adding too using is_rtl() like previous patches.

I'm fine with that.

[nacin]

In [22000]:

Add a login_body_class filter to login_header(). fixes #21133.