From 6fc91fc6988f0388011dd2cfcb929f7e33cd0631 Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Mon, 28 Jun 2021 16:19:38 +0200 Subject: [PATCH] node tasks: fixup overly-short comment lengths The minimum "max length" should be 80cc, and 100cc is also fine. Signed-off-by: Thomas Lamprecht --- src/node/Tasks.js | 20 +++++++------------- 1 file changed, 7 insertions(+), 13 deletions(-) diff --git a/src/node/Tasks.js b/src/node/Tasks.js index 049b527..1fa6a83 100644 --- a/src/node/Tasks.js +++ b/src/node/Tasks.js @@ -9,18 +9,13 @@ Ext.define('Proxmox.node.Tasks', { loadMask: true, sortableColumns: false, - // set extra filter components, - // must have a 'name' property for the parameter, - // and must trigger a 'change' event - // if the value is 'undefined', it will not be sent to the api + // set extra filter components, must have a 'name' property for the parameter, and must + // trigger a 'change' event if the value is 'undefined', it will not be sent to the api extraFilter: [], - // filters that should only be set once and is not changable - // example: - // { - // vmid: 100, - // } + // fixed filters which cannot be changed after instantiation, for example: + // { vmid: 100 } preFilter: {}, controller: { @@ -43,10 +38,9 @@ Ext.define('Proxmox.node.Tasks', { updateLayout: function() { let me = this; - // we want to update the scrollbar on every store load - // since the total count might be different - // the buffered grid plugin does this only on scrolling itself - // and even reduces the scrollheight again when scrolling up + // update the scrollbar on every store load since the total count might be different. + // the buffered grid plugin does this only on (user) scrolling itself and even reduces + // the scrollheight again when scrolling up me.getView().updateLayout(); },