Core themes need to display required text field information

[joedolson]

Following #16206, two of the core themes replace/remove the text that explains required fields when user's are logged-in. These themes should keep this text so that the meaning of the asterisk is explained in text.

https://core.trac.wordpress.org/ticket/16206#comment:70

[sabernhardt]

keep (core) default logged_in_as text for Twenty Nineteen and Twenty Twenty-One

[sabernhardt]

The logged_in_as text has been null since the initial commits for both ​Twenty Nineteen and ​Twenty Twenty-One, so I do not know the reasoning behind that choice.

The simplest way to fix the missing required fields message is to undo that removal.

If we change it to the required fields text only If the default links somehow do not belong in the form with these themes, however, the custom message argument could get rather messy here. Even if I want to use exactly the same required fields text as the core default argument now includes, that seems to require (copying and) re-writing the markup within the context of the theme.

[poena]

I did not understand the last paragraph about "change it to the required fields text only,".

[poena]

Are you referring to that the logged in text and required fields text are on the same line?

[poena]

With patch applied

[sabernhardt]

By setting the logged_in_as message to null, these themes removed the two default links plus their paragraph container. If there was a good reason for removing those, then the themes' custom logged_in_as argument would need the "Required fields are marked *" text on its own inside a paragraph. I edited the comment above so it's hopefully clearer.

[joedolson]

A little history: setting the comments logged_in_as argument to null dates back at least as far as version 1.0 of Seedlet, the theme used as a base for Twenty Twenty One. There's no evidence it was ever discussed during theme development, so we're considering it a circumstantial element inherited from prior art, rather than a theme decision.

[joedolson]

In 52149:

Bundled Theme: Display required text field in core themes.

Fix Twenty Nineteen and Twenty Twenty-One setting the logged_in_as parameter to null, which also eliminated the required field text. See #16206.

Props sabernhardt, poena.
Fixes #54392.

[desrosj]

In 52182:

Twenty Nineteen: Apply coding standards fix from running composer format.

Follow up to [52149].

See #54392.

[sabernhardt]

Bundled themes update note:
​https://make.wordpress.org/core/2022/01/05/bundled-theme-updates-with-wordpress-5-9/