Skip to content

[Cherry-pick] [DYN-8560] Chart nodes: Resize handle and Preview icon overlap#16220

Merged
zeusongit merged 1 commit intoRC3.5.1_masterfrom
cherrypick-16142
May 16, 2025
Merged

[Cherry-pick] [DYN-8560] Chart nodes: Resize handle and Preview icon overlap#16220
zeusongit merged 1 commit intoRC3.5.1_masterfrom
cherrypick-16142

Conversation

@github-actions
Copy link

Cherry-pick of #16142 to RC3.5.1_master

Purpose

This PR addresses DYN-8560, which reported an issue where the resize thumb and the preview icon overlap, making the thumb difficult or impossible to use.

To resolve this, I’ve applied the same approach used in Curve Mapper to all chart nodes by setting their IsResizable property to true. This adjustment shifts the glyph icon to the left, creating room for the resize thumb.

I’ve also updated the cursor appearance when hovering over the resize thumb to match the behavior seen in Curve Mapper.

DYN-8560

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated
  • This PR contains no files larger than 50 MB

Release Notes

Chart nodes are now resizable, with improved cursor behavior and layout adjustments to prevent overlap between the resize thumb and preview icon.

Reviewers

@reddyashish
@zeusongit

FYIs

@dnenov
@achintyabhat

@zeusongit zeusongit merged commit 28f1935 into RC3.5.1_master May 16, 2025
11 of 12 checks passed
@zeusongit zeusongit deleted the cherrypick-16142 branch May 16, 2025 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants