correctly set iframe border width to 0

AzureAD / azure-activedirectory-library-for-js

The code for ADAL.js and ADAL Angular has been moved to the MSAL.js repo. Please open any issues or PRs at the link below.

https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/maintenance/adal-angular

Apache License 2.0

627 stars 374 forks source link

correctly set iframe border width to 0 #933

Closed jessemartin closed 3 years ago

jessemartin commented 3 years ago

The borderWidth attribute was being set directly on the iframe element, when it should instead be set on the iframe's style object.

This bug enabled the token refresh iframe to take up space when borderWidth was not globally set to 0, resulting in elements shifting and extra scrollbars being created.

jessemartin commented 3 years ago

@jasonnutter mind reviewing?