From 4e204ed2233ccd36b2eb7d3e56bfa741b9cb3e14 Mon Sep 17 00:00:00 2001 From: Aaron Bohy Date: Thu, 8 Mar 2018 14:57:43 +0100 Subject: [IMP] web: restore scroll position This rev. improves the way the scroll position is restored when coming back to a view (e.g. open a kanban view, scroll, click on a record to open it in a form view, go back to kanban using the breadcrumbs): - generalize the scroll position handling from web_enterprise to web to make it available in the community webclient as well - also restore left scroll position, not only top (especially useful for grouped Kanban views with a lot of columns) - handle the case where the scrolling area is a sub element of the view (e.g. the grouped Kanban view in mobile). Task 34037 Co-authored-by: Mohammed Shekha Co-authored-by: Ankit Sathvara --- addons/web/static/src/js/views/basic/basic_controller.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'addons/web/static/src/js/views/basic/basic_controller.js') diff --git a/addons/web/static/src/js/views/basic/basic_controller.js b/addons/web/static/src/js/views/basic/basic_controller.js index 05031ca2463..263412fd2a2 100644 --- a/addons/web/static/src/js/views/basic/basic_controller.js +++ b/addons/web/static/src/js/views/basic/basic_controller.js @@ -144,7 +144,7 @@ var BasicController = AbstractController.extend(FieldManagerMixin, { .then(function () { // Reset the scroll position to the top on page changed only if (!limitChanged) { - self.trigger_up('scrollTo', {offset: 0}); + self.trigger_up('scrollTo', {top: 0}); } }) .then(this.pager.enable.bind(this.pager)); -- cgit v1.2.3