From 564042414eacdd22bbbb3f735e2d7c1fd2c6f3f8 Mon Sep 17 00:00:00 2001 From: Eduardo Omine Date: Wed, 22 Jul 2026 14:10:37 -0300 Subject: [PATCH 1/3] Remove jshint headers --- lib/js/affixed-line-breaks.mjs | 3 --- lib/js/char-groups-data.mjs | 2 -- .../components/actors/active-actors-rendering-controller.mjs | 3 --- lib/js/components/actors/actors-base.mjs | 4 ---- lib/js/components/actors/actors-meta.mjs | 3 --- lib/js/components/actors/available-actors.mjs | 4 ---- lib/js/components/actors/layer.mjs | 4 ---- lib/js/components/actors/line-of-text.mjs | 4 ---- lib/js/components/actors/models.mjs | 4 ---- lib/js/components/actors/properties-util.mjs | 4 ---- lib/js/components/actors/simple-svg.mjs | 4 ---- lib/js/components/actors/stage.mjs | 4 ---- lib/js/components/actors/videoproof-array.mjs | 4 ---- lib/js/components/animation-animanion.mjs | 3 --- lib/js/components/animation-fundamentals.mjs | 3 --- lib/js/components/animation-t-generator.mjs | 3 --- lib/js/components/basics.mjs | 3 --- lib/js/components/color.mjs | 4 ---- lib/js/components/dev-layouts/example-key-moments.mjs | 3 --- lib/js/components/dev-layouts/example-react.mjs | 4 ---- lib/js/components/dev-layouts/example.mjs | 3 --- lib/js/components/dev-layouts/pentrek.mjs | 4 ---- lib/js/components/dev-layouts/videoproof-array.mjs | 3 --- lib/js/components/font-loading.mjs | 3 --- lib/js/components/generic.mjs | 3 --- lib/js/components/layouts/motion-stage.mjs | 3 --- lib/js/components/layouts/videoproof.typeroof.jsx | 3 --- lib/js/components/main-model.mjs | 2 -- lib/js/components/main-ui.mjs | 3 --- lib/js/components/registered-properties.mjs | 4 ---- lib/js/components/task-automations/compare-line.mjs | 3 --- .../task-automations/container-task-automations.mjs | 4 ---- lib/js/components/task-automations/inchworm.mjs | 3 --- lib/js/components/task-automations/simple-clock.mjs | 4 ---- lib/js/components/ui-char-groups.mjs | 4 ---- lib/js/components/ui-manual-axis-locations.mjs | 3 --- lib/js/domTool.mjs | 2 -- lib/js/legacy/content-editable.mjs | 2 -- lib/js/legacy/layouts/vartools-grid.mjs | 1 - lib/js/legacy/layouts/vartools-typespec.mjs | 1 - lib/js/legacy/layouts/vartools-waterfall.mjs | 1 - lib/js/legacy/layouts/videoproof-array.mjs | 2 -- lib/js/legacy/layouts/videoproof-contextual.mjs | 2 -- lib/js/legacy/layouts/videoproof-type-your-own.mjs | 2 -- lib/js/legacy/main.mjs | 2 -- lib/js/legacy/text-selection.mjs | 2 -- lib/js/legacy/videoproof-controller.mjs | 1 - lib/js/local-font-storage.mjs | 4 ---- lib/js/main-shell.mjs | 1 - lib/js/model/font.mjs | 2 -- lib/js/shell.mjs | 2 -- lib/js/util.mjs | 2 -- 52 files changed, 151 deletions(-) diff --git a/lib/js/affixed-line-breaks.mjs b/lib/js/affixed-line-breaks.mjs index cfb266a60..678ea0a21 100644 --- a/lib/js/affixed-line-breaks.mjs +++ b/lib/js/affixed-line-breaks.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ - - /** * Setting the widest wdth/wght/opsz combination the font can handle. */ diff --git a/lib/js/char-groups-data.mjs b/lib/js/char-groups-data.mjs index 5ef1ba302..021ae2c54 100644 --- a/lib/js/char-groups-data.mjs +++ b/lib/js/char-groups-data.mjs @@ -1,5 +1,3 @@ -/* jshint esversion: 11, module:true, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ - import { deepFreeze } from './util.mjs'; diff --git a/lib/js/components/actors/active-actors-rendering-controller.mjs b/lib/js/components/actors/active-actors-rendering-controller.mjs index a94bdb5d1..fe7f3b4d4 100644 --- a/lib/js/components/actors/active-actors-rendering-controller.mjs +++ b/lib/js/components/actors/active-actors-rendering-controller.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement import { Path } from '../../metamodel.mjs'; diff --git a/lib/js/components/actors/actors-base.mjs b/lib/js/components/actors/actors-base.mjs index 18d658a33..316a845b6 100644 --- a/lib/js/components/actors/actors-base.mjs +++ b/lib/js/components/actors/actors-base.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { ForeignKey , CoherenceFunction diff --git a/lib/js/components/actors/actors-meta.mjs b/lib/js/components/actors/actors-meta.mjs index 33a751688..0ac678cb4 100644 --- a/lib/js/components/actors/actors-meta.mjs +++ b/lib/js/components/actors/actors-meta.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement import { Path } from '../../metamodel.mjs'; diff --git a/lib/js/components/actors/available-actors.mjs b/lib/js/components/actors/available-actors.mjs index 96d69558c..8b799cfc9 100644 --- a/lib/js/components/actors/available-actors.mjs +++ b/lib/js/components/actors/available-actors.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { _AbstractSimpleOrEmptyModel } from '../../metamodel.mjs'; diff --git a/lib/js/components/actors/layer.mjs b/lib/js/components/actors/layer.mjs index 9cc885a3f..c8030c86b 100644 --- a/lib/js/components/actors/layer.mjs +++ b/lib/js/components/actors/layer.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { _BaseActorModel , ActorsModel diff --git a/lib/js/components/actors/line-of-text.mjs b/lib/js/components/actors/line-of-text.mjs index 4c11ad2a5..acd0e916a 100644 --- a/lib/js/components/actors/line-of-text.mjs +++ b/lib/js/components/actors/line-of-text.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { _BaseActorModel , genericActorMixin diff --git a/lib/js/components/actors/models.mjs b/lib/js/components/actors/models.mjs index c9452658b..a3c1acedd 100644 --- a/lib/js/components/actors/models.mjs +++ b/lib/js/components/actors/models.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { ForeignKey // , unwrapPotentialWriteProxy diff --git a/lib/js/components/actors/properties-util.mjs b/lib/js/components/actors/properties-util.mjs index 7fe129f2b..ca9351395 100644 --- a/lib/js/components/actors/properties-util.mjs +++ b/lib/js/components/actors/properties-util.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { validateOpenTypeTagString } from '../../util.mjs'; diff --git a/lib/js/components/actors/simple-svg.mjs b/lib/js/components/actors/simple-svg.mjs index dfbbcf2df..e13f43ba3 100644 --- a/lib/js/components/actors/simple-svg.mjs +++ b/lib/js/components/actors/simple-svg.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { _BaseActorModel , genericActorMixin diff --git a/lib/js/components/actors/stage.mjs b/lib/js/components/actors/stage.mjs index 859a0b8bc..e8facbae3 100644 --- a/lib/js/components/actors/stage.mjs +++ b/lib/js/components/actors/stage.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { StaticNode } from '../generic.mjs'; diff --git a/lib/js/components/actors/videoproof-array.mjs b/lib/js/components/actors/videoproof-array.mjs index 4b0f07008..e244c4925 100644 --- a/lib/js/components/actors/videoproof-array.mjs +++ b/lib/js/components/actors/videoproof-array.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, module: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { zip } from '../../util.mjs'; diff --git a/lib/js/components/animation-animanion.mjs b/lib/js/components/animation-animanion.mjs index c3555209a..dbaf9df8d 100644 --- a/lib/js/components/animation-animanion.mjs +++ b/lib/js/components/animation-animanion.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import { zip , enumerate diff --git a/lib/js/components/animation-fundamentals.mjs b/lib/js/components/animation-fundamentals.mjs index b9f38621c..3133d514a 100644 --- a/lib/js/components/animation-fundamentals.mjs +++ b/lib/js/components/animation-fundamentals.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import { // Path //, getEntry diff --git a/lib/js/components/animation-t-generator.mjs b/lib/js/components/animation-t-generator.mjs index e1e378be3..5aeee0f5f 100644 --- a/lib/js/components/animation-t-generator.mjs +++ b/lib/js/components/animation-t-generator.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import { _BaseComponent } from './basics.mjs'; diff --git a/lib/js/components/basics.mjs b/lib/js/components/basics.mjs index c65f29969..e0910beab 100644 --- a/lib/js/components/basics.mjs +++ b/lib/js/components/basics.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import DOMTool from '../domTool.mjs'; import { diff --git a/lib/js/components/color.mjs b/lib/js/components/color.mjs index af21e6bfd..1441fe2d7 100644 --- a/lib/js/components/color.mjs +++ b/lib/js/components/color.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { ForeignKey , StringModel diff --git a/lib/js/components/dev-layouts/example-key-moments.mjs b/lib/js/components/dev-layouts/example-key-moments.mjs index dc37afaa1..b82599361 100644 --- a/lib/js/components/dev-layouts/example-key-moments.mjs +++ b/lib/js/components/dev-layouts/example-key-moments.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement import { Path , getEntry diff --git a/lib/js/components/dev-layouts/example-react.mjs b/lib/js/components/dev-layouts/example-react.mjs index 35442c54d..584e4da84 100644 --- a/lib/js/components/dev-layouts/example-react.mjs +++ b/lib/js/components/dev-layouts/example-react.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { _BaseContainerComponent , SimpleProtocolHandler diff --git a/lib/js/components/dev-layouts/example.mjs b/lib/js/components/dev-layouts/example.mjs index ba981c2d0..0540df421 100644 --- a/lib/js/components/dev-layouts/example.mjs +++ b/lib/js/components/dev-layouts/example.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import { _BaseComponent , _BaseContainerComponent diff --git a/lib/js/components/dev-layouts/pentrek.mjs b/lib/js/components/dev-layouts/pentrek.mjs index e34b3c58b..63eba0c45 100644 --- a/lib/js/components/dev-layouts/pentrek.mjs +++ b/lib/js/components/dev-layouts/pentrek.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - - import { default as ModuleBaseFn } from '../../vendor/fbdemo/docs/lerp.mjs'; diff --git a/lib/js/components/dev-layouts/videoproof-array.mjs b/lib/js/components/dev-layouts/videoproof-array.mjs index 943729a12..fcb8b912a 100644 --- a/lib/js/components/dev-layouts/videoproof-array.mjs +++ b/lib/js/components/dev-layouts/videoproof-array.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement import { Path , unwrapPotentialWriteProxy diff --git a/lib/js/components/font-loading.mjs b/lib/js/components/font-loading.mjs index 2249086a8..61d5d7c10 100644 --- a/lib/js/components/font-loading.mjs +++ b/lib/js/components/font-loading.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import { ForeignKey } from '../metamodel.mjs'; diff --git a/lib/js/components/generic.mjs b/lib/js/components/generic.mjs index 65009d2a6..4df89514c 100644 --- a/lib/js/components/generic.mjs +++ b/lib/js/components/generic.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import { identity } from '../util.mjs'; diff --git a/lib/js/components/layouts/motion-stage.mjs b/lib/js/components/layouts/motion-stage.mjs index d3acd3616..32d7b1316 100644 --- a/lib/js/components/layouts/motion-stage.mjs +++ b/lib/js/components/layouts/motion-stage.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement import { Path , getEntry diff --git a/lib/js/components/layouts/videoproof.typeroof.jsx b/lib/js/components/layouts/videoproof.typeroof.jsx index 3e009d98a..ce387c59c 100644 --- a/lib/js/components/layouts/videoproof.typeroof.jsx +++ b/lib/js/components/layouts/videoproof.typeroof.jsx @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement import { Path, getEntry, diff --git a/lib/js/components/main-model.mjs b/lib/js/components/main-model.mjs index e369fbb64..b216361bb 100644 --- a/lib/js/components/main-model.mjs +++ b/lib/js/components/main-model.mjs @@ -1,5 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ - import { unwrapPotentialWriteProxy , _AbstractStructModel diff --git a/lib/js/components/main-ui.mjs b/lib/js/components/main-ui.mjs index 0778913ec..5c01d169e 100644 --- a/lib/js/components/main-ui.mjs +++ b/lib/js/components/main-ui.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import { Path } from '../metamodel.mjs'; diff --git a/lib/js/components/registered-properties.mjs b/lib/js/components/registered-properties.mjs index f1d4b4357..c719546b2 100644 --- a/lib/js/components/registered-properties.mjs +++ b/lib/js/components/registered-properties.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { FreezableMap } from '../metamodel.mjs'; diff --git a/lib/js/components/task-automations/compare-line.mjs b/lib/js/components/task-automations/compare-line.mjs index 7924ca69e..5b8700d60 100644 --- a/lib/js/components/task-automations/compare-line.mjs +++ b/lib/js/components/task-automations/compare-line.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement import { getEntry } from '../../metamodel.mjs'; diff --git a/lib/js/components/task-automations/container-task-automations.mjs b/lib/js/components/task-automations/container-task-automations.mjs index ae8c2dd30..3805dec39 100644 --- a/lib/js/components/task-automations/container-task-automations.mjs +++ b/lib/js/components/task-automations/container-task-automations.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { FreezableMap } from '../../metamodel.mjs'; diff --git a/lib/js/components/task-automations/inchworm.mjs b/lib/js/components/task-automations/inchworm.mjs index a894fa1b0..e67accd1b 100644 --- a/lib/js/components/task-automations/inchworm.mjs +++ b/lib/js/components/task-automations/inchworm.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement import { getEntry } from '../../metamodel.mjs'; diff --git a/lib/js/components/task-automations/simple-clock.mjs b/lib/js/components/task-automations/simple-clock.mjs index 60c129104..97af534a4 100644 --- a/lib/js/components/task-automations/simple-clock.mjs +++ b/lib/js/components/task-automations/simple-clock.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { enumerate } from '../../util.mjs'; diff --git a/lib/js/components/ui-char-groups.mjs b/lib/js/components/ui-char-groups.mjs index 5977334c5..7f4be7ef7 100644 --- a/lib/js/components/ui-char-groups.mjs +++ b/lib/js/components/ui-char-groups.mjs @@ -1,7 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true, elision: true*/ -/* jshint -W008 */ // leading dot in decimals... -/* jshint -W028 */ // labels on if statement - import { _AbstractSimpleOrEmptyModel } from '../metamodel.mjs'; diff --git a/lib/js/components/ui-manual-axis-locations.mjs b/lib/js/components/ui-manual-axis-locations.mjs index cdf302aff..8ab5585dd 100644 --- a/lib/js/components/ui-manual-axis-locations.mjs +++ b/lib/js/components/ui-manual-axis-locations.mjs @@ -1,6 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ -/* jshint -W008 */ // leading dot in decimals... - import { _BaseComponent, setupTooltip diff --git a/lib/js/domTool.mjs b/lib/js/domTool.mjs index 8aa753448..a85c23c63 100644 --- a/lib/js/domTool.mjs +++ b/lib/js/domTool.mjs @@ -1,5 +1,3 @@ -/* jshint browser: true, esversion: 7, laxcomma: true, laxbreak: true */ - // I took this from googlefonts/fontbakery-dashboard and made it into a es module. // `document` is not required/expected to be global anymore, it's injected diff --git a/lib/js/legacy/content-editable.mjs b/lib/js/legacy/content-editable.mjs index 8cb6b9214..8db8962c1 100644 --- a/lib/js/legacy/content-editable.mjs +++ b/lib/js/legacy/content-editable.mjs @@ -1,5 +1,3 @@ -/* jshint browser: true, esversion: 9, laxcomma: true, laxbreak: true, unused:true, undef:true */ - function _compareByStartContainerPosition(rangeA, rangeB) { const { DOCUMENT_POSITION_FOLLOWING, DOCUMENT_POSITION_CONTAINED_BY , DOCUMENT_POSITION_PRECEDING, DOCUMENT_POSITION_CONTAINS diff --git a/lib/js/legacy/layouts/vartools-grid.mjs b/lib/js/legacy/layouts/vartools-grid.mjs index 3c83a9f9c..46df1e455 100644 --- a/lib/js/legacy/layouts/vartools-grid.mjs +++ b/lib/js/legacy/layouts/vartools-grid.mjs @@ -1,4 +1,3 @@ -/* jshint browser: true, esversion: 9, laxcomma: true, laxbreak: true, unused:true, undef:true */ import { handleEditableLine } from '../content-editable.mjs'; function* gridDimensionGenerator({axisTag, stepping, from, to, steppingValue}) { diff --git a/lib/js/legacy/layouts/vartools-typespec.mjs b/lib/js/legacy/layouts/vartools-typespec.mjs index 748f2f406..725565eba 100644 --- a/lib/js/legacy/layouts/vartools-typespec.mjs +++ b/lib/js/legacy/layouts/vartools-typespec.mjs @@ -1,4 +1,3 @@ -/* jshint browser: true, esversion: 9, laxcomma: true, laxbreak: true, unused:true, undef:true */ import { handleEditableLine, handleEditableDiv } from '../content-editable.mjs'; // import diff_match_patch from '../diff_match_patch/diff_match_patch.mjs'; diff --git a/lib/js/legacy/layouts/vartools-waterfall.mjs b/lib/js/legacy/layouts/vartools-waterfall.mjs index b461cc8fe..40c6dbe3e 100644 --- a/lib/js/legacy/layouts/vartools-waterfall.mjs +++ b/lib/js/legacy/layouts/vartools-waterfall.mjs @@ -1,4 +1,3 @@ -/* jshint browser: true, esversion: 9, laxcomma: true, laxbreak: true, unused:true, undef:true */ import { handleEditableLine } from '../content-editable.mjs'; // Duplicates from typespec! diff --git a/lib/js/legacy/layouts/videoproof-array.mjs b/lib/js/legacy/layouts/videoproof-array.mjs index ff62ce277..03f58565e 100644 --- a/lib/js/legacy/layouts/videoproof-array.mjs +++ b/lib/js/legacy/layouts/videoproof-array.mjs @@ -1,5 +1,3 @@ -/* jshint browser: true, esversion: 8, laxcomma: true, laxbreak: true */ - /** * TODO: the controller should take care of these events: * $(document).on('videoproof:fontLoaded.grid', populateGrid); diff --git a/lib/js/legacy/layouts/videoproof-contextual.mjs b/lib/js/legacy/layouts/videoproof-contextual.mjs index 4ce047073..a2c5082f2 100644 --- a/lib/js/legacy/layouts/videoproof-contextual.mjs +++ b/lib/js/legacy/layouts/videoproof-contextual.mjs @@ -1,5 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true */ - import DOMTool from '../../domTool.mjs'; function _testCharType(extendedCharGroups, c, re) { diff --git a/lib/js/legacy/layouts/videoproof-type-your-own.mjs b/lib/js/legacy/layouts/videoproof-type-your-own.mjs index f710afced..d83313997 100644 --- a/lib/js/legacy/layouts/videoproof-type-your-own.mjs +++ b/lib/js/legacy/layouts/videoproof-type-your-own.mjs @@ -1,5 +1,3 @@ -/* jshint browser: true, esversion: 7, laxcomma: true, laxbreak: true */ - function fitToSpace(element) { // Original, when extended text is present, doesn't do // white-space: nowrap, I don't know the reason. diff --git a/lib/js/legacy/main.mjs b/lib/js/legacy/main.mjs index 41b1af132..f4a9f8289 100644 --- a/lib/js/legacy/main.mjs +++ b/lib/js/legacy/main.mjs @@ -1,5 +1,3 @@ -/* jshint browser: true, esversion: 8, laxcomma: true, laxbreak: true */ - import { VideoproofController } from './videoproof-controller.mjs' // Should not require to wait until load (all resources, images etc are loaded), diff --git a/lib/js/legacy/text-selection.mjs b/lib/js/legacy/text-selection.mjs index 697e9871e..0c5fd9d0a 100644 --- a/lib/js/legacy/text-selection.mjs +++ b/lib/js/legacy/text-selection.mjs @@ -1,5 +1,3 @@ -/* jshint esversion: 11, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ - // from varla-varfo justifiction.mjs, sloghtly modified // TODO: use https://developer.mozilla.org/en-US/docs/Web/API/TreeWalker // or https://developer.mozilla.org/en-US/docs/Web/API/NodeIterator diff --git a/lib/js/legacy/videoproof-controller.mjs b/lib/js/legacy/videoproof-controller.mjs index ad2335411..cc1e658fc 100644 --- a/lib/js/legacy/videoproof-controller.mjs +++ b/lib/js/legacy/videoproof-controller.mjs @@ -1,4 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ /*eslint no-prototype-builtins: "off", no-self-assign: "off"*/ import * as opentype from '../vendor/opentype.js/dist/opentype.mjs'; diff --git a/lib/js/local-font-storage.mjs b/lib/js/local-font-storage.mjs index 302981dbe..5e50ef2b8 100644 --- a/lib/js/local-font-storage.mjs +++ b/lib/js/local-font-storage.mjs @@ -1,7 +1,3 @@ -/* jshint browser: true, esversion: 9, laxcomma: true, laxbreak: true, unused:true, undef:true */ - - - function promiseWrap(eventTarget) { return new Promise((resolve, reject)=>{ eventTarget.onsuccess = resolve; diff --git a/lib/js/main-shell.mjs b/lib/js/main-shell.mjs index 55d65e4f8..ecef71d68 100644 --- a/lib/js/main-shell.mjs +++ b/lib/js/main-shell.mjs @@ -1,4 +1,3 @@ -/* jshint browser: true, esversion: 8, laxcomma: true, laxbreak: true */ import { ShellController } from './shell.mjs'; import { MainUIController, Layouts, LAYOUT_GROUPS } from './components/main-ui.mjs'; diff --git a/lib/js/model/font.mjs b/lib/js/model/font.mjs index c43ac1c2a..76d7dbceb 100644 --- a/lib/js/model/font.mjs +++ b/lib/js/model/font.mjs @@ -1,5 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ - import { deepFreeze } from '../util.mjs'; import { OTFeatureInfo } from '../ot-feature-info.mjs' diff --git a/lib/js/shell.mjs b/lib/js/shell.mjs index 4491ff6c3..b2c8acd02 100644 --- a/lib/js/shell.mjs +++ b/lib/js/shell.mjs @@ -1,5 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ - import * as opentype from './vendor/opentype.js/dist/opentype.mjs'; import {getDecompressSync} from './vendor/wawoff2/decompress.mjs'; const woff2decompress = await getDecompressSync(); diff --git a/lib/js/util.mjs b/lib/js/util.mjs index 03f36003c..bf2b06a67 100644 --- a/lib/js/util.mjs +++ b/lib/js/util.mjs @@ -1,5 +1,3 @@ -/* jshint esversion: 11, browser: true, unused:true, undef:true, laxcomma: true, laxbreak: true, devel: true */ - /** * Just like Pythons zip. * From 7089df7a6738bc263a7143f212729a2bb862122c Mon Sep 17 00:00:00 2001 From: Eduardo Omine Date: Wed, 22 Jul 2026 14:17:25 -0300 Subject: [PATCH 2/3] Remove inline directives --- ...n-analysis-self-describing-widget-setup.md | 4 -- lib/js/components/actors/actors-meta.mjs | 2 +- lib/js/components/actors/line-of-text.mjs | 2 - lib/js/components/actors/simple-svg.mjs | 4 -- lib/js/components/actors/videoproof-array.mjs | 6 --- lib/js/components/animation-animanion.mjs | 9 ++-- lib/js/components/animation-t-generator.mjs | 2 +- lib/js/components/axes-math.mjs | 6 --- lib/js/components/basics.mjs | 18 +------ .../dev-layouts/example-key-moments.mjs | 11 ++--- .../components/dev-layouts/example-react.mjs | 1 - lib/js/components/dev-layouts/example.mjs | 2 - lib/js/components/dev-layouts/pentrek.mjs | 2 - .../dev-layouts/videoproof-array.mjs | 8 +--- lib/js/components/font-loading.mjs | 14 +----- lib/js/components/generic.mjs | 27 +---------- lib/js/components/layouts/motion-stage.mjs | 18 +------ .../layouts/type-stage/meta.typeroof.jsx | 2 +- .../type-stage/style-patches.typeroof.jsx | 3 +- .../type-stage/tree-editor.typeroof.jsx | 3 +- .../type-spec-properties.typeroof.jsx | 3 +- .../layouts/type-stage/type-specnion.mjs | 2 +- lib/js/components/layouts/type-tools-grid.mjs | 2 - .../layouts/videoproof.typeroof.jsx | 3 +- lib/js/components/processed-properties.mjs | 2 - .../prosemirror/integration.typeroof.jsx | 3 +- .../task-automations/compare-line.mjs | 2 - .../components/task-automations/inchworm.mjs | 2 - .../task-automations/simple-clock.mjs | 2 - lib/js/components/type-driven-ui-basics.mjs | 4 +- lib/js/components/ui-char-groups.mjs | 2 - lib/js/components/ui-color-chooser.mjs | 4 -- .../components/ui-manual-axis-locations.mjs | 4 +- lib/js/domTool.mjs | 2 +- lib/js/legacy/layouts/vartools-typespec.mjs | 2 +- lib/js/legacy/text-selection.mjs | 2 +- lib/js/legacy/videoproof-controller.mjs | 48 +++++++------------ lib/js/metamodel/base-model.ts | 2 +- lib/js/metamodel/compare.ts | 2 +- lib/js/metamodel/dynamic-struct-model.ts | 2 - lib/js/metamodel/enum-model.ts | 2 - lib/js/metamodel/foreign-key.ts | 7 --- lib/js/metamodel/generic-model.ts | 2 - lib/js/metamodel/list-model.ts | 3 -- lib/js/metamodel/number-model.ts | 2 - lib/js/metamodel/ordered-map-model.ts | 2 - lib/js/metamodel/path.ts | 2 - lib/js/metamodel/potential-write-proxy.ts | 2 - lib/js/metamodel/struct-model.ts | 3 -- lib/js/shell.mjs | 1 - lib/js/util.mjs | 2 +- 51 files changed, 52 insertions(+), 215 deletions(-) diff --git a/docs/planning/design-analysis-self-describing-widget-setup.md b/docs/planning/design-analysis-self-describing-widget-setup.md index 1b2aa0077..0fb210f3e 100644 --- a/docs/planning/design-analysis-self-describing-widget-setup.md +++ b/docs/planning/design-analysis-self-describing-widget-setup.md @@ -99,7 +99,6 @@ resolves those needs at construction time. ```javascript export class UIToggleButton extends _BaseComponent { - // jshint ignore:start static REQUIREMENTS = [ require('settings:internalPropertyName', 'boolean') , require('classToken') @@ -107,7 +106,6 @@ export class UIToggleButton extends _BaseComponent { , require('label', val=>`turn ${val} on`) , require('label', val=>`Toggle ${val}`) ]; - // jshint ignore:end constructor(widgetBus, classToken, labelIsOn, labelIsOff, title) { ... } } @@ -115,7 +113,6 @@ export class UIToggleButton extends _BaseComponent { ```javascript export class UIColorChooser extends _BaseContainerComponent { - // jshint ignore:start static REQUIREMENTS = [ require('settings:rootPath') , require('zones') @@ -124,7 +121,6 @@ export class UIColorChooser extends _BaseContainerComponent { , require('updateDefaultsDependencies') , require('raw:requireUpdateDefaults') ]; - // jshint ignore:end constructor(widgetBus, zones, label, getDefault, updateDefaultsDependencies, requireUpdateDefaults) { ... } diff --git a/lib/js/components/actors/actors-meta.mjs b/lib/js/components/actors/actors-meta.mjs index 0ac678cb4..d991ac885 100644 --- a/lib/js/components/actors/actors-meta.mjs +++ b/lib/js/components/actors/actors-meta.mjs @@ -17,7 +17,7 @@ import { } from '../animation-animanion.mjs'; class ActorsMeta extends _BaseDynamicCollectionContainerComponent { - [HANDLE_CHANGED_AS_NEW] = true; // jshint ignore:line + [HANDLE_CHANGED_AS_NEW] = true; constructor(widgetBus, zones, initAnimanionFn, isInheritingPropertyFn , widgets=[]) { super(widgetBus, zones, widgets); diff --git a/lib/js/components/actors/line-of-text.mjs b/lib/js/components/actors/line-of-text.mjs index acd0e916a..d8a3e941a 100644 --- a/lib/js/components/actors/line-of-text.mjs +++ b/lib/js/components/actors/line-of-text.mjs @@ -45,9 +45,7 @@ export const LineOfTextActorModel = _BaseActorModel.createClass( ); export class LineOfTextActorRenderer extends _BaseComponent { - // jshint ignore:start static TEMPLATE = `(content not initialized)`; - // jshint ignore:end constructor(widgetBus) { super(widgetBus); [this.element] = this.initTemplate(); diff --git a/lib/js/components/actors/simple-svg.mjs b/lib/js/components/actors/simple-svg.mjs index e13f43ba3..a34c3f366 100644 --- a/lib/js/components/actors/simple-svg.mjs +++ b/lib/js/components/actors/simple-svg.mjs @@ -56,13 +56,11 @@ export const CircleActorModel = _BaseActorModel.createClass( ; export class CircleActorRenderer extends _BaseComponent { - // jshint ignore:start static TEMPLATE = ` `; - // jshint ignore:end constructor(widgetBus) { super(widgetBus); [this.element, this.circle] = this.initTemplate(); @@ -125,13 +123,11 @@ export class CircleActorRenderer extends _BaseComponent { } export class RectangleActorRenderer extends _BaseComponent { - // jshint ignore:start static TEMPLATE = ` `; - // jshint ignore:end constructor(widgetBus) { super(widgetBus); [this.element, this.shape] = this.initTemplate(); diff --git a/lib/js/components/actors/videoproof-array.mjs b/lib/js/components/actors/videoproof-array.mjs index e244c4925..95c63e513 100644 --- a/lib/js/components/actors/videoproof-array.mjs +++ b/lib/js/components/actors/videoproof-array.mjs @@ -211,9 +211,7 @@ function _getCellContents(charGroupsData, fonts, propertyValuesMap, previousStat } export class VideoproofArrayActorRenderer extends _BaseComponent { - // jshint ignore:start static TEMPLATE = `
(content not initialized)
`; - // jshint ignore:end constructor(widgetBus, charGroupsData) { super(widgetBus); // This is attached to the ModelType, so we don't have to hard @@ -407,9 +405,7 @@ export const VideoproofArrayV2ActorModel = _BaseActorModel.createClass( ); export class VideoproofArrayV2CellActorRenderer extends _BaseComponent { - // jshint ignore:start static TEMPLATE = `
[?]
`; - // jshint ignore:end constructor(widgetBus) { super(widgetBus); [this.element] = this.initTemplate(); @@ -625,9 +621,7 @@ export class Pointer32StackAlloc { * also only defines those methods. */ export class VideoproofArrayV2ActorRenderer extends _BaseContainerComponent { - // jshint ignore:start static TEMPLATE = `
`; - // jshint ignore:end constructor(widgetBus, _zones, charGroupsData) { widgetBus.wrapper.setProtocolHandlerImplementation( ...SimpleProtocolHandler.create('cellContent@', {treatAdressAsRootPath: false})); diff --git a/lib/js/components/animation-animanion.mjs b/lib/js/components/animation-animanion.mjs index dbaf9df8d..39d01c052 100644 --- a/lib/js/components/animation-animanion.mjs +++ b/lib/js/components/animation-animanion.mjs @@ -464,7 +464,7 @@ const _NOTDEF = Symbol('_NOTDEF'); * innerAnimanion = new Animanion(outerAnimanion, axisLocationsGen, keyMoments, isLoop); */ export class LocalScopeAnimanion { - static _NOTDEF = _NOTDEF // jshint ignore:line + static _NOTDEF = _NOTDEF constructor(propertiesGenerators, keyMoments, isLoop) { this._propertiesGenerators = propertiesGenerators; this.keyMoments = keyMoments; @@ -710,13 +710,13 @@ export class LocalScopeAnimanion { } export class HierarchicalScopeAnimanion { - static _NOTDEF = Symbol('_NOTDEF'); // jshint ignore:line + static _NOTDEF = Symbol('_NOTDEF'); /** * Can be used as an argument to getPropertiesFromGlobalT, * getLocalTs, and getLocalT */ - static LAST_T = Symbol('LAST_T'); // jshint ignore:line + static LAST_T = Symbol('LAST_T'); get LAST_T() { return this.constructor.LAST_T; } @@ -900,7 +900,6 @@ export class AnimationLiveProperties extends _BaseComponent { * (outerAnimanion, propertiesGenerator, keyMoments, isLoop); */ _initAnimanion(keyMoments, isLoop/*, parentProperties=null*/) { - // jshint unused:vars throw new Error(`NOT IMPLEMENTED _initAnimanion(${keyMoments}, ${isLoop})`); } @@ -1073,7 +1072,6 @@ export class AnimationLiveProperties extends _BaseComponent { * This is mainly for development/debugging. */ export class AnimationInfo extends _BaseComponent { - //jshint ignore:start static TEMPLATE = `

Animation Info

Basic Fields

@@ -1081,7 +1079,6 @@ export class AnimationInfo extends _BaseComponent {

Animated Properties

    `; - //jshint ignore:end constructor(widgetBus) { super(widgetBus); this._basicKeys = ['t', 'duration', 'isLoop', 'perpetual', 'playing']; diff --git a/lib/js/components/animation-t-generator.mjs b/lib/js/components/animation-t-generator.mjs index 5aeee0f5f..6065290ed 100644 --- a/lib/js/components/animation-t-generator.mjs +++ b/lib/js/components/animation-t-generator.mjs @@ -150,7 +150,7 @@ export class AnimationTGenerator extends _BaseComponent { } _setRunning(isRunning) { - if(this._running === !!isRunning) // jshint ignore:line + if(this._running === !!isRunning) return; // changed diff --git a/lib/js/components/axes-math.mjs b/lib/js/components/axes-math.mjs index 274cbb9c2..2a62508cc 100644 --- a/lib/js/components/axes-math.mjs +++ b/lib/js/components/axes-math.mjs @@ -505,7 +505,6 @@ export class UIAxesMathLocationValue extends _UIBaseList.UIItem { static TYPE_CLASS_PART = null; static ITEM_DATA_TRANSFER_TYPE_PATH = DATA_TRANSFER_TYPES.AXESMATH_LOCATION_VALUE_PATH; - // jshint ignore: start static TEMPLATE = `
    @@ -245,7 +244,6 @@ class PentrekExampleUI extends _BaseComponent { Speed
    `; - //jshint ignore:end constructor(widgetBus) { super(widgetBus); this._deferredUpdates = null; diff --git a/lib/js/components/dev-layouts/videoproof-array.mjs b/lib/js/components/dev-layouts/videoproof-array.mjs index fcb8b912a..c79e5df9f 100644 --- a/lib/js/components/dev-layouts/videoproof-array.mjs +++ b/lib/js/components/dev-layouts/videoproof-array.mjs @@ -89,7 +89,7 @@ function* calculateKeyframes(orderedFilteredAxisRanges) { let mdmOrder = axis === 'opsz' ? ['min', 'default', 'max'] : ['default', 'min', 'max'] - , mdm = mdmOrder.filter(k=>{ // jshint ignore:line + , mdm = mdmOrder.filter(k=>{ // This was loosely adopted from previous code // where I didn't understand the full reasoning // but for the present examples it produces the @@ -103,7 +103,7 @@ function* calculateKeyframes(orderedFilteredAxisRanges) { return true; return (axisRange[k] !== axisRange['default']); }) - .map(k=>axisRange[k]) // jshint ignore:line + .map(k=>axisRange[k]) ; axesMDM.push(mdm); } @@ -221,7 +221,6 @@ function _getClosestPointFromKeyFrames(keyFramesCoordinates, searchLocation) { } class UIKeyMomentsLinkNavigation extends _BaseComponent { - // jshint ignore: start static TEMPLATE = `