Skip to content

Version 1.1.0 qa #515

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

Merged
merged 3 commits into from
Jan 12, 2017
Merged

Conversation

samtstern
Copy link
Contributor

Did some QA, things are looking mostly great. I had to revert a PR regarding NightMode since it was messing up all of our non-nightmode styles and I would rather be safe than sorry.

  * The auth method picker screen looks really bad in landscape,
    put everything in a ScrollView as a temporary measure until
    we can think of some better UX

  * TestUtils did not compile, had to change some code
    and move the package

Change-Id: I84d3d974390774a7ea3c7cf5deb444b022bb7fd8
@samtstern
Copy link
Contributor Author

@puf or @SUPERCILEX PTAL

Change-Id: I1a43b8cd08672253a1a749cbb3e256da0a7621d7
Copy link
Collaborator

@SUPERCILEX SUPERCILEX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@samtstern LGTM!

@@ -1,36 +1,33 @@
<?xml version="1.0" encoding="utf-8"?>
<LinearLayout
xmlns:android="http://schemas.android.com/apk/res/android"
<ScrollView xmlns:android="http://schemas.android.com/apk/res/android"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome, thanks!

@samtstern samtstern merged commit 7eaa457 into firebase:version-1.1.0-dev Jan 12, 2017
samtstern added a commit to samtstern/FirebaseUI-Android that referenced this pull request Jan 13, 2017
Change-Id: I4e5d3ed02710a2f3681816be8e7df122f977fa65
samtstern added a commit that referenced this pull request Jan 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants