odoo/odoo#189858
Created by Website, Serge Bayet (seba)
Merged
at 247f28fdec788c7eb7c4288db29b931c73a23757
Statuses:
- legal/cla: Contributor License Agreement check
- ci/runbot: Odoo Test Suite
- ci/upgrade_enterprise: Test upgrades for enterprise master
- ci/template: Contact runbot team on discord for help.
- ci/style: Optional style check. Ignore it only if strictly necessary.
- ci/security: Required security check. Can only be ignored by security team.
- label
- odoo-dev:18.0-fix-carousel-controllers-options-seba
- head
- f558caae01ef79801fe2081bae210008d6b80e7e
- merged
- 5 months ago by Website, Quentin Smetz (qsm)
| odoo/odoo | |
|---|---|
| 18.0 | #189858 |
| saas-18.1 | #195841 |
| saas-18.2 | |
| saas-18.3 | |
| master | #195869 |
[FIX] website: correct carousel controllers options handling
Since 1, a method was introduced to prevent the "controllers" options from being centered when arrows and indicators are displayed. However, it was implemented in the wrong overridden method (computeWidgetState), which cannot update the DOM. Additionally, it failed to account for the preview mode.
This commit resolves these issues.
Steps to reproduce:
- Website - Edit mode
- Drag and drop a "carousel intro" snippet onto the page.
- Select the option "Controllers > Arrows > Hidden."
- Then select the option "Controllers > Centered."
- The controllers are now displayed as centered.
- Hover over the "Controllers > Arrows" options and preview a value other than "Hidden."
- The preview does not update correctly.
Related to:
task-4094405