fix enabling / disabling next/previous actions

This commit is contained in:
Antoine d'Aligny 2019-10-06 00:16:35 +02:00 committed by Daniel Micay
parent 95ab6b0374
commit 46117e6e20

View File

@ -121,6 +121,7 @@ public class PdfViewer extends AppCompatActivity implements LoaderManager.Loader
@JavascriptInterface
public void setNumPages(int numPages) {
mNumPages = numPages;
runOnUiThread(PdfViewer.this::invalidateOptionsMenu);
}
@JavascriptInterface
@ -371,11 +372,9 @@ public class PdfViewer extends AppCompatActivity implements LoaderManager.Loader
private static void enableDisableMenuItem(MenuItem item, boolean enable) {
if (enable) {
if (!item.isEnabled()) {
item.setEnabled(true);
item.getIcon().setAlpha(ALPHA_HIGH);
}
} else if (item.isEnabled()) {
item.setEnabled(true);
item.getIcon().setAlpha(ALPHA_HIGH);
} else {
item.setEnabled(false);
item.getIcon().setAlpha(ALPHA_LOW);
}
@ -386,6 +385,7 @@ public class PdfViewer extends AppCompatActivity implements LoaderManager.Loader
mPage = selected_page;
renderPage(0);
showPageNumber();
invalidateOptionsMenu();
}
}
@ -475,6 +475,8 @@ public class PdfViewer extends AppCompatActivity implements LoaderManager.Loader
enableDisableMenuItem(menu.findItem(R.id.action_zoom_in), mZoomRatio != MAX_ZOOM_RATIO);
enableDisableMenuItem(menu.findItem(R.id.action_zoom_out), mZoomRatio != MIN_ZOOM_RATIO);
enableDisableMenuItem(menu.findItem(R.id.action_next), mPage < mNumPages);
enableDisableMenuItem(menu.findItem(R.id.action_previous), mPage > 1);
return true;
}