@datalabz_/dashboard-kit 6.0.3 → 6.0.4

package/README.md

@@ -57,8 +57,8 @@ URLs already pinned by live dashboards.
 
 A parent may post `{ type: 'DK_GET_FILTERS_INFO' }` to a dashboard's frame; the kit answers the parent with
 `{ type: 'DK_FILTERS_INFO', filters: [...] }`. It needs no token or period, so a parent can ask as soon as
-the frame has loaded. The list is the controls in the dashboard's header — what it can be opened
-with — one entry each:
+the frame has loaded. The list is the header controls the dashboard can be opened with, which is
+what the kit can write into a header control at its first boot, one entry each:
 
 ```js
 { id: 'playerUsername', label: 'Player username', type: 'text',  // type: the input's type, 'select' or 'textarea'
@@ -69,11 +69,18 @@ with — one entry each:
   options: [{ value: 'BRL', label: 'Real' }] }  // select only: the first 100 options
 ```
 
-The whole header is read, so a control the dashboard adds to it after `init` is listed too. Left
-out: a control with no `id`, a second control with an `id` already listed, a password field, and
-radio, file, hidden and button controls, which the kit cannot fill. A select's options are those
-it has when asked: one the dashboard fills by a query later has none yet, and `optionsTotal` says how
-many there were. The answer always goes to the dashboard's parent, whoever asked.
+The list follows the fill, so it holds only what a filter can actually be given:
+
+- Left out: a control with no `id`, a second control with an `id` already listed, and radio, file,
+  hidden, password and button controls, which the kit refuses to fill.
+- A select lists only options with a value, since an empty value is refused, and `optionsTotal` counts
+  those. A select with none is left out: it takes no value.
+- What exists when the dashboard boots is what counts. Once it has booted, the answer is the list taken
+  at that moment, so options a section loads later by query, and controls added later, are not in it, as
+  they were not there for the fill. Only `value` and `disabled` are read again. A filter whose options
+  come from a query must therefore use static `<option>`s or a plain input to be openable.
+
+The answer always goes to the dashboard's parent, whoever asked.
 
 ## Rules
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@datalabz_/dashboard-kit",
-  "version": "6.0.3",
+  "version": "6.0.4",
   "description": "DashboardKit runtime + styles, served from jsDelivr",
   "license": "UNLICENSED",
   "files": [

package/v4.js

@@ -20,7 +20,7 @@
   let _activeTabId = '';
   let _sections = [];   // { id, label, load, loaded, errors }
   let _title = '';
-  let _gitVer = '8ed9062'; // replaced at deploy: sed -i "s/8ed9062/$(git rev-parse --short HEAD)/g" v4.js
+  let _gitVer = 'ce29793'; // replaced at deploy: sed -i "s/ce29793/$(git rev-parse --short HEAD)/g" v4.js
   let _calls = [];   // System Info call log
   let _sectionErrors = {}; // sectionId → error count
   let _generation = 0;

package/v5.js

@@ -20,7 +20,7 @@
   let _activeTabId = '';
   let _sections = [];   // { id, label, load, loaded, errors }
   let _title = '';
-  let _gitVer = '8ed9062'; // replaced at deploy: sed -i "s/8ed9062/$(git rev-parse --short HEAD)/g" v4.js
+  let _gitVer = 'ce29793'; // replaced at deploy: sed -i "s/ce29793/$(git rev-parse --short HEAD)/g" v4.js
   let _calls = [];   // System Info call log
   let _sectionErrors = {}; // sectionId → error count
   let _generation = 0;

package/v6.js

@@ -20,7 +20,7 @@
   let _activeTabId = '';
   let _sections = [];   // { id, label, load, loaded, errors }
   let _title = '';
-  let _gitVer = '8ed9062'; // replaced at deploy: sed -i "s/8ed9062/$(git rev-parse --short HEAD)/g" v6.js
+  let _gitVer = 'ce29793'; // replaced at deploy: sed -i "s/ce29793/$(git rev-parse --short HEAD)/g" v6.js
   let _calls = [];   // System Info call log
   let _sectionErrors = {}; // sectionId → error count
   let _generation = 0;
@@ -932,12 +932,17 @@ details.dk-call summary::-webkit-details-marker {
   }
 
   // ── Filters a parent can ask for ──────────────────────────────────────────────
-  // DK_GET_FILTERS_INFO from the parent is answered with DK_FILTERS_INFO: the controls a dashboard keeps in
-  // its header, which are what it can be opened with. It needs no token or period, so a parent can
-  // ask as soon as the frame has loaded. Left out: controls the kit cannot fill (radio, file,
-  // hidden, buttons), a password field, a control with no id, and a second control with an id
-  // already listed. The whole header is read, so a control a dashboard adds to it after init is listed.
-  const FILTER_SKIP_TYPES = ['radio', 'file', 'hidden', 'button', 'submit', 'reset', 'image', 'password'];
+  // DK_GET_FILTERS_INFO from the parent is answered with DK_FILTERS_INFO: the header controls a
+  // dashboard can be opened with, which is what the kit can write into a header control at its first boot. So
+  // the list follows that flow and nothing else: a control it cannot fill is not listed (radio,
+  // file, hidden, password, buttons; no id; a second control with an id already listed), a select
+  // lists only options a value may name, since an empty value is refused, and a select with none
+  // is left out. What exists when the dashboard boots is what counts: once it has booted, the list
+  // is the one taken at that moment, so options a section loads later by query are not in it, as
+  // they were not there for the fill. Only value and disabled are read again. It needs no token or
+  // period, so a parent can ask as soon as the frame has loaded.
+  const UNFILLABLE_TYPES = ['radio', 'file', 'hidden', 'password', 'button', 'submit', 'reset', 'image'];
+  let _bootFilters = null;            // the list as it was when the dashboard booted
   const FILTER_MAX_OPTIONS = 100;
   const FILTER_MAX_TEXT = 500;
 
@@ -957,15 +962,24 @@ details.dk-call summary::-webkit-details-marker {
     return _clip(el.getAttribute('aria-label') || el.getAttribute('placeholder') || '', 200);
   }
 
+  function _controlValue(el) {
+    if (el.tagName === 'SELECT') {
+      return el.multiple ? Array.from(el.selectedOptions).map(function (o) { return o.value; }) : el.value;
+    }
+    return el.type === 'checkbox' ? el.checked : _clip(el.value, FILTER_MAX_TEXT);
+  }
+
   function _dashboardFilters() {
     const header = document.querySelector('[data-dk-header]');
     const out = [];
     if (!header) return out;
     const seen = {};
     header.querySelectorAll('input, select, textarea').forEach(function (el) {
-      if (!el.id || seen[el.id] || FILTER_SKIP_TYPES.indexOf(el.type) >= 0) return;
-      seen[el.id] = true;
+      if (!el.id || seen[el.id] || UNFILLABLE_TYPES.indexOf(el.type) >= 0) return;
       const isSelect = el.tagName === 'SELECT';
+      const named = isSelect ? Array.from(el.options).filter(function (o) { return o.value !== ''; }) : null;
+      if (isSelect && !named.length) return;
+      seen[el.id] = true;
       const f = {
         id: el.id,
         label: _filterLabel(el),
@@ -980,24 +994,35 @@ details.dk-call summary::-webkit-details-marker {
       }
       if (isSelect) {
         f.multiple = el.multiple;
-        f.optionsTotal = el.options.length;
-        f.options = Array.from(el.options).slice(0, FILTER_MAX_OPTIONS).map(function (o) {
+        f.optionsTotal = named.length;
+        f.options = named.slice(0, FILTER_MAX_OPTIONS).map(function (o) {
           return { value: _clip(o.value, 200), label: _clip(o.textContent.trim(), 200) };
         });
-        f.value = el.multiple ? Array.from(el.selectedOptions).map(function (o) { return o.value; }) : el.value;
-      } else {
-        f.value = el.type === 'checkbox' ? el.checked : _clip(el.value, FILTER_MAX_TEXT);
       }
+      f.value = _controlValue(el);
       out.push(f);
     });
     return out;
   }
 
+  // What DK_FILTERS_INFO carries: the booted list once there is one, with the values as they are now.
+  function _filtersInfo() {
+    const list = _bootFilters || _dashboardFilters();
+    const header = document.querySelector('[data-dk-header]');
+    list.forEach(function (f) {
+      const el = header ? header.querySelector('#' + CSS.escape(f.id)) : null;
+      if (!el) return;
+      f.value = _controlValue(el);
+      if (el.disabled) f.disabled = true; else delete f.disabled;
+    });
+    return list;
+  }
+
 
   function _onMessage(e) {
     if (!e.data || typeof e.data !== 'object') return;
     if (e.data.type === 'DK_GET_FILTERS_INFO') {
-      window.parent.postMessage({ type: 'DK_FILTERS_INFO', filters: _dashboardFilters() }, '*');
+      window.parent.postMessage({ type: 'DK_FILTERS_INFO', filters: _filtersInfo() }, '*');
       return;
     }
     if (e.data.type === 'RK_SNAPSHOT') { _snapshot(); return; }
@@ -1027,6 +1052,7 @@ details.dk-call summary::-webkit-details-marker {
     }
 
     if (!_hasBooted && _authToken && _dateFromIso && _dateToIso && _timezone) {
+      _bootFilters = _dashboardFilters(); // before any section runs
       _hasBooted = true;
       const wait = document.getElementById('dk-wait');
       if (wait) wait.hidden = true;