Skip to content

feat: Hoekpunten van oppervlaktemetingen direct bewerken (#872) - #874

Closed
oscar981-op wants to merge 1 commit into
OpenAEC-Foundation:mainfrom
oscar981-op:feat/tauri-area-vertices-872
Closed

oscar981-op wants to merge 1 commit into
OpenAEC-Foundation:mainfrom
oscar981-op:feat/tauri-area-vertices-872

Conversation

@oscar981-op

@oscar981-op oscar981-op commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Een geselecteerde oppervlaktemeting bood geen directe hoekpuntbewerking op willekeurige randposities. Deze wijziging voegt bij rechtsklik een hoekpunt op de aangeklikte positie toe of verwijdert het aangeklikte hoekpunt. Dubbelklik op een rand voegt een punt in het midden toe; dubbelklik op een hoekpunt verwijdert het.

Dit werkt op de buitencontour en op gaten. Iedere ring houdt minstens drie punten. De meetwaarde en het eigenschappenpaneel worden bijgewerkt, met één undo-stap per bewerking. De mutatie controleert het actieve document, de pagina, annotatie-/laagvergrendeling, zichtbaarheid en alleen-lezenstatus opnieuw. Bestaande vertalingen worden hergebruikt.

Closes #872.

Validatie:

  • Vier gerichte geometrie-regressietests: cursorprojectie versus midden, sluitrand, gaten, minimale driehoek, vergrendeling en boogbehoud.
  • Echte Chromium/PDF.js-muisbediening: contextmenu, dubbelklik, undo/redo, meetwaarde, gaten, zoom en 90° weergaverotatie. Alleen-lezen document, vergrendelde laag, andere pagina en verdwenen annotatie weigeren wijzigingen.
  • Volledige unitrun: 6.205 tests, 6.186 geslaagd, 19 overgeslagen, 0 fouten. Typecontrole, productiebuild en versiecontrole geslaagd op deze wijziging.

Beperkingen: deze snelbediening geldt voor rechte polygonale meetcontouren. Boogranden en het verwijderen van een hoek naast een boog blijven bij de bestaande contourbewerking, zodat geen kromme stilzwijgend verandert. Native WebView is lokaal niet getest.

Versie 2026.44.22 volgens de repositoryworkflow. Deze PR blijft concept zolang installercontroles en verplichte reviews ontbreken. Er wordt geen release gepubliceerd.

Installerstatus: dispatch van release.yml op deze branch is door GitHub afgewezen met HTTP 422: “Actions has been disabled for this repository.” Er bestaat daardoor geen installer-run voor deze commit. Conceptrelease v2026.44.22 is aangemaakt en als ongepubliceerd gecontroleerd. Upstream macOS CI faalt op de bestaande test stored definitions stay compact for image watermarks (log gecontroleerd); de afzonderlijke fix #868 heeft alle vijf CI-controles groen. Alle vier overige controles zijn geslaagd: Ubuntu, Windows, render-regression en hosted CSP. Vereiste review ontbreekt en het account heeft geen upstream mergerechten.

@DutchSailor DutchSailor mentioned this pull request Oct 31, 2026
@DutchSailor

Copy link
Copy Markdown
Collaborator

Thank you @oscar981-op! This is released in 2026.44.5 as one squashed commit with you as the author (cce7f69, merged through #915).

Review fixes:

  • Removing a corner now also refreshes the cached box of the area
    (x/y/width/height), as a node drag does. Loaded areas carry that box; it
    places the rotate grip and is the rotation centre, so it went stale.
  • One hit reach (AREA_VERTEX_HIT_PX) for the context menu, the double-click
    and the select tool; the context menu used a smaller reach, so a click
    that kept the selection could miss the edge in the menu.
  • The "Delete vertex" item gets the delete icon, like the polygon menu.
  • New round-trip test with the real saver and loader: after inserting and
    removing vertices (outer ring and hole) on /Rotate 0, 90, 180 and 270, the
    /Vertices, /OPS_Holes, the appearance stream and the reopened model hold
    the same vertices and the same area. It also checks that one edit is one
    undo step and that redo restores points and label.
  • Unit tests for the box refresh and for the zoom-independent hit reach.
  • The browser e2e script no longer writes a screenshot to a fixed path.

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.

[Feature]: Add/remove vertex points to area measurements

2 participants