fix(control): apply control action padding when visibility changes

[ashleyryan]

fixes #182

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?

• [x] Bugfix
• [ ] Feature
• [ ] Code style update (formatting, local variables)
• [ ] Refactoring (no functional changes, no api changes)
• [ ] Build related changes
• [ ] CI related changes
• [ ] Documentation content changes
• [ ] clarity.design website / infrastructure changes
• [ ] Other... Please describe:

What is the current behavior?

The padding is set for the control action only on the initial render, in firstUpdated. If the element is hidden at the time, the padding will be incorrect when the element becomes visible.

Issue Number: #182

What is the new behavior?

There's now an intersection observer listening for visibility changes that will set the padding correctly

Does this PR introduce a breaking change?

• [ ] Yes
• [x] No

Other information

[github-actions[bot]]

👋 @ashleyryan,

• 🙏 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, checkout out 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 Slack channel

Thank you,

🤖 Clarity Release Bot

[github-actions[bot]]

:tada: This PR is included in version 6.2.1 :tada:

The release is available on:

• GitHub release
• v6.2.1

Your semantic-release bot :package::rocket:

[github-actions[bot]]

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.