fix(context menu): Fix for context menu use when zoomed out

jeff-phillips-18 commented 3 weeks ago

What

Closes #247

Description

Adds the option to not raise the label when the node is hovered. This keeps the label on the same layer as the node which avoids getting a mouse leave event when moving from the node to the label. For applications that are raising the node to the TOP_LEVEL on hover, they should pass raiseLabelOnHover={false} to the DefaultNode.

Also, when the context menu is open, treat the node label the same as if hovering so that it remains visible and un-truncated while the context menu is open.

Type of change

[ ] Feature
[x] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

Hover_to_context

patternfly-build commented 3 weeks ago

Preview: https://react-topology-pr-topology-249.surge.sh

jeff-phillips-18 commented 2 weeks ago

Just noticed when the context menu is accessed / clicked from the Context Menu Labels demo , the node icon is no longer visible

@jenny-s51 Fixed. Thanks.

jeff-phillips-18 commented 2 weeks ago

Released in v5.4.1

patternfly / react-topology