Skip to content

Commit f3818c6

Browse files
committed
Remove old and unused retry buttons as we do for their functions
1 parent 915a888 commit f3818c6

File tree

2 files changed

+0
-29
lines changed

2 files changed

+0
-29
lines changed

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

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -189,16 +189,6 @@ public boolean onOptionsItemSelected(MenuItem item) {
189189
// Set wallpaper
190190
setWallpaper(m);
191191
return true;
192-
case R.id.menu_retry_current_image:
193-
// Retry
194-
//((MainActivity) getActivity()).retryUpload(pager.getCurrentItem());
195-
getActivity().getSupportFragmentManager().popBackStack();
196-
return true;
197-
case R.id.menu_cancel_current_image:
198-
// todo: delete image
199-
//((MainActivity) getActivity()).deleteUpload(pager.getCurrentItem());
200-
getActivity().getSupportFragmentManager().popBackStack();
201-
return true;
202192
default:
203193
return super.onOptionsItemSelected(item);
204194
}
@@ -279,8 +269,6 @@ public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
279269
Media m = provider.getMediaAtPosition(pager.getCurrentItem());
280270
if (m != null) {
281271
// Enable default set of actions, then re-enable different set of actions only if it is a failed contrib
282-
menu.findItem(R.id.menu_retry_current_image).setEnabled(false).setVisible(false);
283-
menu.findItem(R.id.menu_cancel_current_image).setEnabled(false).setVisible(false);
284272
menu.findItem(R.id.menu_browser_current_image).setEnabled(true).setVisible(true);
285273
menu.findItem(R.id.menu_share_current_image).setEnabled(true).setVisible(true);
286274
menu.findItem(R.id.menu_download_current_image).setEnabled(true).setVisible(true);
@@ -297,17 +285,13 @@ public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
297285
Contribution c = (Contribution) m;
298286
switch (c.getState()) {
299287
case Contribution.STATE_FAILED:
300-
menu.findItem(R.id.menu_retry_current_image).setEnabled(true).setVisible(true);
301-
menu.findItem(R.id.menu_cancel_current_image).setEnabled(true).setVisible(true);
302288
menu.findItem(R.id.menu_browser_current_image).setEnabled(false).setVisible(false);
303289
menu.findItem(R.id.menu_share_current_image).setEnabled(false).setVisible(false);
304290
menu.findItem(R.id.menu_download_current_image).setEnabled(false).setVisible(false);
305291
menu.findItem(R.id.menu_bookmark_current_image).setEnabled(false).setVisible(false);
306292
break;
307293
case Contribution.STATE_IN_PROGRESS:
308294
case Contribution.STATE_QUEUED:
309-
menu.findItem(R.id.menu_retry_current_image).setEnabled(false).setVisible(false);
310-
menu.findItem(R.id.menu_cancel_current_image).setEnabled(false).setVisible(false);
311295
menu.findItem(R.id.menu_browser_current_image).setEnabled(false).setVisible(false);
312296
menu.findItem(R.id.menu_share_current_image).setEnabled(false).setVisible(false);
313297
menu.findItem(R.id.menu_download_current_image).setEnabled(false).setVisible(false);

app/src/main/res/menu/fragment_image_detail.xml

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,5 @@
2525
android:id="@+id/menu_set_as_wallpaper"
2626
android:title="@string/menu_set_wallpaper"
2727
app:showAsAction="never" />
28-
<item
29-
android:id="@+id/menu_retry_current_image"
30-
android:enabled="false"
31-
android:icon="@drawable/ic_undo_white_24dp"
32-
android:title="@string/menu_retry_upload"
33-
android:visible="false"
34-
app:showAsAction="ifRoom|withText" />
35-
<item
36-
android:id="@+id/menu_cancel_current_image"
37-
android:enabled="false"
38-
android:title="@string/menu_cancel_upload"
39-
android:visible="false"
40-
app:showAsAction="never" />
4128

4229
</menu>

0 commit comments

Comments
 (0)