Skip to content

docs(site): emoji-led changelog tags with unified pill styling - #10

Merged
Div627 merged 1 commit into
mainfrom
docs/changelog-emoji-tags
Jul 6, 2026
Merged

docs(site): emoji-led changelog tags with unified pill styling#10
Div627 merged 1 commit into
mainfrom
docs/changelog-emoji-tags

Conversation

@Div627

@Div627 Div627 commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

🤔 This is a ...

  • 🆕 New feature
  • 🐞 Bug fix
  • 📝 Site / documentation improvement
  • 📽️ Demo improvement
  • 💄 Component style improvement
  • 🤖 TypeScript definition improvement
  • 📦 Bundle size optimization
  • ⚡️ Performance optimization
  • ⭐️ Feature enhancement
  • 🌐 Internationalization
  • 🛠 Refactoring
  • 🎨 Code style optimization
  • ✅ Test Case
  • 🔀 Branch merge
  • ⏩ Workflow
  • ⌨️ Accessibility improvement
  • ❓ Other (about what?)

🔗 Related Issues

Follow-up to #9. The changelog type tags looked inconsistent (four divergent colored chips). This change was part of #9's branch but did not land in the merge, so it is re-submitted here.

💡 Background and Solution

The changelog type tags used four divergent colored chips (dark / blue / gray / outline), which read as cluttered. Replace them with a single flat pill led by a semantic emoji per type — 💥 breaking, 🆕 feature, 🐛 fix, ⚡️ perf. The emoji carries the color so the chip stays uniform, on-brand with the whiteboard palette (no second hue). The tag column is widened to fit emoji + label; the emoji is aria-hidden and the text label is preserved.

TYPE_EMOJI in ChangelogTimeline.tsx maps each ChangeType to its emoji, decoupled from locale copy (copy.types). npm run check:home and npm run build (docs-site) both pass.

📝 Change Log

Language Changelog
🇺🇸 English --
🇨🇳 Chinese --

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 新功能
    • 更新了更新日志时间线中的标签展示方式,改为更直观的图标 + 标签样式,便于快速识别不同类型的变更。
  • 界面优化
    • 调整了时间线左侧列宽与标签间距,整体排版更清晰。
    • 优化了移动端标签对齐方式,减少拥挤感。
    • 统一了标签的外观、颜色和圆角风格,视觉更一致。

Replace the four divergent colored chips (dark/blue/gray/outline) with a
single flat pill led by a semantic emoji per type (💥 breaking, 🆕
feature, 🐛 fix, ⚡️ perf). The emoji carries the color so the chip stays
uniform — on-brand with the whiteboard palette (no second hue). Widen the
tag column to fit emoji + label.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: 711a1947-61ad-4a87-90ab-7b2829edadbc

📥 Commits

Reviewing files that changed from the base of the PR and between f237ee7 and 0ffe098.

📒 Files selected for processing (2)
  • docs-site/public/site.css
  • docs-site/src/demos/changelog/ChangelogTimeline.tsx

📝 Walkthrough

Walkthrough

变更日志标签样式从多种类型颜色 modifier 改为统一扁平 pill 样式,并新增按类型显示的 emoji 前导元素。CSS 调整了网格列宽与移动端对齐规则,组件新增 TYPE_EMOJI 映射表并渲染对应 emoji 节点。

Changes

变更日志标签样式改造

Layer / File(s) Summary
标签CSS样式重写
docs-site/public/site.css
.xmd-cl-tag 从多种 --breaking/--feature/--fix/--perf 分类型配色改为统一 inline-flex pill 样式,网格左列宽度由 68px 调整为 92px,移动端约束由 min-width: 0 改为 justify-self: start
组件中Emoji数据与渲染
docs-site/src/demos/changelog/ChangelogTimeline.tsx
新增 TYPE_EMOJI 常量按 ChangeType 映射对应 emoji,标签内容新增 .xmd-cl-tag-emoji 节点渲染该 emoji。

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

小兔子改标签,涂上彩色小圆脸 🐰
一颗 emoji 领头走,样式统一更清晰
68 变 92,格子宽一寸
移动端也贴心,靠左站得稳
蹦蹦跳跳交差啦!🥕


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands.

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request updates the changelog timeline tags to use a uniform, emoji-led pill design with a flat background color instead of type-specific hues. Feedback suggests increasing the tag column width from 92px to 96px to prevent potential text overflow with wider fallback fonts, and conditionally rendering the emoji span to avoid layout spacing issues if an emoji is missing.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread docs-site/public/site.css
.markdown .xmd-cl-change {
display: grid;
grid-template-columns: 68px 1fr;
grid-template-columns: 92px 1fr;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The fixed width of 92px for the tag column is extremely tight for English labels like "Breaking" or "Improved" when combined with the emoji, padding, and gap (e.g., 18px padding + 12.5px emoji + 5px gap + ~55px text width ≈ 90.5px). On systems with slightly wider fallback fonts, this will exceed 92px and cause the tag to overflow the grid column. Increasing this to 96px or 100px provides a safe margin of error.

Suggested change
grid-template-columns: 92px 1fr;
grid-template-columns: 96px 1fr;

Comment on lines +93 to +95
<span className="xmd-cl-tag-emoji" aria-hidden="true">
{TYPE_EMOJI[item.type]}
</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.

medium

If TYPE_EMOJI[item.type] is ever undefined or missing for a custom or unexpected change type, rendering an empty span will still trigger the CSS gap: 5px on the parent flex container, causing the text label to appear off-center. Conditionally rendering the emoji span only when the emoji is present avoids this layout issue.

                        {TYPE_EMOJI[item.type] && (
                          <span className="xmd-cl-tag-emoji" aria-hidden="true">
                            {TYPE_EMOJI[item.type]}
                          </span>
                        )}

@Div627
Div627 merged commit 154f80a into main Jul 6, 2026
8 checks passed
@Div627
Div627 deleted the docs/changelog-emoji-tags branch July 6, 2026 00:42
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.

1 participant