[FIX] web: properly compute popover position

- avoid updating default position list in place
- ensure computed position takes into account enough margin with border
- fallback to requested position instead of bottom

Part of task-2270731

closes odoo/odoo#58368

X-original-commit: 3fad17421518732db2085c3eec596600cfe34f03
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/addons/web/static/tests/core/popover_tests.js b/addons/web/static/tests/core/popover_tests.js
index 6b8159d73d1..4b5002d3350 100644
--- a/addons/web/static/tests/core/popover_tests.js
+++ b/addons/web/static/tests/core/popover_tests.js
@@ -137,8 +137,8 @@ odoo.define('web.popover_tests', function (require) {
             await changeProps('position', 'left');
             await testUtils.dom.click('#target');
             assert.ok(
-                pointsTo(document.querySelector('.o_popover'), element, 'top'),
-                "Popover should be top-positioned because it doesn't fit left"
+                pointsTo(document.querySelector('.o_popover'), element, 'right'),
+                "Popover should be right-positioned because it doesn't fit left"
             );
             await testUtils.dom.click('#passiveTarget');
             parent.destroy();