-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix - Log in - Promo Colors are changing each time the user writes a new character for new password #5913
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, adding CP Staging label to resolve deploy blocker
Fix - Log in - Promo Colors are changing each time the user writes a new character for new password (cherry picked from commit ebf2c53)
🚀 Cherry-picked to staging by @Jag96 in version: 1.1.8-3 🚀
@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes. |
🚀 Deployed to production by @roryabraham in version: 1.1.8-9 🚀
|
🚀 Deployed to production by @roryabraham in version: 1.1.10-2 🚀
|
Details
We were getting a random
backgroundStyle
everytime the componentSignInPageLayout
rendered.App/src/pages/signin/SignInPageLayout/SignInPageLayoutWide.js
Lines 29 to 32 in 4339b0b
So I just moved it outside of
SignInPageLayoutWide
function.App/src/pages/signin/SignInPageLayout/SignInPageLayoutWide.js
Lines 29 to 32 in f618cac
Fixed Issues
$ #5912
Tests / QA
Tested On
Screenshots
Web
Did a refresh to demonstrate that the colors are random on refresh.
Screencast.from.16-10-21.01_17_06.PM.+03.mp4
Desktop