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

[SDK-1389] Applied appearance styles for Bootstrap in Safari #1815

Merged
merged 1 commit into from
Mar 3, 2020

Conversation

stevehobbsdev
Copy link
Contributor

@stevehobbsdev stevehobbsdev commented Mar 3, 2020

Changes

This makes sure the appearance value is set correctly when Bootstrap is included on the Lock template, which sets appearance: button. This breaks the UI in Safari, which renders the buttons without a background color.

Before

Screenshot 2020-03-03 at 10 14 15

After

Screenshot 2020-03-03 at 10 13 43

References

This originated from an internal service desk ticket.

Testing

This was tested manually by including the Bootstrap CSS on the playground template.

  • This change adds unit test coverage
  • This change adds integration test coverage
  • This change has been tested on the latest version of the platform/language

Checklist

This makes sure the appearance value is set correctly when Bootstrap is
included on the Lock template, which sets `appearance: button`. This
breaks the UI.
@stevehobbsdev stevehobbsdev added bug This points to a verified bug in the code CH: Fixed design labels Mar 3, 2020
@stevehobbsdev stevehobbsdev added this to the vNext milestone Mar 3, 2020
@stevehobbsdev stevehobbsdev requested a review from a team March 3, 2020 10:21
@stevehobbsdev stevehobbsdev merged commit 918293e into master Mar 3, 2020
@stevehobbsdev stevehobbsdev deleted the fix/siwa-bootstrap branch March 3, 2020 16:20
jfromaniello pushed a commit to jfromaniello/auth0-lock that referenced this pull request Mar 5, 2020
This makes sure the appearance value is set correctly when Bootstrap is
included on the Lock template, which sets `appearance: button`. This
breaks the UI.
davidpatrick pushed a commit to davidpatrick/lock that referenced this pull request Jun 12, 2020
This makes sure the appearance value is set correctly when Bootstrap is
included on the Lock template, which sets `appearance: button`. This
breaks the UI.
jfromaniello pushed a commit to jfromaniello/auth0-lock that referenced this pull request Jul 23, 2020
This makes sure the appearance value is set correctly when Bootstrap is
included on the Lock template, which sets `appearance: button`. This
breaks the UI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This points to a verified bug in the code design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants