From f0dc195035022a7d4e7552bf640d6ae70012c18e Mon Sep 17 00:00:00 2001 From: Ashish Date: Mon, 25 Oct 2021 12:25:12 +0530 Subject: [PATCH] Don't show dummy achievements data - show data only when loaded --- .../contributions/ContributionsListFragment.java | 12 +++++++++++- .../profile/achievements/AchievementsFragment.java | 6 ++++-- app/src/main/res/layout/fragment_achievements.xml | 3 +++ 3 files changed, 18 insertions(+), 3 deletions(-) diff --git a/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsListFragment.java b/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsListFragment.java index ebe3596950..86ab4f016c 100644 --- a/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsListFragment.java +++ b/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsListFragment.java @@ -30,6 +30,7 @@ import androidx.recyclerview.widget.SimpleItemAnimator; import butterknife.BindView; import butterknife.ButterKnife; +import butterknife.OnClick; import com.google.android.material.floatingactionbutton.FloatingActionButton; import fr.free.nrw.commons.Media; import fr.free.nrw.commons.R; @@ -137,7 +138,7 @@ public View onCreateView( ButterKnife.bind(this, view); contributionsListPresenter.onAttachView(this); - if (sessionManager.getUserName().equals(userName)) { + if (Objects.equals(sessionManager.getUserName(), userName)) { tvContributionsOfUser.setVisibility(GONE); fab_layout.setVisibility(VISIBLE); } else { @@ -302,6 +303,15 @@ private void setListeners() { }); } + /** + * Launch Custom Selector. + */ + @OnClick(R.id.fab_custom_gallery) + void launchCustomSelector(){ + controller.initiateCustomGalleryPickWithPermission(getActivity()); + animateFAB(isFabOpen); + } + private void animateFAB(final boolean isFabOpen) { this.isFabOpen = !isFabOpen; if (fabPlus.isShown()) { diff --git a/app/src/main/java/fr/free/nrw/commons/profile/achievements/AchievementsFragment.java b/app/src/main/java/fr/free/nrw/commons/profile/achievements/AchievementsFragment.java index 36ce969fd1..7f4b10c69a 100644 --- a/app/src/main/java/fr/free/nrw/commons/profile/achievements/AchievementsFragment.java +++ b/app/src/main/java/fr/free/nrw/commons/profile/achievements/AchievementsFragment.java @@ -325,7 +325,7 @@ private void setUploadProgress(int uploadCount){ if (uploadCount==0){ setZeroAchievements(); }else { - + imagesUploadedProgressbar.setVisibility(View.VISIBLE); imagesUploadedProgressbar.setProgress (100*uploadCount/levelInfo.getMaxUploadCount()); imagesUploadedProgressbar.setProgressTextFormatPattern @@ -337,7 +337,7 @@ private void setUploadProgress(int uploadCount){ private void setZeroAchievements() { AlertDialog.Builder builder = new AlertDialog.Builder(getActivity()) .setMessage( - !sessionManager.getUserName().equals(userName) ? + !Objects.equals(sessionManager.getUserName(), userName) ? getString(R.string.no_achievements_yet, userName) : getString(R.string.you_have_no_achievements_yet) ) @@ -361,6 +361,7 @@ private void setZeroAchievements() { * @param notRevertPercentage */ private void setImageRevertPercentage(int notRevertPercentage){ + imageRevertsProgressbar.setVisibility(View.VISIBLE); imageRevertsProgressbar.setProgress(notRevertPercentage); String revertPercentage = Integer.toString(notRevertPercentage); imageRevertsProgressbar.setProgressTextFormatPattern(revertPercentage + "%%"); @@ -373,6 +374,7 @@ private void setImageRevertPercentage(int notRevertPercentage){ * @param achievements */ private void inflateAchievements(Achievements achievements) { + imagesUsedByWikiProgressBar.setVisibility(View.VISIBLE); thanksReceived.setText(String.valueOf(achievements.getThanksReceived())); imagesUsedByWikiProgressBar.setProgress (100 * achievements.getUniqueUsedImages() / levelInfo.getMaxUniqueImages()); diff --git a/app/src/main/res/layout/fragment_achievements.xml b/app/src/main/res/layout/fragment_achievements.xml index fb9c068923..bac6092329 100644 --- a/app/src/main/res/layout/fragment_achievements.xml +++ b/app/src/main/res/layout/fragment_achievements.xml @@ -144,6 +144,7 @@ app:progress_start_color="#3A3381" app:progress_stroke_width="@dimen/progressbar_stroke" app:progress_text_format_pattern="573/110" + android:visibility="gone" app:progress_text_color="@color/secondaryColor" app:style="solid_line" /> @@ -211,6 +212,7 @@ android:layout_marginRight="@dimen/large_gap" android:layout_marginEnd="@dimen/large_gap" android:progress="50" + android:visibility="gone" android:id="@+id/image_reverts_progressbar" app:progress_end_color="#8C8B98" app:progress_start_color="#3A3381" @@ -278,6 +280,7 @@ app:progress_end_color="#8C8B98" app:progress_start_color="#3A3381" app:progress_stroke_width="2.5dp" + android:visibility="gone" app:progress_text_color="@color/secondaryColor" app:progress_text_format_pattern="12/24" app:style="solid_line" />