From 28f910f5bef256ebb17700dea72fed4552e16357 Mon Sep 17 00:00:00 2001 From: "stefanorigano (SRI)" Date: Thu, 24 Feb 2022 10:12:50 +0000 Subject: [PATCH] [REF] *: odoo_ui_icons Following new policies in terms of product design-language, this commit will partially revert 546477d reducing the number of `oi` icons. It will also improve rendering when icons are rendered at our default font-size (13px), avoiding blur effects when icons use thin lines. Since fontAwesome will coexists with `oi` longer than expected, the commit will also review size utility-classes in order to get consistent result when used in conjunction with fa's ones (notably `fa-lg`). Part-of: odoo/odoo#85651 --- .../messaging_menu/messaging_menu.xml | 2 +- .../tests/messaging_menu_tests.js | 2 +- .../rtc_activity_notice.xml | 8 +- .../rtc_call_participant_card.xml | 4 +- .../rtc_controller/rtc_controller.xml | 20 +- .../src/js/views/activity/activity_view.js | 2 +- addons/mail/static/src/xml/systray.xml | 2 +- .../odoo_ui_icons/fonts/odoo_ui_icons.woff | Bin 16716 -> 5156 bytes .../odoo_ui_icons/fonts/odoo_ui_icons.woff2 | Bin 7580 -> 4268 bytes addons/web/static/lib/odoo_ui_icons/style.css | 338 +----------------- .../web/static/src/core/debug/debug_menu.xml | 2 +- .../legacy/js/control_panel/search_utils.js | 8 +- .../legacy/js/views/calendar/calendar_view.js | 2 +- .../src/legacy/js/views/kanban/kanban_view.js | 2 +- .../src/legacy/js/views/list/list_renderer.js | 2 +- .../src/legacy/js/views/list/list_view.js | 2 +- addons/web/static/src/legacy/xml/base.xml | 16 +- .../src/libs/fontawesome/css/font-awesome.css | 7 +- .../search/control_panel/control_panel.xml | 2 +- .../search/favorite_menu/favorite_menu.xml | 2 +- .../src/search/filter_menu/filter_menu.xml | 2 +- .../search/group_by_menu/group_by_menu.xml | 2 +- addons/web/static/src/search/utils/misc.js | 8 +- .../web/static/src/views/graph/graph_view.js | 2 +- .../web/static/src/views/graph/graph_view.xml | 12 +- .../static/src/views/pivot/pivot_renderer.xml | 4 +- .../web/static/src/views/pivot/pivot_view.js | 2 +- .../webclient/burger_menu/burger_menu.scss | 1 - .../src/webclient/burger_menu/burger_menu.xml | 4 +- addons/web/static/src/webclient/icons.scss | 32 +- .../static/src/webclient/navbar/navbar.xml | 2 +- .../switch_company_menu.xml | 2 +- .../tests/core/debug/debug_manager_tests.js | 4 +- .../control_panel/comparison_menu_tests.js | 2 +- .../control_panel/favorite_menu_tests.js | 2 +- .../legacy/control_panel/filter_menu_tests.js | 2 +- .../control_panel/groupby_menu_tests.js | 2 +- .../static/tests/legacy/views/graph_tests.js | 4 +- .../tests/search/comparison_menu_tests.js | 2 +- .../web/static/tests/search/control_panel.js | 6 +- .../tests/search/favorite_menu_tests.js | 4 +- .../static/tests/search/filter_menu_tests.js | 2 +- .../tests/search/group_by_menu_tests.js | 2 +- .../static/tests/views/graph_view_tests.js | 56 +-- .../views/website_navbar_templates.xml | 6 +- 45 files changed, 147 insertions(+), 443 deletions(-) diff --git a/addons/mail/static/src/components/messaging_menu/messaging_menu.xml b/addons/mail/static/src/components/messaging_menu/messaging_menu.xml index b34e1b0a0bb..be110a8881d 100644 --- a/addons/mail/static/src/components/messaging_menu/messaging_menu.xml +++ b/addons/mail/static/src/components/messaging_menu/messaging_menu.xml @@ -4,7 +4,7 @@
- + diff --git a/addons/mail/static/src/components/messaging_menu/tests/messaging_menu_tests.js b/addons/mail/static/src/components/messaging_menu/tests/messaging_menu_tests.js index 04e1cacb313..0e7ebe0b064 100644 --- a/addons/mail/static/src/components/messaging_menu/tests/messaging_menu_tests.js +++ b/addons/mail/static/src/components/messaging_menu/tests/messaging_menu_tests.js @@ -152,7 +152,7 @@ QUnit.test('basic rendering', async function (assert) { "should have icon on clickable element in messaging menu" ); assert.ok( - document.querySelector(`.o_MessagingMenu_icon`).classList.contains('oi-discuss'), + document.querySelector(`.o_MessagingMenu_icon`).classList.contains('fa-comments'), "should have 'comments' icon on clickable element in messaging menu" ); assert.strictEqual( diff --git a/addons/mail/static/src/components/rtc_activity_notice/rtc_activity_notice.xml b/addons/mail/static/src/components/rtc_activity_notice/rtc_activity_notice.xml index f1965be2e45..8ccada1dd1e 100644 --- a/addons/mail/static/src/components/rtc_activity_notice/rtc_activity_notice.xml +++ b/addons/mail/static/src/components/rtc_activity_notice/rtc_activity_notice.xml @@ -9,10 +9,10 @@ Open conference:
-
diff --git a/addons/mail/static/src/components/rtc_call_participant_card/rtc_call_participant_card.xml b/addons/mail/static/src/components/rtc_call_participant_card/rtc_call_participant_card.xml index 0900cd585a1..397f83315e5 100644 --- a/addons/mail/static/src/components/rtc_call_participant_card/rtc_call_participant_card.xml +++ b/addons/mail/static/src/components/rtc_call_participant_card/rtc_call_participant_card.xml @@ -51,12 +51,12 @@
- + - + diff --git a/addons/mail/static/src/components/rtc_controller/rtc_controller.xml b/addons/mail/static/src/components/rtc_controller/rtc_controller.xml index c8a08c6565e..bf26210adcb 100644 --- a/addons/mail/static/src/components/rtc_controller/rtc_controller.xml +++ b/addons/mail/static/src/components/rtc_controller/rtc_controller.xml @@ -16,8 +16,8 @@
@@ -32,8 +32,8 @@
@@ -49,7 +49,7 @@ t-att-title="cameraTitle" t-on-click="rtcController.onClickCamera">
- +
Stop screen sharing @@ -63,7 +63,7 @@ t-att-title="displayTitle" t-on-click="rtcController.onClickScreen">
- +
@@ -73,7 +73,7 @@ t-att-class="{ 'o-isSmall': rtcController.isSmall }" >
- +
@@ -89,7 +89,7 @@ t-att-disabled="rtcController.callViewer.threadView.thread.hasPendingRtcRequest" t-on-click="rtcController.onClickToggleVideoCall">
- +
@@ -102,7 +102,7 @@ t-att-disabled="rtcController.callViewer.threadView.thread.hasPendingRtcRequest" t-on-click="rtcController.onClickRejectCall">
- +
@@ -115,7 +115,7 @@ t-att-title="phoneTitle" t-on-click="rtcController.onClickToggleAudioCall">
- +
diff --git a/addons/mail/static/src/js/views/activity/activity_view.js b/addons/mail/static/src/js/views/activity/activity_view.js index 938c7dd5d4e..c631f17df8a 100644 --- a/addons/mail/static/src/js/views/activity/activity_view.js +++ b/addons/mail/static/src/js/views/activity/activity_view.js @@ -14,7 +14,7 @@ const _lt = core._lt; const ActivityView = BasicView.extend({ accesskey: "a", display_name: _lt('Activity'), - icon: 'oi-event--schedule', + icon: 'fa fa-clock-o', config: _.extend({}, BasicView.prototype.config, { Controller: ActivityController, Model: ActivityModel, diff --git a/addons/mail/static/src/xml/systray.xml b/addons/mail/static/src/xml/systray.xml index 0ece9b059c4..2bb7df7440d 100644 --- a/addons/mail/static/src/xml/systray.xml +++ b/addons/mail/static/src/xml/systray.xml @@ -51,7 +51,7 @@