-
- Downloads
[REF] web: move ControlPanel inside controllers/actions
This branch introduces a large-scale reorganization of the component tree generated by the web client. The short version is that now, the control panel is a child of the view controller and no longer a sibling. Graphically (and simplified), we go from this: webClient / | \ ... ... actionManager / \ controlPanel viewController / \ to this: webClient / | \ ... ... actionManager | viewController / | \ ControlPanelController / \ CPRenderer CPModel The motivation is that this work moves the code where it should be. Before this commit, it was kind of weird to have code in the controllers to render buttons outside of their root node (in renderButtons). Also, the action manager had to take care of coordinating search view states between view transitions. So, this work simplifies the code. It also makes it easier to extend. We see day after day that Odoo needs to take care of more complex UI needs, and in many cases, these needs were quite difficult to implement (we prefer spaghettis in our plates, not in our code). The changes in this branch should open the way to implement these features. For example, - it will now be easy to add the possibility of views (for example, the search view or a new ControlPanel view) to add custom buttons (of type action or object) in the control panel. - Another need will be to serialize/ restore the state of the search view across action boundaries (needed by the dashboard). - Another example is the possibility for views to customize easily the presence/absence of sub menus (filters/groupbys/favorites/ time range/...) - Another need is an easier way for views/client action to customize their control panel. This commit also contains a large rewrite of the search view (so it is more inline with our architecture and easier to maintain). Part of task 1893568 Co-authored-by:Aaron Bohy <aab@odoo.com> Co-authored-by:
Mathieu Duckerts-Antoine <dam@odoo.com> Co-authored-by:
Martin Geubelle <mge@odoo.com> -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr closes odoo/odoo#29106
Showing
- addons/account/static/src/js/reconciliation/reconciliation_action.js 13 additions, 18 deletions...unt/static/src/js/reconciliation/reconciliation_action.js
- addons/account/static/tests/reconciliation_tests.js 1 addition, 1 deletionaddons/account/static/tests/reconciliation_tests.js
- addons/base_import/static/src/js/import_action.js 8 additions, 7 deletionsaddons/base_import/static/src/js/import_action.js
- addons/board/static/src/js/add_to_board_menu.js 205 additions, 0 deletionsaddons/board/static/src/js/add_to_board_menu.js
- addons/board/static/src/js/board_view.js 9 additions, 28 deletionsaddons/board/static/src/js/board_view.js
- addons/board/static/src/js/favorite_menu.js 0 additions, 136 deletionsaddons/board/static/src/js/favorite_menu.js
- addons/board/static/src/scss/dashboard.scss 1 addition, 1 deletionaddons/board/static/src/scss/dashboard.scss
- addons/board/static/src/xml/board.xml 10 additions, 0 deletionsaddons/board/static/src/xml/board.xml
- addons/board/static/tests/dashboard_tests.js 57 additions, 85 deletionsaddons/board/static/tests/dashboard_tests.js
- addons/board/views/board_templates.xml 1 addition, 1 deletionaddons/board/views/board_templates.xml
- addons/google_spreadsheet/static/src/js/add_to_google_spreadsheet_menu.js 102 additions, 0 deletions...readsheet/static/src/js/add_to_google_spreadsheet_menu.js
- addons/google_spreadsheet/static/src/js/search.js 0 additions, 48 deletionsaddons/google_spreadsheet/static/src/js/search.js
- addons/google_spreadsheet/static/src/xml/addtospreadsheet.xml 4 additions, 4 deletions...ns/google_spreadsheet/static/src/xml/addtospreadsheet.xml
- addons/google_spreadsheet/views/google_spreadsheet_templates.xml 1 addition, 1 deletion...google_spreadsheet/views/google_spreadsheet_templates.xml
- addons/hr_attendance/static/src/js/greeting_message.js 2 additions, 1 deletionaddons/hr_attendance/static/src/js/greeting_message.js
- addons/hr_attendance/static/src/js/my_attendances.js 2 additions, 5 deletionsaddons/hr_attendance/static/src/js/my_attendances.js
- addons/hr_attendance/static/tests/hr_attendance_tests.js 4 additions, 4 deletionsaddons/hr_attendance/static/tests/hr_attendance_tests.js
- addons/mail/static/src/js/discuss.js 17 additions, 51 deletionsaddons/mail/static/src/js/discuss.js
- addons/mail/static/src/js/views/activity/activity_view.js 1 addition, 1 deletionaddons/mail/static/src/js/views/activity/activity_view.js
- addons/mail/static/tests/activity_tests.js 5 additions, 7 deletionsaddons/mail/static/tests/activity_tests.js
Loading
Please register or sign in to comment