- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.1k
Spelling #1086
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
base: master
Are you sure you want to change the base?
Spelling #1086
Changes from all commits
002191e
              a9c17ec
              716b09c
              cd38b0a
              12cae3b
              2e501a8
              0efb4d8
              fc7d95d
              e1eac88
              38c94bd
              2994349
              35cda5e
              6e1b42e
              fe1d916
              fdbb99c
              56f1f7f
              393f857
              5aefb7b
              27c8e15
              4b59e9c
              bf93243
              a5be5ed
              5afae7c
              1f34746
              ef2b553
              94aacec
              3e51541
              f3fa2d0
              dbfaa58
              2fd14b9
              484bc14
              58877f4
              0cbb38e
              38611a6
              0dc101f
              279b4b4
              a167969
              baccb2a
              18f43bb
              3527dd6
              48481c6
              32cefb4
              27dd67f
              2f5b523
              95e1a85
              c49082d
              81c6878
              0bcf908
              e53856b
              4b0adca
              f2354ea
              8fd3f41
              39e0fa7
              061a746
              0ab9f4e
              8182f00
              333fb21
              93a61fc
              4b1a70a
              d641ad2
              424f79f
              File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
|  | @@ -146,7 +146,7 @@ provider you want to use in their own developer app settings. Please read the | |
| - [Google](https://firebase.google.com/docs/auth/web/google-signin#before_you_begin) | ||
| - [Facebook](https://firebase.google.com/docs/auth/web/facebook-login#before_you_begin) | ||
| - [Twitter](https://firebase.google.com/docs/auth/web/twitter-login#before_you_begin) | ||
| - [Github](https://firebase.google.com/docs/auth/web/github-auth#before_you_begin) | ||
| - [GitHub](https://firebase.google.com/docs/auth/web/github-auth#before_you_begin) | ||
| - [Anonymous](https://firebase.google.com/docs/auth/web/anonymous-auth#before_you_begin) | ||
| - [Email link](https://firebase.google.com/docs/auth/web/email-link-auth#before_you_begin) | ||
| - [Apple](https://firebase.google.com/docs/auth/web/apple) | ||
|  | @@ -547,7 +547,7 @@ To see FirebaseUI in action with one-tap sign-up, check out the FirebaseUI | |
| |Google |`firebase.auth.GoogleAuthProvider.PROVIDER_ID` | | ||
| |Facebook |`firebase.auth.FacebookAuthProvider.PROVIDER_ID`| | ||
| |Twitter |`firebase.auth.TwitterAuthProvider.PROVIDER_ID` | | ||
| |Github |`firebase.auth.GithubAuthProvider.PROVIDER_ID` | | ||
| |GitHub |`firebase.auth.GithubAuthProvider.PROVIDER_ID` | | ||
| |Email and password|`firebase.auth.EmailAuthProvider.PROVIDER_ID` | | ||
| |Phone number |`firebase.auth.PhoneAuthProvider.PROVIDER_ID` | | ||
| |Anonymous |`firebaseui.auth.AnonymousAuthProvider.PROVIDER_ID`| | ||
|  | @@ -883,7 +883,7 @@ ui.start('#firebaseui-auth-container', { | |
| provider: 'saml.myProvider', | ||
| providerName: 'MySAMLProvider', | ||
| // To override the full label of the button. | ||
| // fullLabel: 'Constractor Portal', | ||
| There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I couldn't figure out if this was  | ||
| // fullLabel: 'Contractor Portal', | ||
| buttonColor: '#2F2F2F', | ||
| iconUrl: '<icon-url-of-sign-in-button>' | ||
| } | ||
|  | @@ -1053,7 +1053,7 @@ ui.start('#firebaseui-auth-container', { | |
| provider: firebase.auth.EmailAuthProvider.PROVIDER_ID, | ||
| // Use email link authentication and do not require password. | ||
| // Note this setting affects new users only. | ||
| // For pre-existing users, they will still be prompted to provide their | ||
| // For preexisting users, they will still be prompted to provide their | ||
| // passwords on sign-in. | ||
| signInMethod: firebase.auth.EmailAuthProvider.EMAIL_LINK_SIGN_IN_METHOD, | ||
| // Allow the user the ability to complete sign-in cross device, including | ||
|  | @@ -1195,7 +1195,7 @@ ui.start('#firebaseui-auth-container', { | |
| // populate the country selector. So for countries that share the same | ||
| // country code, the selected country may not be the expected one. | ||
| // In that case, pass the 'defaultCountry' instead to ensure the exact | ||
| // country is selected. The 'defaultCountry' and 'defaultNationaNumber' | ||
| // country is selected. The 'defaultCountry' and 'defaultNationalNumber' | ||
| // will always have higher priority than 'loginHint' which will be ignored | ||
| // in their favor. In this case, the default country will be 'GB' even | ||
| // though 'loginHint' specified the country code as '+1'. | ||
|  | @@ -1547,7 +1547,7 @@ ui.start('#firebaseui-auth-container', { | |
| return app.database().ref('users/' + user.uid).set(data); | ||
| }) | ||
| .then(function() { | ||
| // Delete anonymnous user. | ||
| // Delete anonymous user. | ||
| return anonymousUser.delete(); | ||
| }).then(function() { | ||
| // Clear data in case a new user signs in, and the state change | ||
|  | @@ -1837,7 +1837,7 @@ the incoming link when the user clicks it to complete sign-in. | |
| |Google |`firebase.auth.GoogleAuthProvider.PROVIDER_ID` | | ||
| |Facebook |`firebase.auth.FacebookAuthProvider.PROVIDER_ID`| | ||
| |Twitter |`firebase.auth.TwitterAuthProvider.PROVIDER_ID` | | ||
| |Github |`firebase.auth.GithubAuthProvider.PROVIDER_ID` | | ||
| |GitHub |`firebase.auth.GithubAuthProvider.PROVIDER_ID` | | ||
| |Email and password|`firebase.auth.EmailAuthProvider.PROVIDER_ID` | | ||
| |Anonymous |`firebaseui.auth.AnonymousAuthProvider.PROVIDER_ID`| | ||
| |Microsoft |`microsoft.com` | | ||
|  | ||
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
|  | @@ -59,7 +59,7 @@ var translatableMsgs = fullCountryData.map(function(countryData) { | |
| console.log(translatableMsgs); | ||
|  | ||
| // Make the JSON object reference the translatable messages. | ||
| // This is kind of hacky since we set the the value to a string containing the | ||
| There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. duplicated words aren't technically spelling errors, but spell checkers have flagged them for decades... | ||
| // This is kind of hacky since we set the value to a string containing the | ||
| // variable name, and then unquote the string below. | ||
| outputCountryData = outputCountryData.map(function(countryData) { | ||
| countryData.name = getCountryNameVariable(countryData); | ||
|  | ||
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
|  | @@ -105,7 +105,7 @@ element.EmailTestHelper.prototype.testOnTextChangedClearError_ = function() { | |
| this.checkErrorShown(error, | ||
| firebaseui.auth.soy2.strings.errorMissingEmail().toString()); | ||
|  | ||
| // Emulate that a 'u' is typed in to name input. | ||
| // Emulate that a 'u' is typed into name input. | ||
| There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 
 | ||
| goog.dom.forms.setValue(email, 'u'); | ||
| this.fireInputEvent(email, goog.events.KeyCodes.U); | ||
| this.checkInputValid(email); | ||
|  | ||
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.
brand