fix(datagrid): render datagrid in initially hidden and OnPush components

[dtsanevmw]

PR Checklist

Please check if your PR fulfills the following requirements:

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

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• Other... Please describe:

What is the current behavior?

Issue Number: CDE-2562

What is the new behavior?

Does this PR introduce a breaking change?

• Yes
• No

Other information

[github-actions]

👋 @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

[github-actions]

This PR introduces visual changes: 20f7aec
If these changes are intended and correct, please cherry-pick the above commit to this PR.

git checkout dtsanevmw/cde-2562
git fetch https://github.com/vmware-clarity/ng-clarity.git 20f7aec4b751e434bfca09ed6e60b2e66e646149
git cherry-pick 20f7aec4b751e434bfca09ed6e60b2e66e646149
git push

[github-actions]

This PR introduces visual changes: 33997a8
If these changes are intended and correct, please cherry-pick the above commit to this PR.

git checkout dtsanevmw/cde-2562
git fetch https://github.com/vmware-clarity/ng-clarity.git 33997a857c1ee26cda15519249c2b6b952f02015
git cherry-pick 33997a857c1ee26cda15519249c2b6b952f02015
git push

[kevinbuhmann]

I would reword the PR title to "fix(datagrid): render datagrid in tabs with OnPush change detection" or similar. Describe the behavioral bug fix, not the technical code change. That would be better for the changelog.

[kevinbuhmann]

This looks fine to me. I didn't test it myself though. Just have the once small simplification comment and then this is good to merge.