Skip to content

fix(WSIViewport): add event listener for annotation removal#2461

Merged
sedghi merged 9 commits into
cornerstonejs:mainfrom
mbellehumeur:fix/wsiViewport-delete-measurement
Nov 28, 2025
Merged

fix(WSIViewport): add event listener for annotation removal#2461
sedghi merged 9 commits into
cornerstonejs:mainfrom
mbellehumeur:fix/wsiViewport-delete-measurement

Conversation

@mbellehumeur
Copy link
Copy Markdown
Contributor

@mbellehumeur mbellehumeur commented Nov 14, 2025

Context

ticket: 18378695060

Measurement is deleted but the canvas is not refreshed.

DeleteMeasurementProblem.mp4

Changes & Results

Added listener for delete annotation that calls postRender.

Testing

Use patient SIIM, Ravi

DeleteAfter.mp4

Checklist

PR

  • My Pull Request title is descriptive, accurate and follows the
    semantic-release format and guidelines.

Code

  • My code has been well-documented (function documentation, inline comments,
    etc.)

Comment thread packages/core/src/RenderingEngine/WSIViewport.ts Outdated
Comment thread packages/core/src/RenderingEngine/WSIViewport.ts
Comment thread packages/core/src/RenderingEngine/WSIViewport.ts
@mbellehumeur mbellehumeur requested a review from sedghi November 24, 2025 16:08
@sedghi
Copy link
Copy Markdown
Member

sedghi commented Nov 25, 2025

can you pull from main, ci is failing

Comment thread packages/core/src/enums/Events.ts Outdated
@sedghi sedghi merged commit 026c69d into cornerstonejs:main Nov 28, 2025
10 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.

3 participants