Skip to content

LG-15484: Update design of Sign In/Create Account tabs#11872

Merged
aduth merged 5 commits intomainfrom
aduth-lg-15484-sign-in-tabs
Feb 14, 2025
Merged

LG-15484: Update design of Sign In/Create Account tabs#11872
aduth merged 5 commits intomainfrom
aduth-lg-15484-sign-in-tabs

Conversation

@aduth
Copy link
Copy Markdown
Contributor

@aduth aduth commented Feb 12, 2025

🎫 Ticket

LG-15484

🛠 Summary of changes

Updates the visual appearance of the "Sign in" and "Create an Account" tabs on the sign-in page.

The intention of these changes is to improve the design clarity of the buttons being a tab control, since we have observed with the current (previous) designs that many users will click on the active button (e.g. "Sign in") while already on that page, with the expectation of it being a call-to-action button to continue signing in.

Changes also include some standardization of button appearance between the "Sign in" and "Create an account" pages:

  • Always display the button at full-width (previously the button was full-width on the Sign In page, but wide width on the Create an Account page)
  • Always display the button with a "Submit" label (previously the button was "Submit" on the Create an Account page, but "Sign in" on the Sign In page)

📜 Testing Plan

Verify there are no regressions in toggling between "Sign in" and "Create an Account", including expected states of links (hover, focus, active).

Verify there are no regressions with submitting "Sign in" or "Create an Account" page forms.

👀 Screenshots

Screen Before After
Sign In Screenshot 2025-02-12 at 8 57 07 AM Screenshot 2025-02-12 at 8 56 49 AM
Create an Account Screenshot 2025-02-12 at 8 57 10 AM Screenshot 2025-02-12 at 8 56 53 AM

changelog: User-Facing Improvements, Sign In, Improve design clarity of "Sign In" / "Create an Account" tabs
@aduth aduth requested a review from a team February 12, 2025 14:03
Copy link
Copy Markdown
Contributor

@kevinsmaster5 kevinsmaster5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and tests out locally.
Seems to need a handful of specs to be relabeled and should be ready to go 👍

@aduth
Copy link
Copy Markdown
Contributor Author

aduth commented Feb 12, 2025

Seems to need a handful of specs to be relabeled and should be ready to go 👍

Thanks! Those tests are a good example of what we recently talked about with hard-coding labels 😄

@aduth
Copy link
Copy Markdown
Contributor Author

aduth commented Feb 12, 2025

I'm going to hold off on merging this until after tomorrow's deploy, so the changes from #11839 have enough time to collect sufficient data to compare the effectiveness of these changes.

@aduth aduth merged commit 4ab0dcb into main Feb 14, 2025
2 checks passed
@aduth aduth deleted the aduth-lg-15484-sign-in-tabs branch February 14, 2025 15:06
aduth added a commit that referenced this pull request Feb 18, 2025
eileen-nava pushed a commit that referenced this pull request Feb 18, 2025
aduth added a commit that referenced this pull request Feb 18, 2025
aduth added a commit that referenced this pull request Feb 19, 2025
aduth added a commit that referenced this pull request Feb 19, 2025
* Reapply "LG-15484: Update design of Sign In/Create Account tabs (#11872)" (#11893)

This reverts commit 2aacd53.

* Center-align tab navigation text when wrapped

* Add changelog

changelog: User-Facing Improvements, Sign In, Improve design clarity of "Sign In" / "Create an Account" tabs
mitchellhenke pushed a commit that referenced this pull request Feb 20, 2025
* Reapply "LG-15484: Update design of Sign In/Create Account tabs (#11872)" (#11893)

This reverts commit 2aacd53.

* Center-align tab navigation text when wrapped

* Add changelog

changelog: User-Facing Improvements, Sign In, Improve design clarity of "Sign In" / "Create an Account" tabs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants