diff options
| author | Khoi Nguyen <bng@odoo.com> | 2017-10-02 15:59:20 +0200 |
|---|---|---|
| committer | Khoi Nguyen <bng@odoo.com> | 2017-10-02 17:34:07 +0200 |
| commit | 59b55e22218d395083f48db58eaa322919f4fbc9 (patch) | |
| tree | e155f2e0257d75ebe2b15b31f4ad351db97811a2 /addons/web/static/src/js/views/basic/basic_model.js | |
| parent | d236da5b4e695a7dc54a98b2577c89b25b0a1fd2 (diff) | |
[FIX] web: fix TAB key in editable list views
Before this commit, two variables were used to store the current
position in a list view: currentCol and currentRow. This made it
difficult to handle incremental moves (eg: tab, shift+tab) inside a
row, since we have to ignore all buttons. Previously, this was done by
converting 'currentCol' to 'currentFieldIndex', but the conversion
between the two variables was incorrect and does not appear to be
that easy to fix. As an example, it was not possible to TAB-navigate to
the last column of the components list when creating a BoM.
This commit removes all mentions of 'currentCol' and uses
'currentFieldIndex' instead, removing the need for conversions. The TAB
navigation code is simpler as a result, and should now be correct.
Diffstat (limited to 'addons/web/static/src/js/views/basic/basic_model.js')
0 files changed, 0 insertions, 0 deletions