Fix popover repositioning #528
Conversation
Signed-off-by: HichamELBSI <elabbassih@gmail.com>
|
This pull request is being automatically deployed with Vercel (learn more). design-system – ./🔍 Inspect: https://vercel.com/strapijs/design-system/5cGSP7kykBdeqsTSiQnVUCZxogc1 design-system-website – ./website🔍 Inspect: https://vercel.com/strapijs/design-system-website/TCLxrP7436S9LzQAeaJrAGevNNaw |
| ); | ||
|
|
||
| useResizeObserver(source, () => | ||
| useResizeObserver([source, popoverRef], () => |
There was a problem hiding this comment.
ℹ️ Add the popover as a source to observe in order to trigger the setPosition on every changes on this element.
| useEffect(() => { | ||
| const resizeObs = new ResizeObserver(onResize); | ||
| resizeObs.observe(source.current); | ||
| if (Array.isArray(sources)) { |
There was a problem hiding this comment.
ℹ️ Be able to observe multiple sources
There was a problem hiding this comment.
that's so nice, thank you!
ronronscelestes
left a comment
ronronscelestes
left a comment
There was a problem hiding this comment.
YAY well done 🔥🙏
3 participants
Fix popover repositioning. Useful for the Combobox component.