Frame Editor - popup rename, replace scrollbar fixed

minjk-bl · minjk-bl · commit 354ccac485bd · 2021-11-03T15:12:43.000+09:00
diff --git a/css/common/frameEditor.css b/css/common/frameEditor.css
@@ -375,7 +375,7 @@
     padding: 10px;
 }
 .vp-popup-addpage .vp-popup-tab.replace {
-    height: calc(100% - 50px);
+    height: calc(100% - 60px);
 }
 .vp-popup-addtype {
     width: 153px;
@@ -384,6 +384,12 @@
 .vp-popup-delete {
     padding-bottom: 20px;
 }
+.vp-popup-rename-page {
+    max-height: 400px;
+}
+.vp-popup-replace-table {
+    max-height: 350px;
+}
 .vp-popup-replace-add {
     margin-top: 30px
 }
diff --git a/src/common/vpFrameEditor.js b/src/common/vpFrameEditor.js
@@ -638,6 +638,7 @@ define([
 
     FrameEditor.prototype.renderRenamePage = function() {
         var content = new sb.StringBuilder();
+        content.appendFormatLine('<div class="{0} {1}">', 'vp-popup-rename-page', 'vp-scrollbar');
         content.appendLine('<table>');
         content.appendLine('<colgroup><col width="100px"><col width="*"></colgroup>');
         this.state.selected.forEach((col, idx) => {
@@ -648,17 +649,20 @@ define([
             content.appendLine('</tr>');
         });
         content.appendLine('</table>');
+        content.appendLine('</div>');
         return content.toString();
     }
 
     FrameEditor.prototype.renderReplacePage = function() {
         var content = new sb.StringBuilder();
         content.appendFormatLine('<label><input type="checkbox" class="{0}"/><span>{1}</span></label>', 'vp-popup-use-regex', 'Use Regular Expression');
         content.appendLine('<br/><br/>');
-        content.appendFormatLine('<table class="{0}">', 'vp-popup-replace-table');
+        content.appendFormatLine('<div class="{0}">', 'vp-popup-replace-table');
+        content.appendLine('<table>');
         content.appendLine(this.renderReplaceInput(0));
         content.appendFormatLine('<tr><td colspan="3"><button class="{0} {1}">{2}</button></td></tr>', 'vp-button', 'vp-popup-replace-add', '+ Add Key');
         content.appendLine('</table>');
+        content.appendLine('</div>');
         return content.toString();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -375,7 +375,7 @@`
`375`	`375`	`padding: 10px;`
`376`	`376`	`}`
`377`	`377`	`.vp-popup-addpage .vp-popup-tab.replace {`
`378`		`- height: calc(100% - 50px);`
	`378`	`+ height: calc(100% - 60px);`
`379`	`379`	`}`
`380`	`380`	`.vp-popup-addtype {`
`381`	`381`	`width: 153px;`
`@@ -384,6 +384,12 @@`
`384`	`384`	`.vp-popup-delete {`
`385`	`385`	`padding-bottom: 20px;`
`386`	`386`	`}`
	`387`	`+.vp-popup-rename-page {`
	`388`	`+ max-height: 400px;`
	`389`	`+}`
	`390`	`+.vp-popup-replace-table {`
	`391`	`+ max-height: 350px;`
	`392`	`+}`
`387`	`393`	`.vp-popup-replace-add {`
`388`	`394`	`margin-top: 30px`
`389`	`395`	`}`