odoo/odoo#195841
Created by fw-bot
Merged
at 7d88106ea35e6fbff789cc385c62982ba81c5256
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:saas-18.1-18.0-fix-carousel-controllers-options-seba-ld-I-fw
- head
- eb3443ef65ab013034865c446e889b5f0a08b650
- merged
- 4 months ago by Website, Serge Bayet (seba)
| odoo/odoo | |
|---|---|
| 18.0 | #189858 |
| saas-18.1 | #195841 |
| saas-18.2 | |
| saas-18.3 | |
| master | #195869 |
[FW][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
Forward-Port-Of: #189858