Skip to content

UI 布局修复:标题栏压缩、滚动条对齐、composer 精简#3589

Open
ttmouse wants to merge 2 commits into
esengine:main-v2from
ttmouse:pr/ui-fixes
Open

UI 布局修复:标题栏压缩、滚动条对齐、composer 精简#3589
ttmouse wants to merge 2 commits into
esengine:main-v2from
ttmouse:pr/ui-fixes

Conversation

@ttmouse

@ttmouse ttmouse commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

UI 布局修复

标题栏

  • 去掉重复的项目名前缀
  • 字号 18px→15px,padding 缩小

滚动条对齐修复

  • 改用 overflow-y:overlay 悬浮不占布局空间
  • scrollbar-gutter:stable 预占滚动条空间
  • 历史消息与输入框完美左对齐

Composer 精简

  • 移除 caret 符号
  • send 按钮移到 meta bar 作为多状态动作按钮
  • 移除 process-card__body 背景

其他

  • 隐藏 thinking label,用 BrainCircuit 图标替代
  • 右侧分割线背景匹配 dock,移除 footer 上边框
  • CSS 变量统一,消除未定义变量导致的显示问题
  • unread dot 不可见问题修复

ttmouse added 2 commits June 8, 2026 20:46
- 标题栏:去掉重复项目名前缀,字号 18px→15px,padding 缩小
- 滚动条:改用 overflow-y:overlay 悬浮不占位,scrollbar-gutter:stable
- Composer:移除 caret 符号和 slim input area,send 按钮移入 meta bar
- 对话区:composer-card 负 margin 修复左对齐,QuestionJumpBar 固定视口中央
- 其他:隐藏 thinking label,移除 process-card 背景,CSS 变量统一
- gitignore:添加 .alma-snapshots
@github-actions github-actions Bot added v2 Go rewrite (1.x) — main-v2 branch, active development desktop Wails desktop app (desktop/**) and removed v2 Go rewrite (1.x) — main-v2 branch, active development labels Jun 8, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

/* Disable cross-document view transitions — the webview has no navigation. */

P1 Badge Restore the command palette rules

The deleted .palette* and .drawer-backdrop:has(.palette) rules are still the only styles for CommandPalette.tsx (confirmed with repo-wide rg for palette__/.palette). Opening the command palette now falls back to the drawer backdrop's justify-content: flex-end and unstyled native buttons/inputs, so the palette appears as an unformatted side-drawer instead of the centered command dialog.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

{!empty && showQuestionNav && (
<QuestionJumpBar questions={questions} onJump={handleJumpToQuestion} />
)}
<div className="transcript-wrap">

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Keep the transcript wrapper constrained

When there are enough messages to overflow, this new wrapper is the direct child of .main but has no CSS such as flex: 1/min-height: 0/height: 100%; .transcript's height: 100% is now relative to an unconstrained wrapper, so the wrapper expands to content while .main has overflow: hidden. That leaves the scrollRef element without the intended viewport-sized scrollbar and can make older/newer messages inaccessible behind the footer.

Useful? React with 👍 / 👎.

Comment on lines +1666 to +1668
<button
className="composer-meta__action-btn composer-meta__action-btn--send"
onClick={submit}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Restore styling for the moved action button

The send/stop button was moved from .composer__btn--send to the new .composer-meta__action-btn* classes, but a repo-wide search shows no CSS rules for those classes. In the normal non-running composer this makes the primary send control fall back to native button styling/metrics instead of the fixed square primary action, which is a visible regression in the composer layout.

Useful? React with 👍 / 👎.

background: var(--sidebar-hover);
color: var(--fg);
}
.workspace-tree__chev {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Restore changed-history styling

This deletion removes the only .workspace-git-history* styles while WorkspacePanel.tsx still renders those classes in the changed-files history view. When users open the changed/history pane, commit headers and file rows now revert to native button layout and the expanded details lose their spacing/borders, which makes the git history UI visibly broken and harder to use.

Useful? React with 👍 / 👎.

Comment on lines +156 to +157
{(turnTokens ?? 0) > 0 && (
<span className="statusbar__tokens">↓ {fmtTokens(turnTokens ?? 0)} {t("status.tokens")}</span>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Hide turn tokens after the turn completes

After any turn that reports usage, turnTokens remains at the last turn's value until the next turn starts, but this condition renders it whenever it is greater than zero. Previously the token count was part of the running-only activity text, so the status bar now shows a stale per-turn token total after completion, which is misleading while the app is idle.

Useful? React with 👍 / 👎.

Comment on lines 1661 to 1662
ready={state.meta?.ready === true}
turnStartAt={state.turnStartAt}
turnTokens={state.turnTokens}
retry={state.retry}
workspaceRefreshSignal={projectRevision}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Preserve retry attempt status

The controller still stores backend retrying events in state.retry, but these removed props were the only consumer of that state (confirmed with repo-wide rg for state.retry/retry). During API retries the UI now only shows the generic spinner, so users lose the explicit retry attempt/max indication and cannot tell that the app is retrying rather than simply hung.

Useful? React with 👍 / 👎.

@CVEngineer66

Copy link
Copy Markdown
Contributor

给一个效果图看看

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Wails desktop app (desktop/**)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants