Skip to content

Conversation

@malwilley
Copy link
Member

For the navigation tour, we want to both the selected primary item and the elements in the secondary sidebar, so multiple tour elements needs to be used.

This PR prevents the tooltip from being rendered if nothing is passed for the title or description. That allows us to highlight multiple elements where only one displays a tooltip.

CleanShot 2025-03-21 at 16 35 25@2x

@malwilley malwilley requested review from a team and leeandher March 21, 2025 23:39
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 21, 2025
@malwilley malwilley changed the title feat(tour): Make tooltips optional in the tour feat(tour): Allow some elements to not render a tooltip at all Mar 21, 2025
@malwilley malwilley merged commit 7d3bd71 into master Mar 24, 2025
42 checks passed
@malwilley malwilley deleted the malwilley/feat/tour-optional-tooltip branch March 24, 2025 16:42
andrewshie-sentry pushed a commit that referenced this pull request Mar 27, 2025
For the navigation tour, we want to both the selected primary item and
the elements in the secondary sidebar, so multiple tour elements needs
to be used.

This PR prevents the tooltip from being rendered if nothing is passed
for the title or description. That allows us to highlight multiple
elements where only one displays a tooltip.
@github-actions github-actions bot locked and limited conversation to collaborators Apr 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants