enhance: use readonly field for username on mfa form - Githubissues

tapis-project / authenticator

BSD 3-Clause "New" or "Revised" License

1 stars 3 forks source link

enhance: use readonly field for username on mfa form #41

Closed wesleyboar closed 10 months ago

wesleyboar commented 10 months ago

Overview

Make the Username data on the MFA form be in a read-only field.

Related

attempts to improve upon https://github.com/tapis-project/authenticator/commit/d30e96f

Changes

replaces text with label and readonly input

Testing

Run system.
Login.
Visit MFA page.
Verify username field can not be edited.
Check whether Username label and field are legible.
Check whether Username label and field are similar in style to other field.
Verify form still works.

UI

Warning The containers are unable to be run on my system (my OS uses ARM architecture). So I test in situ at https://cep.tacc.utexas.edu/static/ui/components/detail/s-form-page--login.html.

Before	In Situ