[Admin reskin] Register page notice missing white background

[juanfra]

After the admin reskin, notices on the register page don't have a white background color. The background color (#fff) is defined in common.css, but the register page only loads login.css, so the styling isn't applied.

Steps to reproduce

1. Check that anyone can register on your site (Settings > General)
2. Go to yoursite.tld/wp-login.php?action=register
3. Check the "Register for this site" notice and see that's missing the white background.

[juanfra]

Register notice missing background color.

Trac ticket: https://core.trac.wordpress.org/ticket/64715

Adding background color to the .notice in the login.css file, as the notices there don't match the reskin (complete styles live in common.css).

Before	After

[huzaifaalmesbah]

This restores visual consistency with the admin reskin. Looks good to me!

Before Apply Patch	After Apply Patch ✅

Before	After

[audrasjb]

Thanks for the PR.
Self assigning for final review and commit consideration.

Before	After

[joedolson]

I think this is really a duplicate of #64678; the issue is that all notices of this type have a transparent background, and it should be handled globally.

Before	After

[juanfra]

Thanks for looking at this.

I think this is really a duplicate of #64678; the issue is that all notices of this type have a transparent background, and it should be handled globally.

Sorry I missed that ticket! Discovered this while updating the logo for the login page.

Fair point that this should be handled globally. One thing to keep in mind though, the login/register screens don't load common.css where notices are defined, so login.css would still need updating regardless of where we fix this.

Would it make sense to handle the login-specific part here since we've already got the context, or would you prefer to coordinate it with the other ticket?

[joedolson]

Given that the commit has to happen in a different context, we can go ahead and keep them separate; I think that's reasonable.

[audrasjb]

Yes, let's keep that ticket to address login-specific issues.

[audrasjb]

In 61731:

Login and Registration: Add white background to register screen notices.

This changeset backports the changes made to common notices to login.css.

Props juanfra, huzaifaalmesbah, shailu25, joedolson.
Fixes #64715.

[juanfra]

Fantastic, thank you all!

[joedolson]

FYI, I'm reverting this change as part of the commit to fix #64678, as this wasn't complete; it needed to apply to both the .notice and the .message classes.

[joedolson]

In 61737:

Admin: Set info notices to use white background.

The admin reskin changed info notices to a transparent background, matching usage in Gutenberg. But with the gray background in the core admin, these notices didn't show up in an expected manner. Additionally, using a transparent background in a context where the background color is variable is risky for color contrast.

Change .notice-info, .notice-info.notice-alt, and .login .notice, .login .message to use white backgrounds. Reverts changes from [61731] to apply styles also to .message class.

Props mohitbsf, audrasjb, peterwilsoncc, pbiron, fabiankaegy, noruzzaman, joedolson.
Fixes #64678. See #64715.