fix(dom): cross-document/iframe support#2043
Conversation
✅ Deploy Preview for vibrant-gates-22c214 canceled.
|
@floating-ui/dom (1.0.12 → 1.1.0)Refactors
Packages With No ChangesThe following packages have no user facing changes, so won't be released:
|
|
Hey @atomiks, is this fix already included in the latest version of the I gave it a quick look and it doesn't seem to fix the issues that we were trying to fix in https://github.com/WordPress/gutenberg/ |
|
Hey yeah it's in |
|
I'm not quite sure — I gave a quick look at the visual tests and they seem comprehensive. I've opened a draft PR WordPress/gutenberg#46845 to hopefully help debugging what's going on. Would be open to having a chat so that we can more easily discuss the issue at hand? If so, you can find me on the Make WordPress slack as |
|
@ciampo yes sure, I'll join soon |
|
Awesome! Cc'ing @jsnajdr who is potentially interested in collaborating on this too. |
This takes into account the
xandyoffset of aniframeand its scale.Fixes #1594
Fixes #1711
Fixes #1869
Fixes #1870
Any issues? Added nesting support, tests for both inside vs. outside + nesting. Seems to work with
limitShiftas well.@mishushakov
@talldan
@ciampo
@seahindeniz