diff --git a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java index 8e6518c44..d9e11076d 100644 --- a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java +++ b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java @@ -626,6 +626,7 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment implements categoryEditSearchRecyclerViewAdapter.addToCategories(media.getCategories()); updateSelectedCategoriesTextView(categoryEditSearchRecyclerViewAdapter.getCategories()); + categoryRecyclerView.setVisibility(GONE); updateCategoryList(); if (media.getAuthor() == null || media.getAuthor().equals("")) { @@ -661,20 +662,28 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment implements public void updateSelectedCategoriesTextView(List selectedCategories) { if (selectedCategories == null || selectedCategories.size() == 0) { updateCategoriesButton.setClickable(false); - } - if (selectedCategories != null) { + updateCategoriesButton.setAlpha(.5f); + } else { existingCategories.setText(StringUtils.join(selectedCategories,", ")); - updateCategoriesButton.setClickable(true); + if (selectedCategories.equals(media.getCategories())) { + updateCategoriesButton.setClickable(false); + updateCategoriesButton.setAlpha(.5f); + } else { + updateCategoriesButton.setClickable(true); + updateCategoriesButton.setAlpha(1f); + } } } @Override public void noResultsFound() { + categoryRecyclerView.setVisibility(GONE); noResultsFound.setVisibility(VISIBLE); } @Override public void someResultsFound() { + categoryRecyclerView.setVisibility(VISIBLE); noResultsFound.setVisibility(GONE); } diff --git a/app/src/main/res/layout/layout_edit_categories.xml b/app/src/main/res/layout/layout_edit_categories.xml index ccbfe822d..5b1cc28ce 100644 --- a/app/src/main/res/layout/layout_edit_categories.xml +++ b/app/src/main/res/layout/layout_edit_categories.xml @@ -1,114 +1,119 @@ - + - - - - - - - - - - - - - - + android:background="?attr/mainBackground" + android:elevation="30dp"> - + + + android:layout_height="wrap_content" + android:layout_marginTop="@dimen/standard_gap" + > + + + + + + + + + + -