DYN-10112: Notes should not overlay warning/error/info bubbles over nodes#16893
Merged
Conversation
Introduce Configurations.InfoBubbleBaseZIndex (1,000,000) and use it as the default ZIndex for info/error/warning bubbles. Decouple bubble ZIndex from node ZIndex by removing node-driven updates in NodeViewModel and assigning the base ZIndex when building the bubble. Update InfoBubbleView to elevate the ViewModel.ZIndex to int.MaxValue - 1 when showing/fading in the bubble and restore it to the base when hiding/fading out. This ensures bubbles render above nodes and notes reliably when visible.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-10112
Contributor
There was a problem hiding this comment.
Pull request overview
Adjusts InfoBubble Z-indexing in Dynamo’s WPF UI to prevent info/warning/error bubbles from being visually obscured by nodes and notes, improving reliability of message visibility.
Changes:
- Introduces
Configurations.InfoBubbleBaseZIndexas a default/idle ZIndex for info bubbles. - Decouples bubble ZIndex from node ZIndex updates by removing node-driven bubble ZIndex adjustments.
- Updates
InfoBubbleViewto boost ZIndex on show/fade-in and restore on hide/fade-out.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/DynamoCoreWpf/Views/Preview/InfoBubbleView.xaml.cs | Boost/restore bubble ZIndex during show/hide and fade transitions. |
| src/DynamoCoreWpf/ViewModels/Core/NodeViewModel.cs | Removes node-driven bubble ZIndex updates; initializes bubble ZIndex from a base configuration value. |
| src/DynamoCore/Configuration/Configurations.cs | Adds a base ZIndex constant intended to keep bubbles above nodes/notes. |
This reverts commit 3a229e3.
Store the bubble's previous ZIndex and temporarily boost it when the info bubble is hovered to ensure expanded content isn't blocked by other elements. Adds a previousZIndex field, sets ViewModel.ZIndex to int.MaxValue-1 on MouseEnter, and restores the original value on MouseLeave.
|
jasonstratton
approved these changes
Feb 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Simplified approach:
Boosted z-index on mouse enter to max value and restore on mouse leave
Declarations
Check these if you believe they are true
Release Notes