From 4ac9a556a873b730e5a881dc66b7b8750125f772 Mon Sep 17 00:00:00 2001 From: Wang Jiyuan <59059173+EurFelux@users.noreply.github.com> Date: Tue, 10 Jun 2025 18:13:11 +0800 Subject: [PATCH] Fix: bubble-style unnecessary menu background (Plan D) (#7026) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * fix: bubble-style unnecessary menu background * fix: show divider in message only in plain mode * fix: bubble user message style in dark mode * fix: action button hover style * refactor: The rendering position of the message menbar is determined by the settings * fix: bubble style assistant message token usage left align * fix: bubble style * fix: bubble style * fix: text color and bubble edit * fix: bubble editing * fix: bubble editing * fix: bubble editor * fix: editor width * fix: remove redundant tokens usage * fix: not unified token font size and color * fix: unexpected display behavior in plain mode * fix: info style * fix: bubble style * fix: Style fixes for better compatibility * fix: bubble style * fix: Move the menu of the last message to the outside * fix: bubble style * fix: why this happened? * feat: add description for messages divider in settings * fix: 谁想出来的上下margin不一样还是神秘数字 * fix: new context style --- src/renderer/src/assets/styles/container.scss | 10 -- src/renderer/src/assets/styles/index.scss | 29 +++- .../src/components/ContextMenu/index.tsx | 5 +- src/renderer/src/i18n/locales/en-us.json | 1 + src/renderer/src/i18n/locales/ja-jp.json | 1 + src/renderer/src/i18n/locales/ru-ru.json | 1 + src/renderer/src/i18n/locales/zh-cn.json | 1 + src/renderer/src/i18n/locales/zh-tw.json | 1 + src/renderer/src/i18n/translate/el-gr.json | 1 + src/renderer/src/i18n/translate/es-es.json | 1 + src/renderer/src/i18n/translate/fr-fr.json | 1 + src/renderer/src/i18n/translate/pt-pt.json | 1 + .../src/pages/home/Markdown/Markdown.tsx | 12 +- .../pages/home/Messages/Blocks/ErrorBlock.tsx | 2 +- .../home/Messages/Blocks/MainTextBlock.tsx | 2 +- .../src/pages/home/Messages/Blocks/index.tsx | 1 + .../src/pages/home/Messages/Message.tsx | 137 ++++++++++++------ .../pages/home/Messages/MessageContent.tsx | 2 +- .../src/pages/home/Messages/MessageGroup.tsx | 1 + .../src/pages/home/Messages/MessageHeader.tsx | 29 +++- .../pages/home/Messages/MessageMenubar.tsx | 45 ++++-- .../src/pages/home/Messages/MessageTokens.tsx | 13 +- .../src/pages/home/Tabs/SettingsTab.tsx | 7 +- 23 files changed, 201 insertions(+), 103 deletions(-) diff --git a/src/renderer/src/assets/styles/container.scss b/src/renderer/src/assets/styles/container.scss index ab5e8a7de8..8be4027981 100644 --- a/src/renderer/src/assets/styles/container.scss +++ b/src/renderer/src/assets/styles/container.scss @@ -4,13 +4,3 @@ border-top-left-radius: 10px; border-left: 0.5px solid var(--color-border); } - -.group-container { - .context-menu-container { - width: 100%; - } -} - -.context-menu-container { - max-width: 100%; -} diff --git a/src/renderer/src/assets/styles/index.scss b/src/renderer/src/assets/styles/index.scss index 91778848bb..b91b3c3a54 100644 --- a/src/renderer/src/assets/styles/index.scss +++ b/src/renderer/src/assets/styles/index.scss @@ -129,22 +129,29 @@ ul { .message-content-container { margin: 5px 0; border-radius: 8px; - padding: 10px 15px 0 15px; + padding: 0.5rem 1rem; } + + .block-wrapper { + display: flow-root; + } + + .message-content-container > *:last-child { + margin-bottom: 0; + } + .message-thought-container { margin-top: 8px; } + .message-user { color: var(--chat-text-user); - .markdown, - .anticon, - .iconfont, - .lucide, - .message-tokens { + .message-content-container-user .anticon { color: var(--chat-text-user) !important; } - .message-action-button:hover { - background-color: var(--color-white-soft); + + .markdown { + color: var(--chat-text-user); } } .group-grid-container.horizontal, @@ -165,6 +172,12 @@ ul { code { color: var(--color-text); } + .markdown { + display: flow-root; + *:last-child { + margin-bottom: 0; + } + } } .lucide { diff --git a/src/renderer/src/components/ContextMenu/index.tsx b/src/renderer/src/components/ContextMenu/index.tsx index bb3136067c..195fcb2a38 100644 --- a/src/renderer/src/components/ContextMenu/index.tsx +++ b/src/renderer/src/components/ContextMenu/index.tsx @@ -6,9 +6,10 @@ import styled from 'styled-components' interface ContextMenuProps { children: React.ReactNode onContextMenu?: (e: React.MouseEvent) => void + style?: React.CSSProperties } -const ContextMenu: React.FC = ({ children, onContextMenu }) => { +const ContextMenu: React.FC = ({ children, onContextMenu, style }) => { const { t } = useTranslation() const [contextMenuPosition, setContextMenuPosition] = useState<{ x: number; y: number } | null>(null) const [selectedText, setSelectedText] = useState('') @@ -66,7 +67,7 @@ const ContextMenu: React.FC = ({ children, onContextMenu }) => ] return ( - + {contextMenuPosition && ( = ({ block }) => { } as Partial }, [onSaveCodeBlock]) + if (messageContent.includes('