@things-factory/figure-ui 10.1.208 → 10.1.211

package/client/modeller/data-interface.ts

@@ -4,7 +4,7 @@
  * of data is things-scene's; this only lists what there is to choose, drafts a binding the kernel then checks, and edits
  * the schema as a tree. Kept apart from the element so it is tested without a browser.
  */
-import { FIGURE_PART_ID, FIGURE_MOTION_SPEED_MAX, FigureContractError, figureDataRows, parseFigureAccessor } from '@hatiolab/figure-model'
+import { FIGURE_PART_ID, FIGURE_MOTION_SPEED_MAX, FigureContractError, figureAxesOf, figureDataRows, motionStateIdOf, parseFigureAccessor } from '@hatiolab/figure-model'
 import type { FigureAsset, FigureDataBehavior, FigureDataBinding, FigureDataField, FigureDataNode, FigureDataRow, FigureDataValue, FigureDataValueIssue, FigureProposalStep, FigureStatusPort } from '@hatiolab/figure-model'
 import { FigureInputs, type FigureDataResult } from '@hatiolab/things-scene'
 
@@ -286,12 +286,12 @@ export interface EndlessJoint {
   unit: 'deg' | 'rad'
 }
 
-/** The joints that turn without end: revolute, with no limits, leading no one's mimic. */
+/** The axes that turn without end: revolute, with no limits, following no one (ADR-0159: every axis of every joint). */
 export function endlessJointsOf(asset: FigureAsset): EndlessJoint[] {
   const out: EndlessJoint[] = []
-  for (const j of asset.joints ?? []) {
-    if (j.type !== 'revolute' || j.lowerLimit !== undefined || j.upperLimit !== undefined || j.mimic) continue
-    const input = j.state?.id ?? j.id
+  for (const { motion: a } of figureAxesOf(asset.joints)) {
+    if (a.type !== 'revolute' || a.lowerLimit !== undefined || a.upperLimit !== undefined || a.mimic) continue
+    const input = motionStateIdOf(a)
     const unit = asset.document.model.inputs.find(i => i.id === input)?.unit
     if (unit !== 'deg' && unit !== 'rad') continue
     if (!out.some(e => e.input === input)) out.push({ input, label: asset.stateInputs?.[input]?.label ?? input, unit })

package/client/modeller/editor.ts

@@ -6,7 +6,7 @@ import { openPopup } from '@operato/layout'
 import { OxPrompt } from '@operato/popup'
 import { customElement, property, state } from 'lit/decorators.js'
 import { live } from 'lit/directives/live.js'
-import { figurePartFaceOf, PART_LIMIT, parseFigureRecipe, resolveFigureRecipe, figurePartCountOf, figurePartIdFor, figureRecipeIdFor, figureSizeEditOf, figureSizeEditorOf, figureSectionMeasuresOf, figurePlaneAlignChanges, type FigurePlaneContact, figureRowOf, type FigureRowReading, figureFaceOverhangOf, type FigureFaceOverhang } from '@hatiolab/figure-model'
+import { figureFastenAxesOf, figurePartFaceOf, PART_LIMIT, parseFigureRecipe, resolveFigureRecipe, figurePartCountOf, figurePartIdFor, figureRecipeIdFor, figureSizeEditOf, figureSizeEditorOf, figureSectionMeasuresOf, figurePlaneAlignChanges, type FigurePlaneContact, figureRowOf, type FigureRowReading, figureFaceOverhangOf, type FigureFaceOverhang } from '@hatiolab/figure-model'
 import type { RecipeTemplate } from './recipes.js'
 import './figure-recipe-catalog.js'
 import type { FigureAsset, FigureAuthoringAction, FigureEvaluatedShape, FigureInputChange, FigurePartEdit, FigurePrimitiveCreation, FigureProposalStep, FigureRecipe, FigureRecipeValue, FigurePlacement, FigureFaceMappingNeed, FigureFaceRemap } from '@hatiolab/figure-model'
@@ -63,12 +63,13 @@ import { editRefusalOf, partNameRefusalOf, sizeRangeFitsOf, partRefusalOf, place
 import type { FigureRenderPart } from './renderer.js'
 import type { FigureBudgetLine } from './maturity.js'
 import { HelpToggles } from './help.js'
-import { endPlaceKeyOf, fieldWordsOf, followOf, sizeFollowersOf, measureNow, partSummaryOf, instanceRangeOf, instanceSizeToDeclare, proportionToFollow, rangeWith, turnsAboutAJoint, valueNow } from './links.js'
+import { sizeInputOf } from './size-response.js'
+import { canFollow, followableAxes, endPlaceKeyOf, fieldWordsOf, followOf, sizeFollowersOf, measureNow, partSummaryOf, instanceRangeOf, instanceSizeToDeclare, proportionToFollow, rangeWith, turnsAboutAJoint, valueNow } from './links.js'
 import { jointFanOf } from './joint-fan.js'
 import { derivedOf, type FigureRevision } from './revision.js'
 import { jointDragOf } from './joint-drag.js'
 import type { FirstSave } from './first-save.js'
-import { isLive, livePartsOf, liveOverSession, NOTHING_LIVE, scrubbed, scrubStep, type LiveValues } from './live.js'
+import { isLive, livePartsOf, liveOverSession, NOTHING_LIVE, scrubLabel, type LiveValues } from './live.js'
 import { startingShapeKindOfBody } from './starting-shape.js'
 import { groupMenu, GroupMenuStyles } from './group-menu.js'
 
@@ -360,10 +361,14 @@ export class FigureEditor extends LitElement {
   }
   /**
    * A refusal no particular sentence knows: figure-model's code said as a refusal, never its English; any other error
-   * (the network, the server) as it came.
+   * (the network, the server) as it came. A loop is said as a loop whichever way it was made: a command, or the size
+   * editor's steps (a measure made to follow what follows it, 2026-10-24 -- said as a bare refused step before).
    */
   private said(error: unknown): string {
-    return (error as { code?: string })?.code ? editRefusalOf(error, (key, params) => i18next.t(key, params)) : error instanceof Error ? error.message : String(error)
+    const code = (error as { code?: string })?.code
+    if (code === 'DIMENSION_CYCLE') return i18next.t('figure.text.follow-would-loop')
+    if (code === 'FASTEN_CYCLE' || code === 'ATTACH_CYCLE') return i18next.t('figure.text.fasten-would-loop')
+    return code ? editRefusalOf(error, (key, params) => i18next.t(key, params)) : error instanceof Error ? error.message : String(error)
   }
   private run(action: () => unknown) {
     try {
@@ -818,6 +823,10 @@ export class FigureEditor extends LitElement {
       .partId=${this.selected}
       ?locked=${unavailable}
       @command=${(e: CustomEvent) => this.linkCommand(e.detail.action)}
+      @size-recipe=${(e: CustomEvent) => this.applySizeSteps(e.detail.steps)}
+      @size-input=${(e: CustomEvent) => this.writeField({ id: e.detail.input, unit: 'mm' }, e.detail.value)}
+      @size-typing=${(e: CustomEvent) => this.hold(this.heldOf({ id: e.detail.input, unit: 'mm' }, e.detail.value))}
+      @size-let-go=${() => (this.live = NOTHING_LIVE)}
       @fasten-open=${() => this.openFastening()}
       .previewWords=${this.commandPreview.words()}
       .detachWords=${this.commandPreview.detachWords(shown)}
@@ -837,21 +846,25 @@ export class FigureEditor extends LitElement {
    */
   private renderPartGroups(shown: FigureRevision | undefined, geometry: readonly { id: string }[], fields: FieldLike[], unavailable: boolean, held?: HeldValues) {
     const groups = this.withSizeRow(groupFields(fields, this.selected), held)
-    const size = groups.vectors.filter(g => g.kind === 'vector' && g.name === 'size')
+    const integrated = !!shown && canFollow(shown.asset, this.selected)
+    const dimensionInputs = new Set(integrated ? followableAxes(shown!.asset, this.selected).map(axis => sizeInputOf(shown!.asset, this.selected, axis)) : [])
+    const singles = groups.singles.filter(g => g.kind !== 'single' || !dimensionInputs.has(g.field.id))
+    const size = integrated ? [] : groups.vectors.filter(g => g.kind === 'vector' && g.name === 'size')
     const place = groups.vectors.filter(g => g.kind === 'vector' && g.name !== 'size')
     const fastened = !!this.asset()?.joints?.some(j => j.body1.part === ownerOf(this.selected))
     const placeLocked = fastened && place.every(g => g.kind === 'vector' && (['x', 'y', 'z'] as const).every(axis => !g.cells[axis]))
     return this.renderLinks(
       unavailable,
       'shape',
-      html`${size.length || groups.singles.length
-          ? html`<div slot="size" group-fields>
+      html`${size.length || singles.length
+          ? html`<section slot="size" group-fields>
+              <h3>${i18next.t(integrated ? 'figure.label.group-shape' : 'figure.label.group-size')}</h3>
               ${size.map(g => this.renderVector(g, unavailable, held))}
-              ${groups.singles.length
-                ? html`<div form>${groups.singles.map(g => (g.kind === 'single' ? this.renderSingle(g.name, g.field, unavailable) : nothing))}</div>`
+              ${singles.length
+                ? html`<div form>${singles.map(g => (g.kind === 'single' ? this.renderSingle(g.name, g.field, unavailable) : nothing))}</div>`
                 : nothing}
-              ${this.renderSizeKeeps()}
-            </div>`
+              ${!integrated ? this.renderSizeKeeps() : nothing}
+            </section>`
           : nothing}
         ${place.length && !placeLocked
           ? html`<div slot="place" group-fields>${place.map(g => this.renderVector(g, unavailable, held))}${this.renderHeldAxes(place)}</div>`
@@ -860,7 +873,7 @@ export class FigureEditor extends LitElement {
   }
   /** The 「반응형」 tab of a part: the links' response groups, its row's own section put in by slot. */
   private renderResponse(shown: FigureRevision | undefined, geometry: readonly { id: string }[], unavailable: boolean) {
-    return this.renderLinks(unavailable, 'response', this.renderRepeat(shown, geometry, unavailable))
+    return this.renderLinks(unavailable, 'response', html`<div slot="size-note">${this.renderSizeKeeps()}</div>${this.renderRepeat(shown, geometry, unavailable)}`)
   }
   /**
    * 「더 보기」: what few parts need, folded by default and left as this person last left it (chief architect
@@ -967,6 +980,24 @@ export class FigureEditor extends LitElement {
     const done = this.linkCommand({ ...held.action, remap } as FigureAuthoringAction, held.from)
     if (done && held.from === 'section-editor') this.closeSectionEditor()
   }
+  /** The figure's size, declared for a part to follow, in one line (ADR ruling Q1). */
+  private sizeDeclaredWords() {
+    const size = this.designValues()
+    return i18next.t('figure.text.size-declared-to-follow', {
+      x: Math.round(size['size.x'] ?? 0).toLocaleString(),
+      y: Math.round(size['size.y'] ?? 0).toLocaleString(),
+      z: Math.round(size['size.z'] ?? 0).toLocaleString()
+    })
+  }
+  /**
+   * The size editor's steps, one step of the history. A measure made to follow the figure on a figure with no size
+   * declares it on the way; that is said in one line, as follow-size said it (ADR ruling Q1, 2026-10-24).
+   */
+  private applySizeSteps(steps: readonly FigureProposalStep[]) {
+    const hadSize = this.hasSize()
+    this.run(() => this.session!.applyRecipe(steps))
+    if (!hadSize && !this.error && this.hasSize()) this.notice = this.sizeDeclaredWords()
+  }
   /** Runs a command; true when it was taken. */
   private linkCommand(action: FigureAuthoringAction, from: 'section-editor' | 'panel' = 'panel'): boolean {
     /* Committed or refused, a command in hand is let go of: the figure as it stands is drawn again. */
@@ -978,14 +1009,7 @@ export class FigureEditor extends LitElement {
       this.error = ''
       this.faceMapping = undefined
       /* What the command did besides what was asked, in one line: the figure's size declared to follow (ADR ruling Q1). */
-      if (action.kind === 'follow-size' && !hadSize) {
-        const size = this.designValues()
-        this.notice = i18next.t('figure.text.size-declared-to-follow', {
-          x: Math.round(size['size.x'] ?? 0).toLocaleString(),
-          y: Math.round(size['size.y'] ?? 0).toLocaleString(),
-          z: Math.round(size['size.z'] ?? 0).toLocaleString()
-        })
-      }
+      if (action.kind === 'follow-size' && !hadSize) this.notice = this.sizeDeclaredWords()
       this.changed()
       return true
     } catch (error) {
@@ -998,19 +1022,20 @@ export class FigureEditor extends LitElement {
         this.error = ''
         return false
       }
-      this.error =
-        code === 'DIMENSION_CYCLE'
-          ? i18next.t('figure.text.follow-would-loop')
-          : code === 'FASTEN_CYCLE' || code === 'ATTACH_CYCLE'
-            ? i18next.t('figure.text.fasten-would-loop')
-            : this.said(error)
+      this.error = this.said(error)
       return false
     }
   }
   private changeField(field: { id: string; unit: FigureInputChange['unit']; role?: string }, event: Event) {
-    const input = event.target as HTMLInputElement,
-      value = input.valueAsNumber
-    /* A number typed keeps the part's middle, or its fastening -- figure-model's one size function (ADR-0140 4.1). */
+    const input = event.target as HTMLInputElement
+    // Rejected edits must also restore the displayed numeric value.
+    input.value = String(this.writeField(field, input.valueAsNumber))
+  }
+  /**
+   * A number written to an input, from a field, a scrub or the size editor's rows: a size keeps the part's middle, or its
+   * fastening -- figure-model's one size function (ADR-0140 4.1). Gives the value the input holds after, refused or not.
+   */
+  private writeField(field: { id: string; unit: FigureInputChange['unit']; role?: string }, value: number): number {
     const part = this.selected
     const change: FigureInputChange = { kind: field.role === 'state' ? 'set-state' : 'set-design', input: field.id, unit: field.unit, value }
     const asset = this.shown()?.asset
@@ -1019,8 +1044,7 @@ export class FigureEditor extends LitElement {
     /* Typed past the mounting plane: said, with the way to fasten it there (ADR-0140 5). */
     const after = asset && part && field.role !== 'state' ? throughPlaneOf(this.shown()!.asset, drawableFigureParts(this.session!.read().geometry ?? []), part) : 0
     if (after > before + 0.5) this.tellThroughPlane(part, after)
-    // Rejected edits must also restore the displayed numeric value.
-    input.value = String(this.session!.read().fields.find(i => i.id === field.id)!.value)
+    return this.session!.read().fields.find(i => i.id === field.id)!.value
   }
   /** Typing or a held spinner: each number on the way is drawn; the field's change records it. */
   private typing(field: { id: string; unit?: FigureInputChange['unit']; role?: string }, event: Event) {
@@ -1048,39 +1072,24 @@ export class FigureEditor extends LitElement {
    * pointer, and the release records one step. The field's number follows too, so the value in hand is always read.
    */
   private scrub(field: { id: string; unit: FigureInputChange['unit']; role?: string; value: number }, down: PointerEvent, unavailable: boolean) {
-    if (unavailable || down.button !== 0) return
+    if (unavailable) return
     const label = down.currentTarget as HTMLElement
     const input = (label.closest('div[cell]')?.querySelector('input') ?? label.nextElementSibling) as HTMLInputElement | null
     if (!input) return
-    down.preventDefault()
-    label.setPointerCapture(down.pointerId)
-    const start = field.value
-    let value = start
     /* Prepared once for the scrub: each move is a size through the same function its release writes with. */
     const asset = this.shown()?.asset
     const sized = asset && this.selected && field.role !== 'state' ? (() => { try { return figureSizeEditorOf(asset, this.selected) } catch { return undefined } })() : undefined
-    const move = (e: PointerEvent) => {
-      value = scrubbed(start, e.clientX - down.clientX, scrubStep(field.unit, start, e.shiftKey))
-      input.value = String(value)
-      this.hold(this.heldOf(field, value, sized))
-    }
-    const up = (e: PointerEvent) => {
-      label.removeEventListener('pointermove', move)
-      label.removeEventListener('pointerup', up)
-      label.removeEventListener('pointercancel', up)
-      /* A drag the system took away is not the author's value: the field goes back to where it was. */
-      if (e.type === 'pointercancel') value = start
-      if (value === start) {
-        input.value = String(start)
+    scrubLabel(down, field, {
+      move: value => {
+        input.value = String(value)
+        this.hold(this.heldOf(field, value, sized))
+      },
+      release: value => (input.value = String(this.writeField(field, value))),
+      cancel: () => {
+        input.value = String(field.value)
         this.live = NOTHING_LIVE
-        return
       }
-      input.value = String(value)
-      this.changeField(field, { target: input } as unknown as Event)
-    }
-    label.addEventListener('pointermove', move)
-    label.addEventListener('pointerup', up)
-    label.addEventListener('pointercancel', up)
+    })
   }
   /** The sizes a board may give the figure: the ends the author typed, and the rest as far as its parts allow. */
   private renderInstanceRange(shown: FigureRevision | undefined, unavailable: boolean) {
@@ -1243,45 +1252,45 @@ export class FigureEditor extends LitElement {
     const axis = model.nodes.find(n => n.id === `${this.selected}.repeat`)?.params?.axis ?? '?'
     const own = `${this.selected}.repeat.pitch`
     /* The pitch as it stands: the part's own number, a constant, or what a pitch that follows a length comes to. */
-    const pitch = asset.designInputs[own] ?? model.constants.find(c => c.id === layout?.args[4])?.value ?? valueNow(shown, layout?.args[4])
     const fits = layout?.op === 'fit-pitch@1'
-    /* What the pitch gives, in numbers: this many copies at the length it is authored, and how long it may be placed. */
-    const lengthRef = layout?.args[0] ?? ''
-    const length = asset.designInputs[lengthRef]
-    const longest = model.inputs.find(i => i.id === lengthRef)?.max
-    const limit = model.constants.find(c => c.id === layout?.args[6])?.value
+    const pitch = fits ? asset.designInputs[own] ?? model.constants.find(c => c.id === layout?.args[4])?.value ?? valueNow(shown, layout?.args[4]) : this.rowNow(shown)?.pitch
     const field = { id: own, unit: 'mm' as const, role: 'design', value: pitch ?? 0 }
-    const removable = !!asset.joints?.some(j => j.body1.part === this.selected && j.repeat)
+    const record = asset.joints?.find(j => j.body1.part === this.selected && j.repeat)
+    const parent = record && 'part' in record.body0 ? record.body0.part : undefined
+    const removable = !!record
+    const axes = record && 'part' in record.body0 && 'face' in record.body0 ? figureFastenAxesOf(asset, record.body0) : []
+    const update = (change: Partial<NonNullable<typeof record>['repeat']>) => {
+      if (!record?.repeat || unavailable) return
+      this.run(() => this.session!.command({ kind: 'set-joint', joint: { ...record, repeat: { ...record.repeat!, ...change } } }))
+    }
     return html`<section slot="row" row>
       <h3>
         ${i18next.t('figure.label.repeat')}
-        ${groupMenu([{ label: i18next.t('figure.button.remove-repeat'), icon: 'close', disabled: unavailable || !removable, run: () => this.run(() => this.session!.command({ kind: 'remove-repeat', part: this.selected })) }])}
+        <span section-state active>${i18next.t('figure.label.repeat-active')}</span>
       </h3>
-      <div inline-field><label>${i18next.t('figure.label.repeat-axis')}</label><span value>${axis.toUpperCase()}</span></div>
-      <div inline-field><label>${i18next.t('figure.label.repeat-count')}</label><span value>${repeat.count}</span></div>
+      <div choice-row role="radiogroup" aria-label=${i18next.t('figure.label.row-basis')}>
+        <label toggle><input type="radio" name="applied-repeat-mode" .checked=${live(fits)} ?disabled=${unavailable || !record} @change=${() => update({ count: undefined, pitch: pitch && pitch > 0 ? pitch : 200 })}>${i18next.t('figure.label.repeat-fills')}</label>
+        <label toggle><input type="radio" name="applied-repeat-mode" .checked=${live(!fits)} ?disabled=${unavailable || !record} @change=${() => update({ count: repeat.count })}>${i18next.t('figure.label.row-by-count')}</label>
+      </div>
+      <p hint>${i18next.t(fits ? 'figure.text.repeat-mode-pitch' : 'figure.text.repeat-mode-count')}</p>
+      <div inline-field><label>${i18next.t('figure.label.size-source')}</label><span value>${parent ?? '—'}</span></div>
+      <div inline-field><label>${i18next.t('figure.label.repeat-axis')}</label>${axes.length ? html`<select aria-label=${i18next.t('figure.label.repeat-axis')} ?disabled=${unavailable} .value=${live(axis)} @change=${(e: Event) => update({ axis: (e.target as HTMLSelectElement).value.toUpperCase() as 'X' | 'Y' | 'Z' })}>${axes.map(a => html`<option value=${a} ?selected=${a === axis}>${a.toUpperCase()}</option>`)}</select>` : html`<span value>${axis.toUpperCase()}</span>`}</div>
+      <div inline-field><label>${i18next.t('figure.label.repeat-count')}</label>${!fits && record ? html`<input type="number" min="1" step="1" aria-label=${i18next.t('figure.label.repeat-count')} .value=${live(String(repeat.count))} ?disabled=${unavailable} @change=${(e: Event) => update({ count: (e.target as HTMLInputElement).valueAsNumber })}>` : html`<span value>${repeat.count}</span>`}</div>
       <div inline-field>
         <label>${i18next.t('figure.label.repeat-gap')} <span unit>mm</span></label>
-        <input
+        ${fits ? html`<input
           type="number"
           step="any"
-          aria-label=${own}
-          .value=${String(pitch ?? '')}
-          ?disabled=${unavailable || !fits}
+          aria-label=${i18next.t('figure.label.repeat-gap')}
+          .value=${live(String(pitch ?? ''))}
+          ?disabled=${unavailable}
           @change=${(e: Event) => this.setPitch(field, e)}
-        />
+        />` : html`<span value>${pitch === undefined ? '—' : Number(pitch.toFixed(3))}</span>`}
       </div>
-      <ul row-reading>${rowWordsOf(this.rowNow(shown), (key, params) => i18next.t(key, params)).map(line => html`<li>${line}</li>`)}</ul>
-      <p hint>
-        ${fits && length !== undefined && longest !== undefined && limit !== undefined
-          ? i18next.t('figure.text.repeat-what-pitch-gives', {
-              length: length.toLocaleString(),
-              count: repeat.count,
-              longest: longest.toLocaleString(),
-              limit
-            })
-          : i18next.t(fits ? 'figure.text.repeat-fits-length' : 'figure.text.repeat-fixed-count')}
-      </p>
-      ${this.renderPublicPitch(asset, pitch ?? 0, unavailable)}
+      <ul row-reading>${rowWordsOf(this.rowNow(shown), (key, params) => i18next.t(key, params)).slice(1).map(line => html`<li>${line}</li>`)}</ul>
+      <button quiet ?disabled=${unavailable || !removable} @click=${() => this.run(() => this.session!.command({ kind: 'remove-repeat', part: this.selected }))}>${i18next.t('figure.button.remove-repeat')}</button>
+      <p hint>${i18next.t('figure.text.repeat-remove-explained')}</p>
+      ${fits ? this.renderPublicPitch(asset, pitch ?? 0, unavailable) : nothing}
     </section>`
   }
   /** The selected row as it is drawn now, read by figure-model: nothing when it cannot be read. */
@@ -1589,8 +1598,8 @@ export class FigureEditor extends LitElement {
   /** A side tab says what it is in words under its icon: the icons alone had to be learnt (designer 2026-09-27, U05). */
   private renderTab(tab: SideTab, icon: string, label: string) {
     /* Its accessible name is the name it shows (designer 2026-09-27); the longer title stays as its tooltip. */
-    const name = i18next.t('figure.label.tab-short-' + tab)
-    return html`<button tab ?on=${this.tab === tab} title=${label} aria-label=${name} @click=${() => (this.tab = tab)}>
+    const name = i18next.t(tab === 'response' && this.selected ? 'figure.label.tab-arrangement' : 'figure.label.tab-short-' + tab)
+    return html`<button tab ?on=${this.tab === tab} title=${tab === 'response' && this.selected ? name : label} aria-label=${name} @click=${() => (this.tab = tab)}>
       <md-icon>${icon}</md-icon><span tab-label aria-hidden="true">${name}</span>
     </button>`
   }
@@ -1717,8 +1726,8 @@ export class FigureEditor extends LitElement {
     if (!summary) return nothing
     return html`<div subject>
       <div subject-summary>
-        ${summary.kind ? html`<b>${summary.kind}</b>` : nothing}
-        ${summary.size ? html`<small>${summary.size}</small>` : nothing}
+        <b>${ownerOf(this.selected)}</b>
+        ${summary.kind ? html`<small>${summary.kind}</small>` : nothing}
       </div>
     </div>`
   }

package/client/modeller/fasten-geometry.ts

@@ -126,7 +126,8 @@ export function overhangOf(view: FaceView, at: Record<Axis, number>): { touching
 }
 
 /** Whether the part turns about a joint: then 가운데를 면에 is offered -- a roller spinning in place. */
-export const turnsInPlace = (asset: FigureAsset, part: string) => (asset.joints ?? []).some(j => j.body1.part === part && (j.type === 'revolute' || j.origin?.facing === 'centre'))
+export const turnsInPlace = (asset: FigureAsset, part: string) =>
+  (asset.joints ?? []).some(j => j.body1.part === part && ((j.motion ?? []).some(a => a.type === 'revolute') || j.origin?.facing === 'centre'))
 
 /**
  * How the face lies on the screen: X across when it is one of the face's axes, else Z; the other down the screen -- up