Skip to content

Commit a70d585

Browse files
authored
Hide edit options for logged-out users in Explore screen (commons-app#5920)
1 parent 3bd7b53 commit a70d585

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -272,6 +272,12 @@ && getParentFragment() instanceof MediaDetailPagerFragment) {
272272

273273
if (!sessionManager.isUserLoggedIn()) {
274274
binding.categoryEditButton.setVisibility(GONE);
275+
binding.descriptionEdit.setVisibility(GONE);
276+
binding.depictionsEditButton.setVisibility(GONE);
277+
} else {
278+
binding.categoryEditButton.setVisibility(VISIBLE);
279+
binding.descriptionEdit.setVisibility(VISIBLE);
280+
binding.depictionsEditButton.setVisibility(VISIBLE);
275281
}
276282

277283
if(applicationKvStore.getBoolean("login_skipped")){
@@ -400,7 +406,6 @@ public void onGlobalLayout() {
400406
}
401407
);
402408
binding.progressBarEdit.setVisibility(GONE);
403-
binding.descriptionEdit.setVisibility(VISIBLE);
404409
}
405410

406411
@Override
@@ -678,7 +683,9 @@ private void updateCategoryList(final String s) {
678683
// Stick in a filler element.
679684
allCategories.add(getString(R.string.detail_panel_cats_none));
680685
}
681-
binding.categoryEditButton.setVisibility(VISIBLE);
686+
if(sessionManager.isUserLoggedIn()) {
687+
binding.categoryEditButton.setVisibility(VISIBLE);
688+
}
682689
rebuildCatList(allCategories);
683690
}
684691

0 commit comments

Comments
 (0)