X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FSeqPanel.java;h=2d44a0f1fd8c86e70c6cb8772283c5d81c0bc046;hb=1f959f4a5bc07769cde4c2577022ad64e55b4a1a;hp=01653b6221d0efd44a1ba7aa5910d8a301861519;hpb=4099019578bbeb81b76154922851ada38fb0017d;p=jalview.git diff --git a/src/jalview/gui/SeqPanel.java b/src/jalview/gui/SeqPanel.java index 01653b6..2d44a0f 100644 --- a/src/jalview/gui/SeqPanel.java +++ b/src/jalview/gui/SeqPanel.java @@ -21,6 +21,7 @@ package jalview.gui; import jalview.api.AlignViewportI; +import jalview.bin.Cache; import jalview.commands.EditCommand; import jalview.commands.EditCommand.Action; import jalview.commands.EditCommand.Edit; @@ -54,11 +55,12 @@ import java.awt.event.MouseListener; import java.awt.event.MouseMotionListener; import java.awt.event.MouseWheelEvent; import java.awt.event.MouseWheelListener; +import java.util.ArrayList; import java.util.List; -import java.util.Vector; import javax.swing.JOptionPane; import javax.swing.JPanel; +import javax.swing.SwingUtilities; import javax.swing.ToolTipManager; /** @@ -192,8 +194,7 @@ public class SeqPanel extends JPanel implements MouseListener, } int cHeight = av.getAlignment().getHeight() * av.getCharHeight() - + hgap - + seqCanvas.getAnnotationHeight(); + + hgap + seqCanvas.getAnnotationHeight(); int y = evt.getY(); y -= hgap; @@ -250,8 +251,7 @@ public class SeqPanel extends JPanel implements MouseListener, } int cHeight = av.getAlignment().getHeight() * av.getCharHeight() - + hgap - + seqCanvas.getAnnotationHeight(); + + hgap + seqCanvas.getAnnotationHeight(); y -= hgap; @@ -511,6 +511,7 @@ public class SeqPanel extends JPanel implements MouseListener, void insertNucAtCursor(boolean group, String nuc) { + // TODO not called - delete? groupEditing = group; startseq = seqCanvas.cursorY; lastres = seqCanvas.cursorX; @@ -601,7 +602,7 @@ public class SeqPanel extends JPanel implements MouseListener, { lastMousePress = evt.getPoint(); - if (javax.swing.SwingUtilities.isMiddleMouseButton(evt)) + if (SwingUtilities.isMiddleMouseButton(evt)) { mouseWheelPressed = true; return; @@ -668,11 +669,12 @@ public class SeqPanel extends JPanel implements MouseListener, @Override public void highlightSequence(SearchResults results) { - if (results == null || results.equals(lastSearchResults)) { + if (results == null || results.equals(lastSearchResults)) + { return; } lastSearchResults = results; - + if (av.isFollowHighlight()) { /* @@ -681,7 +683,7 @@ public class SeqPanel extends JPanel implements MouseListener, * * @see AlignmentPanel#adjustmentValueChanged */ - ap.setFollowingComplementScroll(true); + ap.setDontScrollComplement(true); if (ap.scrollToPosition(results, false)) { seqCanvas.revalidate(); @@ -696,6 +698,7 @@ public class SeqPanel extends JPanel implements MouseListener, { return this.ap == null ? null : this.ap.av; } + @Override public void updateColours(SequenceI seq, int index) { @@ -767,9 +770,9 @@ public class SeqPanel extends JPanel implements MouseListener, if (av.isShowSequenceFeatures()) { int rpos; - List features = ap.getFeatureRenderer().findFeaturesAtRes( - sequence.getDatasetSequence(), - rpos = sequence.findPosition(res)); + List features = ap.getFeatureRenderer() + .findFeaturesAtRes(sequence.getDatasetSequence(), + rpos = sequence.findPosition(res)); seqARep.appendFeatures(tooltipText, rpos, features, this.ap.getSeqPanel().seqCanvas.fr.getMinMax()); } @@ -801,6 +804,7 @@ public class SeqPanel extends JPanel implements MouseListener, * * @see javax.swing.JComponent#getToolTipLocation(java.awt.event.MouseEvent) */ + @Override public Point getToolTipLocation(MouseEvent event) { int x = event.getX(), w = getWidth(); @@ -904,7 +908,7 @@ public class SeqPanel extends JPanel implements MouseListener, * Convert position in sequence (base 1) to sequence character array * index (base 0) */ - int start = m.getStart() - 1; + int start = m.getStart() - m.getSequence().getStart(); setStatusMessage(seq, start, sequenceIndex); return; } @@ -1034,7 +1038,8 @@ public class SeqPanel extends JPanel implements MouseListener, message.append("Edit group:"); if (editCommand == null) { - editCommand = new EditCommand(MessageManager.getString("action.edit_group")); + editCommand = new EditCommand( + MessageManager.getString("action.edit_group")); } } else @@ -1047,7 +1052,8 @@ public class SeqPanel extends JPanel implements MouseListener, } if (editCommand == null) { - editCommand = new EditCommand(MessageManager.formatMessage("label.edit_params", new String[]{label})); + editCommand = new EditCommand(MessageManager.formatMessage( + "label.edit_params", new String[] { label })); } } @@ -1168,8 +1174,7 @@ public class SeqPanel extends JPanel implements MouseListener, { for (int j = 0; j < startres - lastres; j++) { - if (!jalview.util.Comparison.isGap(groupSeqs[g] - .getCharAt(fixedRight - j))) + if (!Comparison.isGap(groupSeqs[g].getCharAt(fixedRight - j))) { blank = false; break; @@ -1231,7 +1236,7 @@ public class SeqPanel extends JPanel implements MouseListener, continue; } - if (!jalview.util.Comparison.isGap(groupSeqs[g].getCharAt(j))) + if (!Comparison.isGap(groupSeqs[g].getCharAt(j))) { // Not a gap, block edit not valid endEditing(); @@ -1285,14 +1290,13 @@ public class SeqPanel extends JPanel implements MouseListener, { for (int j = lastres; j < startres; j++) { - insertChar(j, new SequenceI[] - { seq }, fixedRight); + insertChar(j, new SequenceI[] { seq }, fixedRight); } } else { - appendEdit(Action.INSERT_GAP, new SequenceI[] - { seq }, lastres, startres - lastres); + appendEdit(Action.INSERT_GAP, new SequenceI[] { seq }, lastres, + startres - lastres); } } else @@ -1309,8 +1313,7 @@ public class SeqPanel extends JPanel implements MouseListener, endEditing(); break; } - deleteChar(startres, new SequenceI[] - { seq }, fixedRight); + deleteChar(startres, new SequenceI[] { seq }, fixedRight); } } else @@ -1328,8 +1331,8 @@ public class SeqPanel extends JPanel implements MouseListener, if (max > 0) { - appendEdit(Action.DELETE_GAP, new SequenceI[] - { seq }, startres, max); + appendEdit(Action.DELETE_GAP, new SequenceI[] { seq }, + startres, max); } } } @@ -1339,14 +1342,13 @@ public class SeqPanel extends JPanel implements MouseListener, { for (int j = lastres; j < startres; j++) { - insertChar(j, new SequenceI[] - { seq }, fixedRight); + insertChar(j, new SequenceI[] { seq }, fixedRight); } } else { - appendEdit(Action.INSERT_NUC, new SequenceI[] - { seq }, lastres, startres - lastres); + appendEdit(Action.INSERT_NUC, new SequenceI[] { seq }, lastres, + startres - lastres); } } } @@ -1366,7 +1368,7 @@ public class SeqPanel extends JPanel implements MouseListener, for (blankColumn = fixedColumn; blankColumn > j; blankColumn--) { - if (jalview.util.Comparison.isGap(seq[s].getCharAt(blankColumn))) + if (Comparison.isGap(seq[s].getCharAt(blankColumn))) { // Theres a space, so break and insert the gap break; @@ -1402,8 +1404,7 @@ public class SeqPanel extends JPanel implements MouseListener, final Edit edit = new EditCommand().new Edit(action, seq, pos, count, av.getAlignment().getGapCharacter()); - editCommand.appendEdit(edit, av.getAlignment(), - true, null); + editCommand.appendEdit(edit, av.getAlignment(), true, null); } void deleteChar(int j, SequenceI[] seq, int fixedColumn) @@ -1469,21 +1470,23 @@ public class SeqPanel extends JPanel implements MouseListener, av.setSelectionGroup(null); } - List features = seqCanvas.getFeatureRenderer().findFeaturesAtRes( - sequence.getDatasetSequence(), - sequence.findPosition(findRes(evt))); + List features = seqCanvas.getFeatureRenderer() + .findFeaturesAtRes(sequence.getDatasetSequence(), + sequence.findPosition(findRes(evt))); - if (features != null && features.size()> 0) + if (features != null && features.size() > 0) { SearchResults highlight = new SearchResults(); - highlight.addResult(sequence, features.get(0).getBegin(), - features.get(0).getEnd()); + highlight.addResult(sequence, features.get(0).getBegin(), features + .get(0).getEnd()); seqCanvas.highlightSearchResults(highlight); } - if (features != null && features.size()> 0) + if (features != null && features.size() > 0) { - seqCanvas.getFeatureRenderer().amendFeatures(new SequenceI[] - { sequence }, features.toArray(new SequenceFeature[features.size()]), false, ap); + seqCanvas.getFeatureRenderer().amendFeatures( + new SequenceI[] { sequence }, + features.toArray(new SequenceFeature[features.size()]), + false, ap); seqCanvas.highlightSearchResults(null); } @@ -1596,23 +1599,24 @@ public class SeqPanel extends JPanel implements MouseListener, } - if (javax.swing.SwingUtilities.isRightMouseButton(evt)) + if (evt.isPopupTrigger()) { - List allFeatures = ap.getFeatureRenderer().findFeaturesAtRes( - sequence.getDatasetSequence(), sequence.findPosition(res)); - Vector links = new Vector(); - for (SequenceFeature sf:allFeatures) + List allFeatures = ap.getFeatureRenderer() + .findFeaturesAtRes(sequence.getDatasetSequence(), + sequence.findPosition(res)); + List links = new ArrayList(); + for (SequenceFeature sf : allFeatures) { if (sf.links != null) { - for (int j = 0; j < sf.links.size(); j++) + for (String link : sf.links) { - links.addElement(sf.links.elementAt(j)); + links.add(link); } } } - jalview.gui.PopupMenu pop = new jalview.gui.PopupMenu(ap, null, links); + PopupMenu pop = new PopupMenu(ap, null, links); pop.show(this, evt.getX(), evt.getY()); return; } @@ -1917,8 +1921,7 @@ public class SeqPanel extends JPanel implements MouseListener, // handles selection messages... // TODO: extend config options to allow user to control if selections may be // shared between viewports. - boolean iSentTheSelection = (av == source - || (source instanceof AlignViewport && ((AlignmentViewport) source) + boolean iSentTheSelection = (av == source || (source instanceof AlignViewport && ((AlignmentViewport) source) .getSequenceSetId().equals(av.getSequenceSetId()))); if (iSentTheSelection || !av.followSelection) { @@ -1944,28 +1947,28 @@ public class SeqPanel extends JPanel implements MouseListener, // do we want to thread this ? (contention with seqsel and colsel locks, I // suspect) - // rules are: colsel is copied if there is a real intersection between - // sequence selection + /* + * only copy colsel if there is a real intersection between + * sequence selection and this panel's alignment + */ boolean repaint = false; - boolean copycolsel = true; + boolean copycolsel = false; SequenceGroup sgroup = null; if (seqsel != null && seqsel.getSize() > 0) { if (av.getAlignment() == null) { - jalview.bin.Cache.log.warn("alignviewport av SeqSetId=" + Cache.log.warn("alignviewport av SeqSetId=" + av.getSequenceSetId() + " ViewId=" + av.getViewId() + " 's alignment is NULL! returning immediately."); return; } sgroup = seqsel.intersect(av.getAlignment(), (av.hasHiddenRows()) ? av.getHiddenRepSequences() : null); - if ((sgroup == null || sgroup.getSize() == 0) - || (colsel == null || colsel.size() == 0)) + if ((sgroup != null && sgroup.getSize() > 0)) { - // don't copy columns if the region didn't intersect. - copycolsel = false; + copycolsel = true; } } if (sgroup != null && sgroup.getSize() > 0) @@ -1983,7 +1986,7 @@ public class SeqPanel extends JPanel implements MouseListener, { // the current selection is unset or from a previous message // so import the new colsel. - if (colsel == null || colsel.size() == 0) + if (colsel == null || colsel.isEmpty()) { if (av.getColumnSelection() != null) { @@ -2034,11 +2037,13 @@ public class SeqPanel extends JPanel implements MouseListener, protected boolean selectionFromTranslation(SequenceGroup seqsel, ColumnSelection colsel, SelectionSource source) { - if (!(source instanceof AlignViewportI)) { + if (!(source instanceof AlignViewportI)) + { return false; } final AlignViewportI sourceAv = (AlignViewportI) source; - if (sourceAv.getCodingComplement() != av && av.getCodingComplement() != sourceAv) + if (sourceAv.getCodingComplement() != av + && av.getCodingComplement() != sourceAv) { return false; } @@ -2056,7 +2061,6 @@ public class SeqPanel extends JPanel implements MouseListener, ColumnSelection cs = MappingUtils.mapColumnSelection(colsel, sourceAv, av); av.setColumnSelection(cs); - av.isColSelChanged(true); PaintRefresher.Refresh(this, av.getSequenceSetId());