From 1ea9cc1c3821ba11157de9f0ff3bb3a8bed329c0 Mon Sep 17 00:00:00 2001 From: "Arthur Detroux (ard)" Date: Tue, 6 Aug 2024 10:40:44 +0200 Subject: [PATCH] [WIP] fixup! [REF] web_editor, website: convert s_website_form options to Owl Kept in a separate commit as this fix only makes sense if the changes that are in the parent commit are accepted --- .../static/tests/tours/website_form_editor.js | 32 +++++++++---------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/addons/website/static/tests/tours/website_form_editor.js b/addons/website/static/tests/tours/website_form_editor.js index f623f66d6d1e1..dae331f104cac 100644 --- a/addons/website/static/tests/tours/website_form_editor.js +++ b/addons/website/static/tests/tours/website_form_editor.js @@ -272,31 +272,31 @@ ...addCustomField('one2many', 'checkbox', 'Products', true), { content: "Change Option 1 label", - trigger: 'we-list table input:eq(0)', + trigger: 'we-list:visible table input:eq(0)', run: "edit Iphone", }, { content: "Change Option 2 label", - trigger: 'we-list table input:eq(1)', + trigger: 'we-list:visible table input:eq(1)', run: "edit Galaxy S", },{ content: "Change first Option 3 label", - trigger: 'we-list table input:eq(2)', + trigger: 'we-list:visible table input:eq(2)', run: "edit Xperia", }, { // TODO: Fix code to avoid this behavior content: "Click outside focused element before click on add new checkbox otherwise button does'nt work", - trigger: "we-list we-title", + trigger: "we-list:visible we-title", run: "click", }, { content: "Click on Add new Checkbox", - trigger: 'we-list we-button.o_we_list_add_optional', + trigger: 'we-list:visible we-button.o_we_list_add_optional', run: "click", }, { content: "Change added Option label", - trigger: 'we-list table input:eq(3)', + trigger: 'we-list:visible table input:eq(3)', run: "edit Wiko Stairway", }, { content: "Check the resulting field", @@ -321,15 +321,15 @@ ...addCustomField('selection', 'radio', 'Service', true), { content: "Change Option 1 label", - trigger: 'we-list table input:eq(0)', + trigger: 'we-list:visible table input:eq(0)', run: "edit After-sales Service", }, { content: "Change Option 2 label", - trigger: 'we-list table input:eq(1)', + trigger: 'we-list:visible table input:eq(1)', run: "edit Invoicing Service", }, { content: "Change first Option 3 label", - trigger: 'we-list table input:eq(2)', + trigger: 'we-list:visible table input:eq(2)', run: "edit Development Service", }, { @@ -344,7 +344,7 @@ run: "click", }, { content: "Change last Option label", - trigger: 'we-list table input:eq(3)', + trigger: 'we-list:visible table input:eq(3)', run: "edit Management Service", }, { content: "Mark the field as not required", @@ -373,15 +373,15 @@ // Customize custom selection field { content: "Change Option 1 Label", - trigger: 'we-list table input:eq(0)', + trigger: 'we-list:visible table input:eq(0)', run: "edit Germany", }, { content: "Change Option 2 Label", - trigger: 'we-list table input:eq(1)', + trigger: 'we-list:visible table input:eq(1)', run: "edit Belgium", }, { content: "Change first Option 3 label", - trigger: 'we-list table input:eq(2)', + trigger: 'we-list:visible table input:eq(2)', run: "edit France", }, { @@ -403,12 +403,12 @@ }, { content: "Change last Option label", - trigger: 'we-list table input:eq(3)', + trigger: 'we-list:visible table input:eq(3)', // TODO: Fix code to avoid blur event run: "edit Canada", }, { content: "Remove Germany Option", - trigger: '.o_we_select_remove_option:eq(0)', + trigger: '.o_we_select_remove_option:visible:eq(0)', run: "click", }, { @@ -423,7 +423,7 @@ run: "click", }, { content: "Change last option label with a number", - trigger: 'we-list table input:eq(3)', + trigger: 'we-list:visible table input:eq(3)', run: "edit 44 - UK", }, { content: "Check that the input value is the full option value",