Skip to content

[Tooltip, PreviewCard] Use FloatingPortalLite#1278

Merged
atomiks merged 2 commits intomui:masterfrom
atomiks:feat/floating-portal-lite
Jan 8, 2025
Merged

[Tooltip, PreviewCard] Use FloatingPortalLite#1278
atomiks merged 2 commits intomui:masterfrom
atomiks:feat/floating-portal-lite

Conversation

@atomiks
Copy link
Contributor

@atomiks atomiks commented Jan 2, 2025

Part of #1246. Depends on refactors made in #1222

This prevents bringing in tabbable logic (tabbable dependency and extra code from Floating UI) if just importing the Tooltip or PreviewCard components, saving bundle size

@atomiks atomiks added component: tooltip Changes related to the tooltip component. component: preview card Changes related to the preview card component. labels Jan 2, 2025
@mui-bot
Copy link

mui-bot commented Jan 2, 2025

Netlify deploy preview

https://deploy-preview-1278--base-ui.netlify.app/

Generated by 🚫 dangerJS against 077387c

@atomiks atomiks force-pushed the feat/floating-portal-lite branch from 6a6c05d to 077387c Compare January 7, 2025 11:15
@atomiks atomiks marked this pull request as ready for review January 7, 2025 11:19
Copy link
Member

@michaldudak michaldudak left a comment

Choose a reason for hiding this comment

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

LGTM.

Side note: I'd appreciate it if you could write more verbose PR descriptions, so in a year from now, it's easier to figure out why a particular piece of code was added.

@atomiks
Copy link
Contributor Author

atomiks commented Jan 8, 2025

@michaldudak updated description

@atomiks atomiks merged commit 31cea62 into mui:master Jan 8, 2025
@atomiks atomiks deleted the feat/floating-portal-lite branch January 8, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: preview card Changes related to the preview card component. component: tooltip Changes related to the tooltip component.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants