From 74b9986b30b1476a7bb0d00e7d29996dbd965c17 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexandre=20K=C3=BChn?= Date: Tue, 4 Jul 2023 16:19:32 +0000 Subject: [PATCH] [FIX] mail: improve horizontal spacing between msg avatar and bubble MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Before this commit, horizontal spacing in message between avatar author and bubble was too big. This makes reading messages slightly hard, because this small gap gives the impression that the message and avatar are 2 distinct part of UI. This commit fixes the issue by reducing the size of message sidebar, so that avatar and bubble are much closer together. Also the message avatar was too close to the border of chat windows. This commit fixes the issue by slightly increasing the horizontal padding of messages in chat window (`px-1` => `px-2`). closes odoo/odoo#127341 X-original-commit: 226c6588a89bf21e8c0a47010bffb8c59cbb4e37 Signed-off-by: Sébastien Theys (seb) Signed-off-by: Alexandre Kühn (aku) --- addons/mail/static/src/core/common/message.js | 2 +- addons/mail/static/src/core/common/primary_variables.scss | 2 +- addons/mail/static/src/scss/variables/primary_variables.scss | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/addons/mail/static/src/core/common/message.js b/addons/mail/static/src/core/common/message.js index fe1aa6a12d3..f457546903a 100644 --- a/addons/mail/static/src/core/common/message.js +++ b/addons/mail/static/src/core/common/message.js @@ -159,7 +159,7 @@ export class Message extends Component { "py-1": !this.props.squashed, "mt-2": !this.props.squashed && this.props.thread, "px-3": !this.props.isInChatWindow, - "px-1": this.props.isInChatWindow, + "px-2": this.props.isInChatWindow, "opacity-50": this.props.messageToReplyTo?.isNotSelected( this.props.thread, this.props.message diff --git a/addons/mail/static/src/core/common/primary_variables.scss b/addons/mail/static/src/core/common/primary_variables.scss index cfaa3aa79ef..9a92623d913 100644 --- a/addons/mail/static/src/core/common/primary_variables.scss +++ b/addons/mail/static/src/core/common/primary_variables.scss @@ -9,7 +9,7 @@ $o-mail-LinkPreview-width: 320px !default; $o-mail-LinkPreview-height: 240px !default; $o-mail-LinkPreviewCard-height: 80px !default; -$o-mail-Message-sidebarWidth: 48px !default; +$o-mail-Message-sidebarWidth: 42px !default; $o-mail-NavigableList-zIndex: 11; $o-mail-Chatter-minWidth: 530px !default; $o-mail-Discuss-inspector: 300px !default; diff --git a/addons/mail/static/src/scss/variables/primary_variables.scss b/addons/mail/static/src/scss/variables/primary_variables.scss index e8bbff64643..0dffc7ddce0 100644 --- a/addons/mail/static/src/scss/variables/primary_variables.scss +++ b/addons/mail/static/src/scss/variables/primary_variables.scss @@ -9,7 +9,7 @@ $o-mail-partner-status-icon-size: 1.2em !default; $o-RoundedRectangle-small: .2rem !default; $o-RoundedRectangle-large: 3 * $o-RoundedRectangle-small !default; -$o-mail-Message-sidebarWidth: 48px !default; +$o-mail-Message-sidebarWidth: 42px !default; $o-mail-composer-text-input-height: 40px !default; $o-mail-LinkPreview-width: 320px !default;