Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix(compass-components): don't cut off list view value editor tooltips COMPASS-8581 #6541

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Nov 29, 2024

I don't know why the portal was turned off there. The expand/collapse buttons are very similar in that they have tooltips and they use portals, so it should be safe to do that here too?

I noticed that the key editor's error tooltip doesn't work at all. Can't immediately tell why. I filed a ticket. The only possible error there seems to be a duplicate key. It does go red to show that it is an error, so the validation does work. The tooltip just doesn't trigger.

@github-actions github-actions bot added the fix label Nov 29, 2024
@lerouxb lerouxb changed the title fix(compass-components): don't cut off list view value editor tooltips fix(compass-components): don't cut off list view value editor tooltips COMPASS-8581 Nov 29, 2024
Copy link
Member

@Anemy Anemy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These were without portal as they would teleport to the top left of the screen when a user had one open and then navigated to another tab. It may have been an old tooltip issue that was fixed along the way.
Related pr #3574

@lerouxb lerouxb merged commit 8fec5d4 into main Dec 4, 2024
30 checks passed
@lerouxb lerouxb deleted the fix-tooltip-nesting branch December 4, 2024 08:09
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