Skip to content

Commit b0cfab1

Browse files
thatfiredevsamtstern
authored andcommitted
refactor(rtdb): replace ProgressDialog with ProgressBar (firebase#1018)
1 parent d278e42 commit b0cfab1

File tree

5 files changed

+43
-37
lines changed

5 files changed

+43
-37
lines changed

database/app/src/main/java/com/google/firebase/quickstart/database/java/BaseActivity.java

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,30 @@
11
package com.google.firebase.quickstart.database.java;
22

3-
import android.app.ProgressDialog;
3+
import android.view.View;
4+
import android.widget.ProgressBar;
5+
46
import androidx.appcompat.app.AppCompatActivity;
57

68
import com.google.firebase.auth.FirebaseAuth;
79

810

911
public class BaseActivity extends AppCompatActivity {
1012

11-
private ProgressDialog mProgressDialog;
13+
private ProgressBar mProgressBar;
1214

13-
public void showProgressDialog() {
14-
if (mProgressDialog == null) {
15-
mProgressDialog = new ProgressDialog(this);
16-
mProgressDialog.setCancelable(false);
17-
mProgressDialog.setMessage("Loading...");
18-
}
15+
public void setProgressBar(int resId) {
16+
mProgressBar = findViewById(resId);
17+
}
1918

20-
mProgressDialog.show();
19+
public void showProgressBar() {
20+
if (mProgressBar != null) {
21+
mProgressBar.setVisibility(View.VISIBLE);
22+
}
2123
}
2224

23-
public void hideProgressDialog() {
24-
if (mProgressDialog != null && mProgressDialog.isShowing()) {
25-
mProgressDialog.dismiss();
25+
public void hideProgressBar() {
26+
if (mProgressBar != null) {
27+
mProgressBar.setVisibility(View.INVISIBLE);
2628
}
2729
}
2830

database/app/src/main/java/com/google/firebase/quickstart/database/java/SignInActivity.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ protected void onCreate(Bundle savedInstanceState) {
4545
mPasswordField = findViewById(R.id.fieldPassword);
4646
mSignInButton = findViewById(R.id.buttonSignIn);
4747
mSignUpButton = findViewById(R.id.buttonSignUp);
48+
setProgressBar(R.id.progressBar);
4849

4950
// Click listeners
5051
mSignInButton.setOnClickListener(this);
@@ -67,7 +68,7 @@ private void signIn() {
6768
return;
6869
}
6970

70-
showProgressDialog();
71+
showProgressBar();
7172
String email = mEmailField.getText().toString();
7273
String password = mPasswordField.getText().toString();
7374

@@ -76,7 +77,7 @@ private void signIn() {
7677
@Override
7778
public void onComplete(@NonNull Task<AuthResult> task) {
7879
Log.d(TAG, "signIn:onComplete:" + task.isSuccessful());
79-
hideProgressDialog();
80+
hideProgressBar();
8081

8182
if (task.isSuccessful()) {
8283
onAuthSuccess(task.getResult().getUser());
@@ -94,7 +95,7 @@ private void signUp() {
9495
return;
9596
}
9697

97-
showProgressDialog();
98+
showProgressBar();
9899
String email = mEmailField.getText().toString();
99100
String password = mPasswordField.getText().toString();
100101

@@ -103,7 +104,7 @@ private void signUp() {
103104
@Override
104105
public void onComplete(@NonNull Task<AuthResult> task) {
105106
Log.d(TAG, "createUser:onComplete:" + task.isSuccessful());
106-
hideProgressDialog();
107+
hideProgressBar();
107108

108109
if (task.isSuccessful()) {
109110
onAuthSuccess(task.getResult().getUser());
Lines changed: 10 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,33 +1,26 @@
11
package com.google.firebase.quickstart.database.kotlin
22

3-
import android.app.ProgressDialog
3+
import android.view.View
4+
import android.widget.ProgressBar
45
import androidx.appcompat.app.AppCompatActivity
56
import com.google.firebase.auth.FirebaseAuth
67

78
open class BaseActivity : AppCompatActivity() {
89

9-
private var progressDialog: ProgressDialog? = null
10+
private var progressBar: ProgressBar? = null
1011

1112
val uid: String
1213
get() = FirebaseAuth.getInstance().currentUser!!.uid
1314

14-
fun showProgressDialog() {
15-
if (progressDialog == null) {
16-
val pd = ProgressDialog(this)
17-
pd.setCancelable(false)
18-
pd.setMessage("Loading...")
19-
20-
progressDialog = pd
21-
}
15+
fun setProgressBar(resId: Int) {
16+
progressBar = findViewById(resId)
17+
}
2218

23-
progressDialog?.show()
19+
fun showProgressBar() {
20+
progressBar?.visibility = View.VISIBLE
2421
}
2522

26-
fun hideProgressDialog() {
27-
progressDialog?.let {
28-
if (it.isShowing) {
29-
it.dismiss()
30-
}
31-
}
23+
fun hideProgressBar() {
24+
progressBar?.visibility = View.INVISIBLE
3225
}
3326
}

database/app/src/main/java/com/google/firebase/quickstart/database/kotlin/SignInActivity.kt

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ class SignInActivity : BaseActivity(), View.OnClickListener {
2929
database = FirebaseDatabase.getInstance().reference
3030
auth = FirebaseAuth.getInstance()
3131

32+
setProgressBar(R.id.progressBar)
33+
3234
// Click listeners
3335
buttonSignIn.setOnClickListener(this)
3436
buttonSignUp.setOnClickListener(this)
@@ -49,14 +51,14 @@ class SignInActivity : BaseActivity(), View.OnClickListener {
4951
return
5052
}
5153

52-
showProgressDialog()
54+
showProgressBar()
5355
val email = fieldEmail.text.toString()
5456
val password = fieldPassword.text.toString()
5557

5658
auth.signInWithEmailAndPassword(email, password)
5759
.addOnCompleteListener(this) { task ->
5860
Log.d(TAG, "signIn:onComplete:" + task.isSuccessful)
59-
hideProgressDialog()
61+
hideProgressBar()
6062

6163
if (task.isSuccessful) {
6264
onAuthSuccess(task.result?.user!!)
@@ -73,14 +75,14 @@ class SignInActivity : BaseActivity(), View.OnClickListener {
7375
return
7476
}
7577

76-
showProgressDialog()
78+
showProgressBar()
7779
val email = fieldEmail.text.toString()
7880
val password = fieldPassword.text.toString()
7981

8082
auth.createUserWithEmailAndPassword(email, password)
8183
.addOnCompleteListener(this) { task ->
8284
Log.d(TAG, "createUser:onComplete:" + task.isSuccessful)
83-
hideProgressDialog()
85+
hideProgressBar()
8486

8587
if (task.isSuccessful) {
8688
onAuthSuccess(task.result?.user!!)

database/app/src/main/res/layout/activity_sign_in.xml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,21 @@
88
android:paddingRight="@dimen/activity_horizontal_margin"
99
android:paddingTop="@dimen/activity_vertical_margin">
1010

11+
<ProgressBar
12+
android:id="@+id/progressBar"
13+
android:indeterminate="true"
14+
android:layout_width="match_parent"
15+
android:layout_height="wrap_content"
16+
android:visibility="invisible"
17+
style="?android:attr/progressBarStyleHorizontal"/>
1118

1219
<ImageView
1320
android:id="@+id/icon"
1421
android:layout_width="wrap_content"
1522
android:layout_height="wrap_content"
1623
android:layout_centerHorizontal="true"
1724
android:layout_marginBottom="@dimen/activity_vertical_margin"
25+
android:layout_below="@+id/progressBar"
1826
android:src="@drawable/firebase_lockup_400" />
1927

2028
<LinearLayout

0 commit comments

Comments
 (0)