X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FPopupMenu.java;h=037f0070997b302d4913198437b53744c3d9a477;hb=6235128370382f0fecdf3ba1cc683052c5aa11e7;hp=e812530c901193acb574caba11c06a5b8fd4142e;hpb=797df64fa2a0a30773d0f48f5494d4155e5a8be3;p=jalview.git diff --git a/src/jalview/gui/PopupMenu.java b/src/jalview/gui/PopupMenu.java index e812530..037f007 100644 --- a/src/jalview/gui/PopupMenu.java +++ b/src/jalview/gui/PopupMenu.java @@ -1,13 +1,13 @@ /* - * Jalview - A Sequence Alignment Editor and Viewer (Version 2.7) - * Copyright (C) 2011 J Procter, AM Waterhouse, J Engelhardt, LM Lui, G Barton, M Clamp, S Searle + * Jalview - A Sequence Alignment Editor and Viewer (Version 2.8) + * Copyright (C) 2012 J Procter, AM Waterhouse, LM Lui, J Engelhardt, G Barton, M Clamp, S Searle * * This file is part of Jalview. * * Jalview is free software: you can redistribute it and/or * modify it under the terms of the GNU General Public License * as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version. - * + * * Jalview is distributed in the hope that it will be useful, but * WITHOUT ANY WARRANTY; without even the implied warranty * of MERCHANTABILITY or FITNESS FOR A PARTICULAR @@ -24,7 +24,6 @@ import java.awt.event.*; import javax.swing.*; -import MCview.*; import jalview.analysis.*; import jalview.commands.*; import jalview.datamodel.*; @@ -71,6 +70,7 @@ public class PopupMenu extends JPopupMenu protected JRadioButtonMenuItem BLOSUM62Colour = new JRadioButtonMenuItem(); protected JRadioButtonMenuItem purinePyrimidineColour = new JRadioButtonMenuItem(); + protected JRadioButtonMenuItem RNAInteractionColour = new JRadioButtonMenuItem(); // protected JRadioButtonMenuItem covariationColour = new // JRadioButtonMenuItem(); @@ -85,8 +85,12 @@ public class PopupMenu extends JPopupMenu JMenuItem sequenceName = new JMenuItem(); - SequenceI sequence; + JMenuItem sequenceDetails = new JMenuItem(); + JMenuItem sequenceSelDetails = new JMenuItem(); + + SequenceI sequence; + JMenuItem createGroupMenuItem = new JMenuItem(); JMenuItem unGroupMenuItem = new JMenuItem(); JMenuItem outline = new JMenuItem(); @@ -118,7 +122,11 @@ public class PopupMenu extends JPopupMenu JMenu pdbMenu = new JMenu(); JMenuItem pdbFromFile = new JMenuItem(); - + // JBPNote: Commented these out - Should add these services via the web services menu system. + // JMenuItem ContraFold = new JMenuItem(); + + // JMenuItem RNAFold = new JMenuItem(); + JMenuItem enterPDB = new JMenuItem(); JMenuItem discoverPDB = new JMenuItem(); @@ -162,8 +170,8 @@ public class PopupMenu extends JPopupMenu * @param links * @param groupLinks */ - public PopupMenu(final AlignmentPanel ap, final SequenceI seq, Vector links, - Vector groupLinks) + public PopupMenu(final AlignmentPanel ap, final SequenceI seq, + Vector links, Vector groupLinks) { // ///////////////////////////////////////////////////////// // If this is activated from the sequence panel, the user may want to @@ -189,6 +197,7 @@ public class PopupMenu extends JPopupMenu colours.add(PIDColour); colours.add(BLOSUM62Colour); colours.add(purinePyrimidineColour); + colours.add(RNAInteractionColour); // colours.add(covariationColour); for (int i = 0; i < jalview.io.FormatAdapter.WRITEABLE_FORMATS.length; i++) @@ -236,8 +245,10 @@ public class PopupMenu extends JPopupMenu { public void actionPerformed(ActionEvent e) { - // TODO re JAL-860: optionally open dialog or provide a menu entry allowing user to open just one structure per sequence - new AppJmol(pdb, ap.av.collateForPDB(new PDBEntry[] { pdb })[0], null, ap); + // TODO re JAL-860: optionally open dialog or provide a menu entry + // allowing user to open just one structure per sequence + new AppJmol(pdb, ap.av.collateForPDB(new PDBEntry[] + { pdb })[0], null, ap); // new PDBViewer(pdb, seqs2, null, ap, AppletFormatAdapter.FILE); } @@ -255,30 +266,43 @@ public class PopupMenu extends JPopupMenu } else { - if (ap.av.alignment.isNucleotide() == false) + if (ap.av.getAlignment().isNucleotide() == false) { - structureMenu.remove(viewStructureMenu); + structureMenu.remove(viewStructureMenu); } // structureMenu.remove(colStructureMenu); } - if (ap.av.alignment.isNucleotide() == true) + if (ap.av.getAlignment().isNucleotide() == true) { - AlignmentAnnotation[] aa = ap.av.alignment.getAlignmentAnnotation(); + AlignmentAnnotation[] aa = ap.av.getAlignment() + .getAlignmentAnnotation(); for (int i = 0; i < aa.length; i++) { if (aa[i].getRNAStruc() != null) { final String rnastruc = aa[i].getRNAStruc(); - + final String structureLine = aa[i].label; menuItem = new JMenuItem(); - menuItem.setText("RNA structure - consensus"); + menuItem.setText("2D RNA " + structureLine); menuItem.addActionListener(new java.awt.event.ActionListener() + { public void actionPerformed(ActionEvent e) { - new AppVarna(seq.getSequenceAsString(), rnastruc, seq - .getName(), ap); + //System.out.println("1:"+structureLine); + System.out.println("1:sname"+seq.getName()); + System.out.println("2:seq"+seq); + + //System.out.println("3:"+seq.getSequenceAsString()); + System.out.println("3:strucseq"+rnastruc); + //System.out.println("4:struc"+seq.getRNA()); + System.out.println("5:name"+seq.getName()); + System.out.println("6:ap"+ap); + new AppVarna(structureLine, seq, seq.getSequenceAsString(), rnastruc, seq + .getName(), ap); + //new AppVarna(seq.getName(),seq,rnastruc,seq.getRNA(), seq.getName(), ap); + System.out.println("end"); } }); viewStructureMenu.add(menuItem); @@ -295,17 +319,19 @@ public class PopupMenu extends JPopupMenu if (seqAnno[i].getRNAStruc() != null) { final String rnastruc = seqAnno[i].getRNAStruc(); - - // TODO: make rnastrucF a bit more nice + + // TODO: make rnastrucF a bit more nice menuItem = new JMenuItem(); - menuItem.setText("RNA structure - "+seq.getName()); + menuItem.setText("2D RNA - " + seq.getName()); menuItem.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(ActionEvent e) { // TODO: VARNA does'nt print gaps in the sequence - new AppVarna(seq.getSequenceAsString(), rnastruc, seq - .getName(), ap); + + new AppVarna(seq.getName() + " structure", seq, seq + .getSequenceAsString(), rnastruc, seq.getName(), + ap); } }); viewStructureMenu.add(menuItem); @@ -313,7 +339,6 @@ public class PopupMenu extends JPopupMenu } } - } menuItem = new JMenuItem("Hide Sequences"); @@ -340,9 +365,9 @@ public class PopupMenu extends JPopupMenu sequenceMenu.add(menuItem); } - if (ap.av.hasHiddenRows) + if (ap.av.hasHiddenRows()) { - final int index = ap.av.alignment.findIndex(seq); + final int index = ap.av.getAlignment().findIndex(seq); if (ap.av.adjustForHiddenSeqs(index) - ap.av.adjustForHiddenSeqs(index - 1) > 1) @@ -364,7 +389,8 @@ public class PopupMenu extends JPopupMenu } } // for the case when no sequences are even visible - if (ap.av.hasHiddenRows) { + if (ap.av.hasHiddenRows()) + { { menuItem = new JMenuItem("Reveal All"); menuItem.addActionListener(new ActionListener() @@ -385,10 +411,11 @@ public class PopupMenu extends JPopupMenu } SequenceGroup sg = ap.av.getSelectionGroup(); + boolean isDefinedGroup = (sg!=null) ? ap.av.getAlignment().getGroups().contains(sg) : false; - if (sg != null&& sg.getSize()>0) - { - groupName.setText("Name: "+sg.getName()); + if (sg != null && sg.getSize() > 0) + { + groupName.setText("Name: " + sg.getName()); groupName.setText("Edit name and description of current group."); if (sg.cs instanceof ZappoColourScheme) @@ -439,6 +466,8 @@ public class PopupMenu extends JPopupMenu { purinePyrimidineColour.setSelected(true); } + + /* * else if (sg.cs instanceof CovariationColourScheme) { * covariationColour.setSelected(true); } @@ -462,36 +491,43 @@ public class PopupMenu extends JPopupMenu buildGroupURLMenu(sg, groupLinks); } // Add a 'show all structures' for the current selection - Hashtable pdbe=new Hashtable(); - SequenceI sqass=null; - for (SequenceI sq: ap.av.getSequenceSelection()) + Hashtable pdbe = new Hashtable(); + SequenceI sqass = null; + for (SequenceI sq : ap.av.getSequenceSelection()) { - Vector pes = (Vector) sq.getDatasetSequence().getPDBId(); - if (pes!=null) { - for (PDBEntry pe: pes) + Vector pes = (Vector) sq.getDatasetSequence() + .getPDBId(); + if (pes != null) + { + for (PDBEntry pe : pes) { - pdbe.put(pe.getId(), pe); - if (sqass==null) + pdbe.put(pe.getId(), pe); + if (sqass == null) { sqass = sq; } } } } - if (pdbe.size()>0) + if (pdbe.size() > 0) { - final PDBEntry[] pe = pdbe.values().toArray(new PDBEntry[pdbe.size()]); + final PDBEntry[] pe = pdbe.values().toArray( + new PDBEntry[pdbe.size()]); final JMenuItem gpdbview; - if (pdbe.size()==1) + if (pdbe.size() == 1) + { + structureMenu.add(gpdbview = new JMenuItem("View structure for " + + sqass.getDisplayId(false))); + } + else { - structureMenu.add(gpdbview=new JMenuItem("View structure for "+sqass.getDisplayId(false))); - } else { - structureMenu.add(gpdbview=new JMenuItem("View all "+pdbe.size()+" structures.")); + structureMenu.add(gpdbview = new JMenuItem("View all " + + pdbe.size() + " structures.")); } gpdbview.setToolTipText("Open a new Jmol view with all structures associated with the current selection and superimpose them using the alignment."); gpdbview.addActionListener(new ActionListener() { - + @Override public void actionPerformed(ActionEvent e) { @@ -506,9 +542,15 @@ public class PopupMenu extends JPopupMenu editMenu.setVisible(false); } - if (!ap.av.alignment.getGroups().contains(sg)) + if (!isDefinedGroup) { + createGroupMenuItem.setVisible(true); unGroupMenuItem.setVisible(false); + jMenu1.setText("Edit New Group"); + } else { + createGroupMenuItem.setVisible(false); + unGroupMenuItem.setVisible(true); + jMenu1.setText("Edit Group"); } if (seq == null) @@ -542,7 +584,7 @@ public class PopupMenu extends JPopupMenu continue; } final String label = urlLink.getLabel(); - if (seq!=null && urlLink.isDynamic()) + if (seq != null && urlLink.isDynamic()) { // collect matching db-refs @@ -895,6 +937,23 @@ public class PopupMenu extends JPopupMenu sequenceName_actionPerformed(); } }); + sequenceDetails.setText("Sequence Details ..."); + sequenceDetails.addActionListener(new java.awt.event.ActionListener() + { + public void actionPerformed(ActionEvent e) + { + sequenceDetails_actionPerformed(); + } + }); + sequenceSelDetails.setText("Sequence Details ..."); + sequenceSelDetails + .addActionListener(new java.awt.event.ActionListener() + { + public void actionPerformed(ActionEvent e) + { + sequenceSelectionDetails_actionPerformed(); + } + }); PIDColour.setFocusPainted(false); unGroupMenuItem.setText("Remove Group"); unGroupMenuItem.addActionListener(new java.awt.event.ActionListener() @@ -904,6 +963,14 @@ public class PopupMenu extends JPopupMenu unGroupMenuItem_actionPerformed(); } }); + createGroupMenuItem.setText("Create Group"); + createGroupMenuItem.addActionListener(new java.awt.event.ActionListener() + { + public void actionPerformed(ActionEvent e) + { + createGroupMenuItem_actionPerformed(); + } + }); outline.setText("Border colour"); outline.addActionListener(new java.awt.event.ActionListener() @@ -1007,6 +1074,32 @@ public class PopupMenu extends JPopupMenu pdbFromFile_actionPerformed(); } }); +// RNAFold.setText("From RNA Fold with predict2D"); +// RNAFold.addActionListener(new ActionListener() +// { +// public void actionPerformed(ActionEvent e) +// { +// try { +// RNAFold_actionPerformed(); +// } catch (Exception e1) { +// // TODO Auto-generated catch block +// e1.printStackTrace(); +// } +// } +// }); +// ContraFold.setText("From Contra Fold with predict2D"); +// ContraFold.addActionListener(new ActionListener() +// { +// public void actionPerformed(ActionEvent e) +// { +// try { +// ContraFold_actionPerformed(); +// } catch (Exception e1) { +// // TODO Auto-generated catch block +// e1.printStackTrace(); +// } +// } +// }); enterPDB.setText("Enter PDB Id"); enterPDB.addActionListener(new ActionListener() { @@ -1052,22 +1145,25 @@ public class PopupMenu extends JPopupMenu editSequence_actionPerformed(actionEvent); } }); + /* * annotationMenuItem.setText("By Annotation"); * annotationMenuItem.addActionListener(new ActionListener() { public void * actionPerformed(ActionEvent actionEvent) { * annotationMenuItem_actionPerformed(actionEvent); } }); */ - + groupMenu.add(sequenceSelDetails); add(groupMenu); - add(sequenceMenu); this.add(structureMenu); groupMenu.add(editMenu); groupMenu.add(outputMenu); groupMenu.add(sequenceFeature); + groupMenu.add(createGroupMenuItem); + groupMenu.add(unGroupMenuItem); groupMenu.add(jMenu1); sequenceMenu.add(sequenceName); + sequenceMenu.add(sequenceDetails); colourMenu.add(textColour); colourMenu.add(noColourmenuItem); colourMenu.add(clustalColour); @@ -1082,6 +1178,8 @@ public class PopupMenu extends JPopupMenu colourMenu.add(buriedColour); colourMenu.add(nucleotideMenuItem); if (ap.getAlignment().isNucleotide()) { + // JBPNote - commented since the colourscheme isn't functional + // colourMenu.add(RNAInteractionColour); colourMenu.add(purinePyrimidineColour); } // colourMenu.add(covariationColour); @@ -1117,10 +1215,12 @@ public class PopupMenu extends JPopupMenu editMenu.add(lowerCase); editMenu.add(toggle); pdbMenu.add(pdbFromFile); + // JBPNote: These shouldn't be added here - should appear in a generic 'apply web service to this sequence menu' + // pdbMenu.add(RNAFold); + // pdbMenu.add(ContraFold); pdbMenu.add(enterPDB); pdbMenu.add(discoverPDB); jMenu1.add(groupName); - jMenu1.add(unGroupMenuItem); jMenu1.add(colourMenu); jMenu1.add(showBoxes); jMenu1.add(showText); @@ -1245,6 +1345,8 @@ public class PopupMenu extends JPopupMenu purinePyrimidineColour_actionPerformed(); } }); + + /* * covariationColour.addActionListener(new java.awt.event.ActionListener() { * public void actionPerformed(ActionEvent e) { @@ -1262,6 +1364,44 @@ public class PopupMenu extends JPopupMenu }); } + protected void sequenceSelectionDetails_actionPerformed() + { + createSequenceDetailsReport(ap.av.getSequenceSelection()); + } + + protected void sequenceDetails_actionPerformed() + { + createSequenceDetailsReport(new SequenceI[] + { sequence }); + } + + public void createSequenceDetailsReport(SequenceI[] sequences) + { + CutAndPasteHtmlTransfer cap = new CutAndPasteHtmlTransfer(); + StringBuffer contents = new StringBuffer(); + for (SequenceI seq : sequences) + { + contents.append("

Annotation for " + seq.getDisplayId(true) + + "

"); + new SequenceAnnotationReport(null) + .createSequenceAnnotationReport( + contents, + seq, + true, + true, + false, + (ap.seqPanel.seqCanvas.fr != null) ? ap.seqPanel.seqCanvas.fr.minmax + : null); + contents.append("

"); + } + cap.setText("" + contents.toString() + ""); + + Desktop.instance.addInternalFrame(cap, "Sequence Details for " + + (sequences.length == 1 ? sequences[0].getDisplayId(true) + : "Selection"), 500, 400); + + } + protected void showNonconserved_actionPerformed() { getGroup().setShowNonconserved(displayNonconserved.isSelected()); @@ -1288,9 +1428,7 @@ public class PopupMenu extends JPopupMenu protected void clustalColour_actionPerformed() { SequenceGroup sg = getGroup(); - sg.cs = new ClustalxColourScheme( - sg.getSequences(ap.av.hiddenRepSequences), - ap.av.alignment.getWidth()); + sg.cs = new ClustalxColourScheme(sg, ap.av.getHiddenRepSequences()); refresh(); } @@ -1396,6 +1534,7 @@ public class PopupMenu extends JPopupMenu refresh(); } + /* * protected void covariationColour_actionPerformed() { getGroup().cs = new * CovariationColourScheme(sequence.getAnnotation()[0]); refresh(); } @@ -1417,8 +1556,8 @@ public class PopupMenu extends JPopupMenu if (abovePIDColour.isSelected()) { sg.cs.setConsensus(AAFrequency.calculate( - sg.getSequences(ap.av.hiddenRepSequences), sg.getStartRes(), - sg.getEndRes() + 1)); + sg.getSequences(ap.av.getHiddenRepSequences()), + sg.getStartRes(), sg.getEndRes() + 1)); int threshold = SliderPanel.setPIDSliderSource(ap, sg.cs, getGroup() .getName()); @@ -1471,8 +1610,8 @@ public class PopupMenu extends JPopupMenu SequenceGroup sg = getGroup(); sg.cs = new PIDColourScheme(); sg.cs.setConsensus(AAFrequency.calculate( - sg.getSequences(ap.av.hiddenRepSequences), sg.getStartRes(), - sg.getEndRes() + 1)); + sg.getSequences(ap.av.getHiddenRepSequences()), + sg.getStartRes(), sg.getEndRes() + 1)); refresh(); } @@ -1489,8 +1628,8 @@ public class PopupMenu extends JPopupMenu sg.cs = new Blosum62ColourScheme(); sg.cs.setConsensus(AAFrequency.calculate( - sg.getSequences(ap.av.hiddenRepSequences), sg.getStartRes(), - sg.getEndRes() + 1)); + sg.getSequences(ap.av.getHiddenRepSequences()), + sg.getStartRes(), sg.getEndRes() + 1)); refresh(); } @@ -1524,12 +1663,12 @@ public class PopupMenu extends JPopupMenu if (conservationMenuItem.isSelected()) { Conservation c = new Conservation("Group", - ResidueProperties.propHash, 3, - sg.getSequences(ap.av.hiddenRepSequences), sg.getStartRes(), + ResidueProperties.propHash, 3, sg.getSequences(ap.av + .getHiddenRepSequences()), sg.getStartRes(), sg.getEndRes() + 1); c.calculate(); - c.verdict(false, ap.av.ConsPercGaps); + c.verdict(false, ap.av.getConsPercGaps()); sg.cs.setConservation(c); @@ -1557,7 +1696,7 @@ public class PopupMenu extends JPopupMenu sequence.getAnnotation()[0], null, AnnotationColourGradient.NO_THRESHOLD); - acg.predefinedColours = true; + acg.setPredefinedColours(true); sg.cs = acg; refresh(); @@ -1599,7 +1738,7 @@ public class PopupMenu extends JPopupMenu // this method won't add a new group if it already exists if (sg != null) { - ap.av.alignment.addGroup(sg); + ap.av.getAlignment().addGroup(sg); } return sg; @@ -1653,10 +1792,15 @@ public class PopupMenu extends JPopupMenu void unGroupMenuItem_actionPerformed() { SequenceGroup sg = ap.av.getSelectionGroup(); - ap.av.alignment.deleteGroup(sg); + ap.av.getAlignment().deleteGroup(sg); ap.av.setSelectionGroup(null); refresh(); } + void createGroupMenuItem_actionPerformed() + { + getGroup(); // implicitly creates group - note - should apply defaults / use standard alignment window logic for this + refresh(); + } /** * DOCUMENT ME! @@ -1807,8 +1951,8 @@ public class PopupMenu extends JPopupMenu } ChangeCaseCommand caseCommand = new ChangeCaseCommand(description, - sg.getSequencesAsArray(ap.av.hiddenRepSequences), startEnd, - caseChange); + sg.getSequencesAsArray(ap.av.getHiddenRepSequences()), + startEnd, caseChange); ap.alignFrame.addHistoryItem(caseCommand); @@ -1834,7 +1978,8 @@ public class PopupMenu extends JPopupMenu ColumnSelection csel = new ColumnSelection(ap.av.getColumnSelection()); omitHidden = ap.av.getViewAsString(true); Alignment oal = new Alignment(ap.av.getSequenceSelection()); - AlignmentAnnotation[] nala = ap.av.alignment.getAlignmentAnnotation(); + AlignmentAnnotation[] nala = ap.av.getAlignment() + .getAlignmentAnnotation(); if (nala != null) { for (int i = 0; i < nala.length; i++) @@ -1853,8 +1998,10 @@ public class PopupMenu extends JPopupMenu jalview.io.JalviewFileChooser chooser = new jalview.io.JalviewFileChooser( jalview.bin.Cache.getProperty("LAST_DIRECTORY")); chooser.setFileView(new jalview.io.JalviewFileView()); - chooser.setDialogTitle("Select a PDB file for "+sequence.getDisplayId(false)); - chooser.setToolTipText("Load a PDB file and associate it with sequence '"+sequence.getDisplayId(false)+"'"); + chooser.setDialogTitle("Select a PDB file for " + + sequence.getDisplayId(false)); + chooser.setToolTipText("Load a PDB file and associate it with sequence '" + + sequence.getDisplayId(false) + "'"); int value = chooser.showOpenDialog(null); @@ -1862,11 +2009,23 @@ public class PopupMenu extends JPopupMenu { String choice = chooser.getSelectedFile().getPath(); jalview.bin.Cache.setProperty("LAST_DIRECTORY", choice); - new AssociatePdbFileWithSeq().associatePdbWithSeq(choice, jalview.io.AppletFormatAdapter.FILE, sequence, true); + new AssociatePdbFileWithSeq().associatePdbWithSeq(choice, + jalview.io.AppletFormatAdapter.FILE, sequence, true); } } - + // JBNote: commented out - these won't be instantiated here...! +// public void RNAFold_actionPerformed() throws Exception +// { +// Predict2D P2D = new Predict2D(); +// P2D.getStructure2DFromRNAFold("toto"); +// } +// +// public void ContraFold_actionPerformed() throws Exception +// { +// Predict2D P2D = new Predict2D(); +// P2D.getStructure2DFromContraFold("toto"); +// } public void enterPDB_actionPerformed() { String id = JOptionPane.showInternalInputDialog(Desktop.desktop, @@ -1883,9 +2042,9 @@ public class PopupMenu extends JPopupMenu public void discoverPDB_actionPerformed() { - final SequenceI[] sequences = ((ap.av.selectionGroup == null) ? new SequenceI[] + final SequenceI[] sequences = ((ap.av.getSelectionGroup() == null) ? new SequenceI[] { sequence } - : ap.av.selectionGroup.getSequencesInOrder(ap.av.alignment)); + : ap.av.getSequenceSelection()); Thread discpdb = new Thread(new Runnable() { public void run() @@ -1948,16 +2107,16 @@ public class PopupMenu extends JPopupMenu public void colourByStructure(String pdbid) { - Annotation[] anots = ap.av.getStructureSelectionManager().colourSequenceFromStructure( - sequence, pdbid); + Annotation[] anots = ap.av.getStructureSelectionManager() + .colourSequenceFromStructure(sequence, pdbid); AlignmentAnnotation an = new AlignmentAnnotation("Structure", "Coloured by " + pdbid, anots); - ap.av.alignment.addAnnotation(an); + ap.av.getAlignment().addAnnotation(an); an.createSequenceMapping(sequence, 0, true); // an.adjustForAlignment(); - ap.av.alignment.setAnnotationIndex(an, 0); + ap.av.getAlignment().setAnnotationIndex(an, 0); ap.adjustAnnotationHeight(); @@ -1984,8 +2143,8 @@ public class PopupMenu extends JPopupMenu EditCommand editCommand = new EditCommand("Edit Sequences", EditCommand.REPLACE, dialog.getName().replace(' ', ap.av.getGapCharacter()), - sg.getSequencesAsArray(ap.av.hiddenRepSequences), - sg.getStartRes(), sg.getEndRes() + 1, ap.av.alignment); + sg.getSequencesAsArray(ap.av.getHiddenRepSequences()), + sg.getStartRes(), sg.getEndRes() + 1, ap.av.getAlignment()); ap.alignFrame.addHistoryItem(editCommand);