Skip to content

Commit 1ea9cc1

Browse files
committed
[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
1 parent 0565709 commit 1ea9cc1

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

addons/website/static/tests/tours/website_form_editor.js

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -272,31 +272,31 @@
272272
...addCustomField('one2many', 'checkbox', 'Products', true),
273273
{
274274
content: "Change Option 1 label",
275-
trigger: 'we-list table input:eq(0)',
275+
trigger: 'we-list:visible table input:eq(0)',
276276
run: "edit Iphone",
277277
}, {
278278
content: "Change Option 2 label",
279-
trigger: 'we-list table input:eq(1)',
279+
trigger: 'we-list:visible table input:eq(1)',
280280
run: "edit Galaxy S",
281281
},{
282282
content: "Change first Option 3 label",
283-
trigger: 'we-list table input:eq(2)',
283+
trigger: 'we-list:visible table input:eq(2)',
284284
run: "edit Xperia",
285285
},
286286
{
287287
// TODO: Fix code to avoid this behavior
288288
content: "Click outside focused element before click on add new checkbox otherwise button does'nt work",
289-
trigger: "we-list we-title",
289+
trigger: "we-list:visible we-title",
290290
run: "click",
291291
},
292292
{
293293
content: "Click on Add new Checkbox",
294-
trigger: 'we-list we-button.o_we_list_add_optional',
294+
trigger: 'we-list:visible we-button.o_we_list_add_optional',
295295
run: "click",
296296
},
297297
{
298298
content: "Change added Option label",
299-
trigger: 'we-list table input:eq(3)',
299+
trigger: 'we-list:visible table input:eq(3)',
300300
run: "edit Wiko Stairway",
301301
}, {
302302
content: "Check the resulting field",
@@ -321,15 +321,15 @@
321321
...addCustomField('selection', 'radio', 'Service', true),
322322
{
323323
content: "Change Option 1 label",
324-
trigger: 'we-list table input:eq(0)',
324+
trigger: 'we-list:visible table input:eq(0)',
325325
run: "edit After-sales Service",
326326
}, {
327327
content: "Change Option 2 label",
328-
trigger: 'we-list table input:eq(1)',
328+
trigger: 'we-list:visible table input:eq(1)',
329329
run: "edit Invoicing Service",
330330
}, {
331331
content: "Change first Option 3 label",
332-
trigger: 'we-list table input:eq(2)',
332+
trigger: 'we-list:visible table input:eq(2)',
333333
run: "edit Development Service",
334334
},
335335
{
@@ -344,7 +344,7 @@
344344
run: "click",
345345
}, {
346346
content: "Change last Option label",
347-
trigger: 'we-list table input:eq(3)',
347+
trigger: 'we-list:visible table input:eq(3)',
348348
run: "edit Management Service",
349349
}, {
350350
content: "Mark the field as not required",
@@ -373,15 +373,15 @@
373373
// Customize custom selection field
374374
{
375375
content: "Change Option 1 Label",
376-
trigger: 'we-list table input:eq(0)',
376+
trigger: 'we-list:visible table input:eq(0)',
377377
run: "edit Germany",
378378
}, {
379379
content: "Change Option 2 Label",
380-
trigger: 'we-list table input:eq(1)',
380+
trigger: 'we-list:visible table input:eq(1)',
381381
run: "edit Belgium",
382382
}, {
383383
content: "Change first Option 3 label",
384-
trigger: 'we-list table input:eq(2)',
384+
trigger: 'we-list:visible table input:eq(2)',
385385
run: "edit France",
386386
},
387387
{
@@ -403,12 +403,12 @@
403403
},
404404
{
405405
content: "Change last Option label",
406-
trigger: 'we-list table input:eq(3)',
406+
trigger: 'we-list:visible table input:eq(3)',
407407
// TODO: Fix code to avoid blur event
408408
run: "edit Canada",
409409
}, {
410410
content: "Remove Germany Option",
411-
trigger: '.o_we_select_remove_option:eq(0)',
411+
trigger: '.o_we_select_remove_option:visible:eq(0)',
412412
run: "click",
413413
},
414414
{
@@ -423,7 +423,7 @@
423423
run: "click",
424424
}, {
425425
content: "Change last option label with a number",
426-
trigger: 'we-list table input:eq(3)',
426+
trigger: 'we-list:visible table input:eq(3)',
427427
run: "edit 44 - UK",
428428
}, {
429429
content: "Check that the input value is the full option value",

0 commit comments

Comments
 (0)