Merge branch 'bug/JAL-3166npeAllRowsHidden' into develop

diff --combined src/jalview/gui/IdPanel.java
index 34bda10,fb37445..3888a74
--- 1/src/jalview/gui/IdPanel.java
--- 2/src/jalview/gui/IdPanel.java
+++ b/src/jalview/gui/IdPanel.java
@@@ -330,8 -330,11 +330,11 @@@ public class IdPanel extends JPane
       *  build a new links menu based on the current links
       *  and any non-positional features
       */
+     List<SequenceFeature> features = null;
+     if (sq != null)
+     {
      List<String> nlinks = Preferences.sequenceUrlLinks.getLinksForMenu();
-     List<SequenceFeature> features = sq.getFeatures().getNonPositionalFeatures();
+       features = sq.getFeatures().getNonPositionalFeatures();
      for (SequenceFeature sf : features)
      {
        if (sf.links != null)
@@@ -342,6 -345,7 +345,7 @@@
          }
        }
      }
+     }
  
      PopupMenu pop = new PopupMenu(alignPanel, sq, features,
              Preferences.getGroupURLLinks());
@@@ -358,7 -362,7 +362,7 @@@
      lastid = seq;
  
      SequenceI pickedSeq = av.getAlignment().getSequenceAt(seq);
 -    av.getSelectionGroup().addOrRemove(pickedSeq, true);
 +    av.getSelectionGroup().addOrRemove(pickedSeq, false);
    }
  
    /**
@@@ -393,7 -397,7 +397,7 @@@
      for (int i = start; i <= end; i++)
      {
        av.getSelectionGroup().addSequence(av.getAlignment().getSequenceAt(i),
 -              i == end);
 +              false);
      }
    }