visualpython#10 - scroll to applied block

minjk-bl · minjk-bl · commit 96b829004a79 · 2021-05-10T16:26:07.000+09:00
diff --git a/src/api_block/blockContainer.js b/src/api_block/blockContainer.js
@@ -261,9 +261,9 @@ define([
                 this.reRenderAllBlock_asc();
                 this.resetBlockListAndRenderThisBlock(block);
             }
-            return true;
+            return block;
         } else {
-            return false;
+            return null;
         }
     }
 
@@ -292,9 +292,9 @@ define([
                     this.resetOptionPage();
                     this.reRenderAllBlock_asc();
                 }
-                return true;
+                return block;
             } else {
-                return false;
+                return null;
             }
             
     }
diff --git a/src/api_block/init.js b/src/api_block/init.js
@@ -443,11 +443,15 @@ define([
 
         /** option page - apply 버튼 클릭 함수 바인딩 */
         $(document).on(STR_CLICK, VP_ID_PREFIX + VP_APIBLOCK_BOARD_OPTION_APPLY_BUTTON, function() {
-            var isApplied = blockContainer.applyBlock();
+            var appliedBlock = blockContainer.applyBlock();
 
-            if (isApplied) {
+            if (appliedBlock) {
                 // #11 applied! popup
                 vpCommon.renderSuccessMessage('Applied!');
+
+                // #10 scroll to selected/applied block
+                var appliedBlockDom = appliedBlock.getBlockMainDom();
+                $(VP_CLASS_PREFIX + VP_CLASS_APIBLOCK_BOARD).animate({scrollTop: $(appliedBlockDom).position().top}, "fast");
             }
         });
 

Original file line number	Diff line number	Diff line change
`@@ -261,9 +261,9 @@ define([`
`261`	`261`	`this.reRenderAllBlock_asc();`
`262`	`262`	`this.resetBlockListAndRenderThisBlock(block);`
`263`	`263`	`}`
`264`		`- return true;`
	`264`	`+ return block;`
`265`	`265`	`} else {`
`266`		`- return false;`
	`266`	`+ return null;`
`267`	`267`	`}`
`268`	`268`	`}`
`269`	`269`
`@@ -292,9 +292,9 @@ define([`
`292`	`292`	`this.resetOptionPage();`
`293`	`293`	`this.reRenderAllBlock_asc();`
`294`	`294`	`}`
`295`		`- return true;`
	`295`	`+ return block;`
`296`	`296`	`} else {`
`297`		`- return false;`
	`297`	`+ return null;`
`298`	`298`	`}`
`299`	`299`
`300`	`300`	`}`