-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
FirebaseUI not adjusted according to the rotation and numbers of providers #1434
Comments
@JitenderSha thanks for reporting. What is the bug in portrait mode? That looks correct to me. Also you say this is "Android version 6" but from the look of the screenshots that device is running Android 4.x ... can you provide a screenshot from a device running Lollipop, Marshmallow, or higher? I just want to see if this is Android version specific. |
@samtstern I have the same issue, in horizontal mode the layout looks wrong. Looks identical to @JitenderSha pictures, Logo off center and login buttons stuck to the very right. I am using Android API 28 |
#1457 shows this issue on a modern device, which is the confirmation I need. |
@samtstern any idea of a rough eta til fixed? |
@JitenderSha Probably 4.2.1 😊 |
This has been fixed and released in version |
Describe your environment
Step 3: Describe the problem:
In Portrait mode
As in screenshot 1, the blank space is leaved if used two providers.
In Landscape mode
As in screenshot 2, the blank space is leaved at the bottom of the logo and logo is not in vertical center.
Steps to reproduce:
startActivityForResult(
AuthUI.getInstance()
.createSignInIntentBuilder()
.setAvailableProviders(providers)
.setLogo(R.mipmap.tb_logo) // Set logo drawable
.setTheme(R.style.AppTheme) // Set theme
.build(),
RC_SIGN_IN);
Observed Results:
Expected Results:
In Portrait mode, if two providers are used the space should evenly distributed between the logo and buttons.
In Landscape mode, the logo should be placed in vertical center of the screen.
Relevant Code:
Just copy paste the following :
// Choose authentication providers
The text was updated successfully, but these errors were encountered: