kireji 0.8.2 → 0.9.0

package/src/parts/abstract/mesh/routeID-model-to.js

@@ -1,4 +1,3 @@
-// TODO: Validate MODEL and handle out-of-range points.
 const triIndex = mesh.triThatContainsPoint(MODEL)
 const triData = mesh.triTable[triIndex]
 const row = triData.rows[Math.floor(MODEL.y) - triData.range.min]

package/src/parts/abstract/part/mathML.js

@@ -15,8 +15,6 @@ const expression = (() => {
  return (LABELS ? "<munder>" + (needsParentheses ? "" : "<mrow>") : "") + (needsParentheses ? "<mrow class=parenthetic><mo>(</mo>" : "") + terms.join("") + (needsParentheses ? `<mo>)</mo></mrow>` : "") + (LABELS ? (needsParentheses ? "" : "</mrow>") + `<munder><mo stretchy="true">&#x23df;</mo>${recurse(0, "none", false, false)}</munder></munder>` : "")
 })()
 
-// TODO: handle parenthesis!
-
 const showBracketBelow = DEPTH > 2
 
 return /* html */`${WRAP_IN_MATH_TAG ? "<math displaystyle=true><mrow>" : ""}${equationVariable}${expression}${equationValue}${WRAP_IN_MATH_TAG ? "</mrow></math>" : ""}`

package/src/parts/abstract/part/type.d.ts

@@ -64,6 +64,8 @@ declare interface IPart<TOwner, TSubpart>
  readonly cssVariableOfImage(IMAGE_NAME: string): string
  /** Collects every build function in the part's prototype chain and then calls them all on the part itself. */
  readonly startBuild(): void
+ /** Performs `part.modelToRouteID()` on MODEL and then performs `part.setRouteID()` on the resulting route ID. */
+ readonly setModel(MODEL: any)
  /** Sets the part's routeID, propagating it leafward and rootward and updating all views. If DELTA is true, then ROUTE_ID is added to the part's current route ID. */
  readonly setRouteID(ROUTE_ID: bigint, DELTA: boolean = false): void
  /** Recomputes and then updates the part's routeID in response to a change in the the given subpart's routeID.

package/src/parts/abstract/part-outliner/itemHTML-recursive.js

@@ -8,6 +8,4 @@ return partOutliner.getChildren(SUBJECT).map((childPart, i, childArray) => {
  const label = `<span class="label${childPart.isAbstract ? " abstract" : ""}">${partOutliner.getLabel(childPart)}</span>`
  const summary = `<summary ${partOutliner.pointAttr("point", partIndex)} data-index="${partIndex}"${partOutliner.isActive(childPart) ? " data-active" : ""}${IS_LAST_OF_TYPE && !hasSubparts ? ' id="lastOutlinerItem"' : ""}><outliner-spacer style="--depth:${DEPTH + +!hasSubparts}"></outliner-spacer>${handle}${symbol}${label}</summary>`
  return `<details${hasSubparts ? "" : ` class=empty`} ${isOpen ? "open" : ""}>${summary}${recurse(childPart, DEPTH + 1, IS_LAST_OF_TYPE && i === childArray.length - 1)}</details>`
-}).join("")
-
-/* TODO: the children of collapsed outliner items should not be rendered into HTML as it bloats the fetch size. Instead, they should be rendered during hydration. */
+}).join("")

package/src/parts/abstract/permutation/build.js

@@ -1,21 +1,3 @@
-
-/* TODO:
- * 1. Instantiate a concrete innstance of an abstract part per-instance. Offload all per-instance logic there.
- *    - Such a part will have optional subparts 'instance' and 'meta'
- *    - "instance" corresponds to each instance part's statically-sized data model
- *      - which also receives its specific subject assigment from the outside (not part of it's route ID)
- *    - "meta" (name pending) corresponds to a non-instance-specific part whose cardinality is dependant upon the number of instances.
- *      - get its implementation details by generalizing the active and preview tab data in the tab group. 
- *      - At build time, the parent permutation will need to be able to query this part to get its cardinality once for every instance up to the max count.
- *      - This part can support mixes, matches, etc. and might even be the ideal place to place a sub-permutation.
- * 2. Convert the tab group to use this abstract type.
- * Considerations;
- * - How do we inspect the states of existing tab instances in `kireji.app`?
- *   - Files will be easy: they are part of the abstract.
- *   - Perhaps the state space box depicts an array of instance space states instead of the single state? Perhaps that is all that is needed?
- *   - Then, how do we depict the states of subparts of the parent part? Do we simply have an array for the entire instance part's tree?
- *   - We might need "push", "pop", "insert", "splice" and/or similar array methods to make it easier to modify the instance set on the fly. */
-
 // Memoize the permutation data.
 const instanceOffsets = [0n]
 const instanceBitDepths = [0n, 0n]

package/src/parts/abstract/permutation/mathML-subpart.js

@@ -1,7 +1,5 @@
 const n = permutation.supersetSize
 
-// TODO: factor in other aspects when expanding to the collection type.
-
 return [
  "<mn>1</mn>",
  "<mo>+</mo>",

package/src/parts/abstract/permutation/view-hydrate.js

@@ -1,3 +1,2 @@
-// TODO: Replace with subparts and let their own view controls manage this.
 permutation.viewedPermutationRouteID = permutation.permutationRouteID
 permutation.viewedInstances = [...permutation.instances]

package/src/parts/abstract/permutation/view-update.js

@@ -1,6 +1,4 @@
-// TODO: Consider if this should be distribute view update instead of just view update.
 if (permutation.instances.length !== permutation.viewedInstances.length || permutation.viewedPermutationRouteID !== permutation.permutationRouteID) {
- // TODO: This is a dead-simple approach ... more performant approaches exist.
  const existingInstanceCount = permutation.viewedInstances.length
  const targetInstanceCount = permutation.instances.length
  const maxLength = Math.max(existingInstanceCount, targetInstanceCount)

package/src/parts/abstract/scroller/build.js

@@ -1,4 +1,3 @@
-// TODO: consider moving this check (and the cardinality type check) to a post-build "validation" step?
 if (!scroller.query)
  throw new ReferenceError(`Scrollers must define a css selector at \`scroller.query\` which can select the parent element of the scroller (from ${scroller.host}).`)
 

package/src/parts/abstract/type.d.ts

@@ -3,14 +3,21 @@ declare interface IAbstract
 
  // Subparts.
  readonly application: IApplication<IAbstract, null>
+ readonly bodyMode: IBodyMode<IAbstract>
+ readonly boolean: IBoolean<IAbstract>
  readonly clip: IClip<IAbstract, null>
+ readonly comingSoon: IComingSoonApplication<IAbstract>
  readonly error: IErrorApplication<IAbstract>
  readonly facet: IFacet<IAbstract, null>
+ readonly issue: ITrackedIssue<IAbstract>
  readonly match: IMatch<IAbstract, null>
+ readonly mesh: IMesh<IAbstract>
  readonly mix: IMix<IAbstract, null>
  readonly part: IPart<IAbstract, null>
+ readonly partMask: IPartMask<IAbstract, null>
+ readonly partOutliner: IPartOutliner<IAbstract>
+ readonly permutation: IPermutation<IAbstract, null, null, null>
  readonly scroller: IScroller<IAbstract, null>
- readonly section: IApplicationSection<IAbstract, null>
  readonly sections: IApplicationSections<IAbstract, null>
  readonly tld: ITopLevelDomain<IAbstract, null>
 }

package/src/parts/core/client/async-install.js

@@ -40,7 +40,7 @@ logScope(0, "Finalizing Hydration", log => {
   client: _.parts.core.client
  })
 
- // Handle navigation. TODO: fix bugs here.
+ // Handle forward/back button navigation.
  window.addEventListener("pageshow", pageTransitionEvent => {
   log("Setting Initial State")
   addressBar.useRoute()

package/src/parts/core/server/sync-install.js

@@ -23,7 +23,6 @@ const
   'Content-Type': 'text/html;charset=UTF-8',
   "Document-Policy": "force-load-at-top",
   "Vary": "Sec-CH-Prefers-Color-Scheme",
-  // TODO: Finish the security setup.
   // 'Content-Security-Policy': "default-src 'none'; style-src 'unsafe-inline'; image-src data:; script-src self 'unsafe-inline'",
   // 'Permissions-Policy': 'microphone=(), camera=(), web-share=(self), full-screen=(self)',
   'Cross-Origin-Embedder-Policy': 'require-corp',

package/src/parts/desktop/task-bar/tray/fullscreen/point.js

@@ -1,4 +1,3 @@
-// TODO: check for support and show the tray item only if the browser supports it. Prefer instructing user to enter the browser's native fullscreen instead of JS fullscreen (which doesn't have cross-origin support).
 pointer.handle({
  click() {
   if (!document.fullscreenElement) {

package/src/parts/desktop/task-bar/tray/share/point.js

@@ -1,5 +1,3 @@
-// TODO: Check if nav.share truly exists. If not, hide the tray button.
-
 pointer.handle({
  click() {
   nav.share({

package/src/{kireji.schema.json → schema.kireji.json}

@@ -1,6 +1,6 @@
 {
  "$schema": "http://json-schema.org/draft-07/schema#",
- "title": "KirejiConfig",
+ "title": "Kireji Framework Config",
  "description": "Schema for the kireji.json configuration file.",
  "type": "object",
  "properties": {

package/src/schema.part.json

@@ -0,0 +1,30 @@
+{
+ "$schema": "http://json-schema.org/draft-07/schema#",
+ "title": "Kireji Part Config",
+ "description": "Schema for part.json files defining component behavior and methods.",
+ "type": "object",
+ "properties": {
+  "abstract": {
+   "description": "If true, this part serves as a base and cannot be instantiated directly.",
+   "type": "boolean"
+  },
+  "extends": {
+   "description": "The relative or absolute host name of the part that this part inherits from. The default value is `part`.",
+   "type": "string"
+  },
+  "methods": {
+   "description": "A map of method names to their expected arguments.",
+   "type": "object",
+   "additionalProperties": {
+    "description": "An array of strings representing the parameter names for this method.",
+    "type": "array",
+    "items": {
+     "type": "string"
+    }
+   }
+  }
+ },
+ "additionalProperties": {
+  "description": "Custom property specific to this part implementation."
+ }
+}

package/src/static.css

@@ -301,7 +301,6 @@ body.dark warning- {
  font-size: revert;
 }
 
-/* TODO: Make title-bar and other desktop experience css dynamic (because it can be disabled in the config). */
 title-bar {
  display: flex;
  flex-flow: row nowrap;