tapis-project / authenticator

BSD 3-Clause "New" or "Revised" License
1 stars 3 forks source link

feat: wp-63 redesign login form #36

Closed wesleyboar closed 1 year ago

wesleyboar commented 1 year ago

Overview

Redesign login form. Minimal changes to other files to support the redesign.

Related

Changes

Testing

Function

  1. Run system.
  2. Visit login page.
  3. Compare style to https://www.tacc.utexas.edu/portal/login.
  4. Verify links work.
  5. Verify login still works.

Styles

  1. Footer is pinned to bottom of page.
  2. Footer does not overlap form.
  3. Form styles mimic https://www.tacc.utexas.edu/portal/login.
  4. Login form supports two logos.
  5. On wide screens:
    • (if logos are not too wide)logos are side-by-side, with space between
    • the "Security" and "Policy" links are right-aligned
    • the login form has gray space to the left and right of it
  6. On narrow screens:
    • (if logos are too wide) logos are atop each other, with space between
    • the "Security" and "Policy" links are center-aligned
    • the login form has gray space to the left and right of it
  7. On very narrow screens:
    • (if logos are too wide) logos are atop each other, with space between
    • the login form has no gray space to the left and right of it

UI

Warning The containers are unable to be run on my system (my OS uses ARM architecture). So I test in situ at https://portals.tapis.io/v3/oauth2/login?client_id=PROD.CEP&….

Tall Page Short Page, Scroll Start Short Page, Scroll End
b
e
f
o
r
e
login form - before login form scroll start - before login form scroll end - before
a
f
t
e
r
login form - after login form scroll start - after login form scroll end - after
Narrow Very Narrow
b
e
f
o
r
e
login form narrow - before login form very narrow - before
a
f
t
e
r
login form narrow - after login form very narrow - after

Notes

The markup should look like that pasted in the first comment.

wesleyboar commented 1 year ago

The markup should look like this:

Estimate of Markup on Login Page ``` Tapis Authentication
```
wesleyboar commented 1 year ago

Tested by @joestubbs.

Screenshot ![image-from-js](https://github.com/tapis-project/authenticator/assets/62723358/c474bd7c-2957-4e43-93f8-adcc72351143)