Skip to content

[WC-3219] Fix: Rich Text edit link tooltip clipped#2033

Merged
gjulivan merged 2 commits intomainfrom
rte-floating-tooltip
Jan 27, 2026
Merged

[WC-3219] Fix: Rich Text edit link tooltip clipped#2033
gjulivan merged 2 commits intomainfrom
rte-floating-tooltip

Conversation

@gjulivan
Copy link
Copy Markdown
Collaborator

@gjulivan gjulivan commented Jan 16, 2026

adding vanilla floating ui with flip and shift to prevent tooltip gets out of bound.

@gjulivan gjulivan requested a review from a team as a code owner January 16, 2026 13:46
iobuhov
iobuhov previously approved these changes Jan 16, 2026
@gjulivan gjulivan force-pushed the rte-floating-tooltip branch from fe508c5 to 0b54901 Compare January 16, 2026 14:31
@gjulivan gjulivan changed the title [WC_3219] Fix: Rich Text edit link tooltip clipped [WC-3219] Fix: Rich Text edit link tooltip clipped Jan 22, 2026
@gjulivan gjulivan force-pushed the rte-floating-tooltip branch from 9529ade to 1837841 Compare January 26, 2026 11:05
@gjulivan gjulivan changed the base branch from main to rte-update January 26, 2026 11:05
leonardomendix
leonardomendix previously approved these changes Jan 26, 2026
@gjulivan gjulivan changed the base branch from rte-update to main January 26, 2026 15:16
@gjulivan gjulivan dismissed leonardomendix’s stale review January 26, 2026 15:16

The base branch was changed.

@gjulivan gjulivan force-pushed the rte-floating-tooltip branch from 1837841 to 7c56bba Compare January 26, 2026 15:16
samuelreichert
samuelreichert previously approved these changes Jan 26, 2026
Copy link
Copy Markdown
Contributor

@samuelreichert samuelreichert left a comment

Choose a reason for hiding this comment

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

:shipit: LGTM

@gjulivan gjulivan merged commit 4914fa6 into main Jan 27, 2026
11 of 12 checks passed
@gjulivan gjulivan deleted the rte-floating-tooltip branch January 27, 2026 14:44
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.

4 participants