diff options
| author | qsm-odoo <qsm@odoo.com> | 2017-02-06 17:13:27 +0100 |
|---|---|---|
| committer | Aaron Bohy <aab@odoo.com> | 2017-04-11 19:44:38 +0200 |
| commit | c74ab1a146034a4eb99f1ac78743a3765dfa5169 (patch) | |
| tree | 9d9645872f93ea14b5525d5d754869993918e44e /addons/web/static/test/fields/basic_fields_tests.js | |
| parent | 043384d72fafa86a550e5b8545c2e2d520f207fe (diff) | |
[IMP] web: domain & field selector usability (+ design)
+ Document and refactor the files using the new guidelines
* Now only non-technical names are used when building field chains and
domains (technical names can be used in debug mode).
* Better warnings are displayed when using the field and/or domain
selector widgets (also checks domain validity in both domain field
AND domain selector widget).
Also improve design (thanks to original work of @stefanorigano):
- Hovering 'remove' highlights the element that will be removed
- Hovering 'add node' btns shows where the new node will be created
- Restyle debug domain input field (make it similar to a text editor)
- Improve design and layout
- Resize inner fields
Diffstat (limited to 'addons/web/static/test/fields/basic_fields_tests.js')
| -rw-r--r-- | addons/web/static/test/fields/basic_fields_tests.js | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/addons/web/static/test/fields/basic_fields_tests.js b/addons/web/static/test/fields/basic_fields_tests.js index 09d590f40c2..004653ab680 100644 --- a/addons/web/static/test/fields/basic_fields_tests.js +++ b/addons/web/static/test/fields/basic_fields_tests.js @@ -1576,7 +1576,7 @@ QUnit.module('basic_fields', { // Focusing the field selector input should open the field selector // popover - $fieldSelector.find("> input").trigger('focusin'); + $fieldSelector.trigger('focusin'); var $fieldSelectorPopover = $fieldSelector.find(".o_field_selector_popover"); assert.ok($fieldSelectorPopover.is(":visible"), "field selector popover should be visible"); @@ -1605,8 +1605,8 @@ QUnit.module('basic_fields', { // "color" field form.$buttons.find('.o_form_button_save').click(); $domain = form.$(".o_form_field_domain"); - assert.ok($domain.html().indexOf("color") >= 0, - "field selector readonly value should now contain 'color'"); + assert.ok($domain.html().indexOf("Color index") >= 0, + "field selector readonly value should now contain 'Color index'"); form.destroy(); }); @@ -1642,7 +1642,7 @@ QUnit.module('basic_fields', { "there should be a field selector"); // Focusing its input should open the field selector popover - $fieldSelector.find("> input").trigger('focusin'); + $fieldSelector.trigger('focusin'); var $fieldSelectorPopover = $fieldSelector.find(".o_field_selector_popover"); assert.ok($fieldSelectorPopover.is(":visible"), "field selector popover should be visible"); @@ -1667,7 +1667,7 @@ QUnit.module('basic_fields', { // Refocusing the field selector input should open the popover again $fieldSelector = form.$(".o_field_selector"); - $fieldSelector.find("> input").trigger('focusin'); + $fieldSelector.trigger('focusin'); $fieldSelectorPopover = $fieldSelector.find(".o_field_selector_popover"); assert.ok($fieldSelectorPopover.is(":visible"), "field selector popover should be visible"); |
