Skip to content

Parametrische symboolvoorinstellingen en bewerkbare wapening - #317

Merged
DutchSailor merged 7 commits into
mainfrom
codex/parametric-nl-ifc-symbols
Jul 23, 2026
Merged

Parametrische symboolvoorinstellingen en bewerkbare wapening#317
DutchSailor merged 7 commits into
mainfrom
codex/parametric-nl-ifc-symbols

Conversation

@DutchSailor

@DutchSailor DutchSailor commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Samenvatting

  • toont de eigenschappen van een parametrisch palette-element direct vóór plaatsing;
  • bewaart gevalideerde voorinstellingen afzonderlijk per symbooltype en gebruikt dezelfde parameters voor preview en definitieve plaatsing;
  • ondersteunt 1–4 vlaggen op een wapeningsstaaf, boven of onder de staaf;
  • maakt labels van wapeningsstaaf, netwapening en wapeningskorf direct bewerkbaar met rotatiebewuste hit-testing;
  • verwerkt bevestigen als één undo-stap en annuleert veilig bij Escape, tool- of paginawissel;
  • herstelt na labelbewerking de focus op het juiste annotatiecanvas, ook in doorlopende weergave;
  • blokkeert onbekende symboolsjablonen vóór documentmutatie;
  • synchroniseert de app-entry in de Tauri-lockfile met versie 1.82.0.

Waarom

Tot nu toe verschenen parametrische eigenschappen pas na plaatsing en gebruikte de creator altijd de kale sjabloonwaarden. Daardoor konden gebruikers een element niet vooraf configureren en waren wapeningsteksten niet direct op het object wijzigbaar. Deze PR voegt één gedeelde parameterstroom toe voor palette, preview, plaatsing en nabewerking.

Validatie

  • parametrische defaults: PASS;
  • label/lifecycle/undo/focus: 60/60;
  • NL IFC-componenten: 87/87;
  • tweepunts- en parameterpersistentie: 42/42;
  • wapeningskorf: 144/144;
  • stavenreeks: 325/325;
  • npm run typecheck: geslaagd;
  • npm run test:unit: 16/16;
  • npm run test:quality: 23/23;
  • npm run build: geslaagd, 1067 modules.

De branchbrede code-review is na fixes als merge-gereed beoordeeld.

@DutchSailor
DutchSailor marked this pull request as ready for review July 23, 2026 20:03
@DutchSailor
DutchSailor merged commit 3020cff into main Jul 23, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant