Config to disable scrolling result sets

codefori · Jul 7, 2022 · b5a8c7e · b5a8c7e
1 parent cb17fb7
commit b5a8c7e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/package.json b/package.json
@@ -51,13 +51,18 @@
       "properties": {
         "vscode-db2i.validator": {
           "type": "boolean",
-          "description": "Enable/disable the SQL validator",
+          "description": "Enable the SQL validator",
           "default": false
         },
+        "vscode-db2i.scrollingResultSet": {
+          "type": "boolean",
+          "description": "Enable scrolling result sets for basic SELECT statements.",
+          "default": true
+        },
         "vscode-db2i.pageSize": {
           "type": "number",
           "description": "Page size for Schema browser",
-          "default": 100
+          "default": 500
         }
       }
     },

diff --git a/src/language/results/index.js b/src/language/results/index.js
@@ -3,6 +3,7 @@ const vscode = require(`vscode`);
 const csv = require(`csv/sync`);
 
 const getInstance = require(`../../base`);
+const Configuration = require("../../configuration");
 const html = require(`./html`);
 
 class ResultSetPanelProvider {
@@ -157,7 +158,8 @@ exports.initialise = (context) => {
                 });
 
               } else {
-                if (statement.type === `statement` && this.isBasicStatement(statement.content)) {
+                const scrollingEnabled = Configuration.get(`scrollingResultSet`);
+                if (scrollingEnabled && statement.type === `statement` && this.isBasicStatement(statement.content)) {
                 // If it's a basic statement, we can let it scroll!
                   resultSetProvider.setScrolling(statement.content);