From 6253934041436194fa73dd6af26d534247cea6de Mon Sep 17 00:00:00 2001 From: hansonr Date: Thu, 18 Apr 2019 12:59:40 -0500 Subject: [PATCH] // BH 2018.04.18 comment: addNotify() is not appropriate here. We // are not changing ancestors, and keyboard action listeners do // not need to be reset. addNotify() is a very expensive operation, // requiring a full re-layout of all parents and children. // Note in JComponent: // This method is called by the toolkit internally and should // not be called directly by programs. // I note that addNotify() is called in several areas of Jalview. --- src/jalview/gui/AlignmentPanel.java | 1 + 1 file changed, 1 insertion(+) diff --git a/src/jalview/gui/AlignmentPanel.java b/src/jalview/gui/AlignmentPanel.java index 20f8cd1..9cb690f 100644 --- a/src/jalview/gui/AlignmentPanel.java +++ b/src/jalview/gui/AlignmentPanel.java @@ -550,6 +550,7 @@ public class AlignmentPanel extends GAlignmentPanel implements // Note in JComponent: // This method is called by the toolkit internally and should // not be called directly by programs. + // I note that addNotify() is called in several areas of Jalview. int annotationHeight = getAnnotationPanel().adjustPanelHeight(); annotationHeight = getAnnotationPanel() -- 1.7.10.2