pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FIX] sale_product_configurator: fix alignment value and radio button

Browse Source 
Issue:
   Currently there is no alignment between radio button and value,
   we fix this commit.

Fix:
   We move the input box inside the label.
step:
    - Install sale app
    - Create product(test) and add Attributes & Variants
    - Select Brand  (Display Type: radio) Attributes and add value
    - Create sale order and add product(test)
    - Check Configure

closes odoo/odoo#134734

X-original-commit: 9e32999d3eea96da6d0ab39642e3f2d612a8690a
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
This commit is contained in:
Prakash Prajapati
2023-09-08 12:24:48 +00:00
committed by Victor Feyens
parent adc5448adf
commit 4a3bf1f847
2 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/sale_product_configurator/static/src/js/product_template_attribute_line/product_template_attribute_line.xml
+8 -8
View File
@@ -48,15 +48,15 @@
t-if="ptav.price_extra"
price="ptav.price_extra"
currencyId="this.env.currencyId"/>
<input
type="radio"
class="form-check-input"
t-att-id="ptav.id"
t-att-value="ptav.id"
t-att-checked="this.props.selected_attribute_value_ids.includes(ptav.id)"
t-att-name="'ptal-' + this.props.id"
t-on-change="updateSelectedPTAV"/>
</label>
<input
type="radio"
class="form-check-input"
t-att-id="ptav.id"
t-att-value="ptav.id"
t-att-checked="this.props.selected_attribute_value_ids.includes(ptav.id)"
t-att-name="'ptal-' + this.props.id"
t-on-change="updateSelectedPTAV"/>
</div>
</li>
</ul>
addons/test_sale_product_configurators/static/tests/tours/product_configurator_edition_ui.js
+3 -3
View File
@@ -40,13 +40,13 @@ registry.category("web_tour.tours").add('sale_product_configurator_edition_tour'
}, {
trigger: '.fa-pencil',
}, {
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk"))>td:nth-child(2)>ul:nth-child(3)>li:nth-child(2)>div label:contains("Aluminium") ~ input:checked',
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk"))>td:nth-child(2)>ul:nth-child(3)>li:nth-child(2)>div label:contains("Aluminium") > input:checked',
isCheck: true, // check updated legs
}, {
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk"))>td:nth-child(4) span:contains("800.40")',
isCheck: true, // check updated price
}, {
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk")) label:contains("Custom") ~ input',
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk")) label:contains("Custom") > input',
}, {
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk"))>td:nth-child(2)>input',
run: 'text nice custom value'
@@ -83,7 +83,7 @@ registry.category("web_tour.tours").add('sale_product_configurator_edition_tour'
}, {
trigger: '.fa-pencil',
}, {
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk")) label:contains("Steel") ~ input',
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk")) label:contains("Steel") > input',
}, {
trigger: 'tr:has(.o_sale_product_configurator_name:contains("Customizable Desk (TEST) (Steel, Black)"))',
isCheck: true,