Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(datagrid): propagate cell rendering changes only to owned rows (backport to 16.x) #1619

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Backport 6c4b36e from #1489.
This change prevents propagation of cell rendering changes to rows owned by nested datagrids.

CDE-2199
closes #1485

PR Checklist

  • Tests for the changes have been added (for bug fixes / features)
  • [N/A] Docs have been added / updated (for bug fixes / features)
  • [N/A] If applicable, have a visual design approval

PR Type

Bugfix

What is the current behavior?

If the detail pane of a datagrid contains another datagrid with less columns than the outer datagrid, closing the detail pane results in the error: row.cells.get(...) is undefined.

Issue Number: #1485, CDE-2199

What is the new behavior?

No error is thrown, and the correct cells are hidden.

Does this PR introduce a breaking change?

No.

…1489)

This change prevents propagation of cell rendering changes to rows owned
by nested datagrids.

CDE-2199
closes #1485

(cherry picked from commit 6c4b36e)
Copy link
Contributor Author

github-actions bot commented Nov 18, 2024

👋 @dtsanevmw,

  • 🙏 The Clarity team thanks you for opening a pull request
  • 🎉 The build for this PR has succeeded
  • 🔍 The PR is now ready for review
  • 🍿 In the meantime, view a preview of this PR
  • 🖐 You can always follow up here. If you're a VMware employee, you can also reach us on our internal Clarity Support space

Thank you,

🤖 Clarity Release Bot

@dtsanevmw dtsanevmw merged commit d79482c into 16.x Nov 18, 2024
8 checks passed
@dtsanevmw dtsanevmw deleted the backport-1489-to-16.x branch November 18, 2024 11:15
Copy link
Contributor Author

🎉 This PR is included in version 16.5.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link
Contributor Author

github-actions bot commented Dec 4, 2024

Hi there 👋, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed PRs after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants