Auto-adjust window location when taller than page

bbimber · bbimber · commit 9d9fe886eaa5 · 2024-10-08T21:00:15.000-07:00
diff --git a/SequenceAnalysis/resources/web/SequenceAnalysis/panel/AnalysisSectionPanel.js b/SequenceAnalysis/resources/web/SequenceAnalysis/panel/AnalysisSectionPanel.js
@@ -8,7 +8,6 @@
  */
 Ext4.define('SequenceAnalysis.panel.AnalysisSectionPanel', {
     extend: 'Ext.form.Panel',
-    // TODO: align to top of window
     alias: 'widget.sequenceanalysis-analysissectionpanel',
     singleTool: false,
     allowDuplicateSteps: false,
@@ -196,8 +195,7 @@ Ext4.define('SequenceAnalysis.panel.AnalysisSectionPanel', {
             });
         }
 
-        // TODO: alignment
-        var win = Ext4.create('Ext.window.Window', {
+        Ext4.create('Ext.window.Window', {
             modal: true,
             ownerPanel: this,
             bodyStyle: 'padding: 5px;',
@@ -221,8 +219,7 @@ Ext4.define('SequenceAnalysis.panel.AnalysisSectionPanel', {
                     }
                 }
             }
-        });
-        win.show(btn);
+        }).show(btn);
     },
 
     getDescriptionButton: function(cfg){

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,6 @@`
`8`	`8`	`*/`
`9`	`9`	`Ext4.define('SequenceAnalysis.panel.AnalysisSectionPanel', {`
`10`	`10`	`extend: 'Ext.form.Panel',`
`11`		`- // TODO: align to top of window`
`12`	`11`	`alias: 'widget.sequenceanalysis-analysissectionpanel',`
`13`	`12`	`singleTool: false,`
`14`	`13`	`allowDuplicateSteps: false,`
`@@ -196,8 +195,7 @@ Ext4.define('SequenceAnalysis.panel.AnalysisSectionPanel', {`
`196`	`195`	`});`
`197`	`196`	`}`
`198`	`197`
`199`		`- // TODO: alignment`
`200`		`- var win = Ext4.create('Ext.window.Window', {`
	`198`	`+ Ext4.create('Ext.window.Window', {`
`201`	`199`	`modal: true,`
`202`	`200`	`ownerPanel: this,`
`203`	`201`	`bodyStyle: 'padding: 5px;',`
`@@ -221,8 +219,7 @@ Ext4.define('SequenceAnalysis.panel.AnalysisSectionPanel', {`
`221`	`219`	`}`
`222`	`220`	`}`
`223`	`221`	`}`
`224`		`- });`
`225`		`- win.show(btn);`
	`222`	`+ }).show(btn);`
`226`	`223`	`},`
`227`	`224`
`228`	`225`	`getDescriptionButton: function(cfg){`