-
Notifications
You must be signed in to change notification settings - Fork 554
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
[SDK-1361] Adjust styling for social buttons and Apple compliance #1801
Conversation
- Moved icon element inside text element so they can be center-aligned - Applied flex styling to text element and center justified - Text font, letter spacing and casing changes to match new style
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.
Nothing blocking here but ... are we sure that we're not going to run afoul of another provider by making this change? Hopefully no one enforces left-aligned 😆
@joshcanhelp That may be about to change.. |
@joshcanhelp This has now been changed so that the buttons have their text left-aligned once again. Screenshots in the original post have been updated. |
…th0#1801) * Reworked social button styling - Moved icon element inside text element so they can be center-aligned - Applied flex styling to text element and center justified - Text font, letter spacing and casing changes to match new style * Modified button text on login and signup to "Sign in with.." * Fixed button hover styling, added retina media query * Adjusted styling for Google login button * Modified border color for Google login button * Updated snapshots * Reverted 'sign up with..' text * Re-aligned text to the left * Updated snapshots * CSS formatting
…th0#1801) * Reworked social button styling - Moved icon element inside text element so they can be center-aligned - Applied flex styling to text element and center justified - Text font, letter spacing and casing changes to match new style * Modified button text on login and signup to "Sign in with.." * Fixed button hover styling, added retina media query * Adjusted styling for Google login button * Modified border color for Google login button * Updated snapshots * Reverted 'sign up with..' text * Re-aligned text to the left * Updated snapshots * CSS formatting
…th0#1801) * Reworked social button styling - Moved icon element inside text element so they can be center-aligned - Applied flex styling to text element and center justified - Text font, letter spacing and casing changes to match new style * Modified button text on login and signup to "Sign in with.." * Fixed button hover styling, added retina media query * Adjusted styling for Google login button * Modified border color for Google login button * Updated snapshots * Reverted 'sign up with..' text * Re-aligned text to the left * Updated snapshots * CSS formatting
Changes
This PR primarily adjusts the "Sign in with Apple" button to better match Apple's guidelines.
A knock-on effect is that the other social login buttons have had their designs updates to match, for consistency.
Additionally, "Sign up with.." text has been changed into "Sign in with.." across the signup and login tabs.
Before
After
References
This was raised through an internal customer support ticket.
Testing
This PR contains no functional changes, only visual.
Checklist