Merge pull request #1429 from mfietz/issue/episodes-share
*EpisodesFragment: Fix context menu sharing
This commit is contained in:
commit
87daa77ce4
|
@ -261,9 +261,13 @@ public class AllEpisodesFragment extends Fragment {
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean onContextItemSelected(MenuItem item) {
|
public boolean onContextItemSelected(MenuItem item) {
|
||||||
|
Log.d(TAG, "onContextItemSelected() called with: " + "item = [" + item + "]");
|
||||||
if(!isVisible()) {
|
if(!isVisible()) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
if(item.getItemId() == R.id.share_item) {
|
||||||
|
return true; // avoids that the position is reset when we need it in the submenu
|
||||||
|
}
|
||||||
int pos = listAdapter.getPosition();
|
int pos = listAdapter.getPosition();
|
||||||
if(pos < 0) {
|
if(pos < 0) {
|
||||||
return false;
|
return false;
|
||||||
|
|
|
@ -320,6 +320,7 @@ public class QueueFragment extends Fragment {
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean onContextItemSelected(MenuItem item) {
|
public boolean onContextItemSelected(MenuItem item) {
|
||||||
|
Log.d(TAG, "onContextItemSelected() called with: " + "item = [" + item + "]");
|
||||||
if(!isVisible()) {
|
if(!isVisible()) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue