Skip to content

feat: Sakura Macaron theme + Modern UI style overrides - #83

Closed
player-Muteki wants to merge 4 commits into
otty-shell:mainfrom
player-Muteki:main
Closed

feat: Sakura Macaron theme + Modern UI style overrides#83
player-Muteki wants to merge 4 commits into
otty-shell:mainfrom
player-Muteki:main

Conversation

@player-Muteki

Copy link
Copy Markdown

Port of the user's VS Code Sakura Macaron light theme to otty, plus Modern UI-style surface design:

  • 32-slot palette (adds sidebar / activity_bar / accent tokens)
  • Configurable font size via settings.json
  • System SSD decorations (decorations: true), removed custom header
  • Tiered corner radius (control 4px / inner 6px / outer 8px)
  • Editor area as hero card (1px border + 12% foreground mix + 8px radius)
  • Transparent pill tabs with adaptive highlight
  • color-mix style adaptive blending (mix_color)
  • Rose accent for all UI emphasis; teal/green reserved for terminal ANSI

- 主题:亮色樱花粉(32 槽调色板,新增 sidebar/activity_bar/accent token)
- 字号:支持 settings.json font.size 配置
- 窗口:系统 SSD 装饰(decorations: true),移除内部自绘顶栏
- Modern UI(对齐 VS Code styleOverrides 范式):
  - 分级圆角 token(control 4px / inner 6px / outer 8px)
  - 编辑器区域 = 主角卡片(1px 边框 + 前景12%混合 + 8px 圆角 + clip)
  - 标签页 = 透明药丸 + 前景混合高亮
  - color-mix 自适应混合(mix_color)
  - UI 强调统一玫瑰红 accent,蓝绿只留终端 ANSI
@kemokempo

Copy link
Copy Markdown
Contributor
image

Hi, thanks for your PR but our UI is broken by this PR

  1. Custom window frame was changed to system (it's not suitable for this project)
  2. Theme changes the default otty dark theme instead of creating new one
  3. There're some UI bugs.

- 将 SIDEBAR_RAIL_WIDTH 移入 layout.rs 作为唯一来源,删除 sidebar/constants.rs
- workspace 关闭但侧边栏可见时,editor_width 改为 split_width(扣除 PaneGrid 内部 gap)
- 清理 tab_bar.rs 死代码,为 invalid hex 配色补充守卫测试
- 补充 workspace 关闭/侧边隐藏的几何测试
@player-Muteki player-Muteki closed this by deleting the head repository Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants