Skip to content

Disable form on admin appearance sign-in page preview

Drew Blessing requested to merge dblessing_admin_appearance_sign_in_form into master

What does this MR do and why?

Fixes #24238 (closed)

The login form on the Admin Appearance preview page was previously active. While a security issue was fixed by !63895 (merged). I think this change is still useful in further reducing confusion by disabling the form completely. This will make it more clear that this is just a preview and is not the sign-in form.

Screenshots or screen recordings

Screen_Shot_2021-09-20_at_3.36.35_PM

On hover, title popover says the form is preview only and that's why it's disabled:

Screen_Shot_2021-09-22_at_3.39.46_PM

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Drew Blessing

Merge request reports