Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Popped out widget crashes viewport when exiting ConfigurableUiContent (Electron) #951

Open
jsnaras opened this issue Aug 8, 2024 · 0 comments
Assignees
Labels
bug Something isn't working popouts This issue is related to popouts

Comments

@jsnaras
Copy link

jsnaras commented Aug 8, 2024

Describe the bug

In an Electron app, when a measurement widget is active and popped out and the user navigates away from a page that contains the ConfigurableUiContent component, the popped out widget is not deinitialized properly and causes a crash.

To Reproduce

Prerequisites: 2 separate "pages" in the app, one of them having the ConfigurableUiContent and another one without it; measurement tools available.

  1. Open the page with ConfigurableUiContent, activate a measurement tool, select target, observe docked measurement tool result viewport working correctly
  2. Pop out the newly opened measurement viewport
  3. Navigate away from the page (to the one not containing ConfigurableUiContent), the popped out widget closes (as expected)
  4. Navigate back to the page with ConfigurableUiContent, the app crashes with an assertion error

Expected Behavior

The popped out measurement widget window is closed correctly when unmounting ConfigurableUiContent and does not crash when re-entering ConfigurableUiContent

Screenshots

No response

Desktop (please complete the applicable information)

  • OS: Windows 11
  • Browser: Chromium (Electron)
  • Browser version: 124.0.6367.49
  • iTwin.js AppUI version: 4.14.2

Additional context

No response

@jsnaras jsnaras added the bug Something isn't working label Aug 8, 2024
@GerardasB GerardasB self-assigned this Aug 8, 2024
@GerardasB GerardasB added the popouts This issue is related to popouts label Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working popouts This issue is related to popouts
Projects
None yet
Development

No branches or pull requests

2 participants