Part of #8820
While checking some functional test failures in Kibana with the new grid layout, I noticed that the popover position is incorrect in some cases.
In this case, we have a popover that doesn't have repositionOnScroll. It used to scroll correctly, but with the new layout, it doesn't anymore.
The reason for this is that the main application scroll has moved from the document to the inner grid cell.
It is hard to estimate how widespread this issue is, but I'd like to avoid adding repositionOnScroll to each popover manually. Moreover, this seems to be an issue not only specific to the grid layout but also to any popover that we position inside a nested scroll (e.g., flyout or modal).
Proposals:
- I wonder if we should make
repositionOnScroll the default?
- Or at least allow make it configurable as default via default props?
- Is this option a significant performance overhead?
- any other options or considerations?
Part of #8820
While checking some functional test failures in Kibana with the new grid layout, I noticed that the popover position is incorrect in some cases.
In this case, we have a popover that doesn't have
repositionOnScroll. It used to scroll correctly, but with the new layout, it doesn't anymore.The reason for this is that the main application scroll has moved from the document to the inner grid cell.
It is hard to estimate how widespread this issue is, but I'd like to avoid adding
repositionOnScrollto each popover manually. Moreover, this seems to be an issue not only specific to the grid layout but also to any popover that we position inside a nested scroll (e.g., flyout or modal).Proposals:
repositionOnScrollthe default?- Is this option a significant performance overhead?