Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

consolidate sign-in forms and use standard tailwind styles #5746

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sqs
Copy link
Member

@sqs sqs commented Sep 29, 2024

  • Consolidates the sign-in forms (previously there was a misnomer OnboardingExperiment and then a ClientSignInForm in 2 separate files)
  • Uses Tailwind styles instead of the old-style and more verbose CSS
  • Fixes the display of the "invalid URL" warning (FormMessage was not triggering the matcher because <input type="url"> meant it failed at an even earlier stage)
  • Updates the storybook

No behavior change, slight look-and-feel change but no change in wording or layout.

VS Code

image

JetBrains

image

Test plan

Try signing in using VS Code.

@sqs sqs requested review from abeatrix and a team September 29, 2024 21:52
- Consolidates the sign-in forms (previously there was a misnomer OnboardingExperiment and then a ClientSignInForm in 2 separate files)
- Uses Tailwind styles instead of the old-style and more verbose CSS
- Updates the storybook

No behavior change, slight look-and-feel change but no change in wording or layout.
@dominiccooney
Copy link
Contributor

@abeatrix , @olafurpg does VS/Eclipse use this alternative login form? AFAIK it currently does not work in JetBrains and is only displayed in error, although @pkukielka is working on fixing one of them.

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