chore: redesign GitHub org profile README#52
Merged
yuanyuanxin merged 8 commits intomainfrom Mar 13, 2026
Merged
Conversation
- Replace shields.io badge wall with flat brand-colored badges - Add YouTube video thumbnails with play button overlay for Hand and Glove - Update product descriptions based on actual codebase capabilities - Simplify layout: remove repo list, streamline product cards - Split intro text into two lines for better readability Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
Walkthrough重构 Changes
Sequence Diagram(s)（本次更改为文档重构，未引入跨组件控制流，故无序列图） 预计代码审查工作量🎯 3 (Moderate) | ⏱️ ~20 minutes 可能相关的 PR
建议审查者
诗
✅ Pre-merge checks override appliedThe pre-merge checks have been overridden successfully. You can now proceed with the merge. Overridden by ❌ Failed checks (1 error)
✅ Passed checks (6 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@profile/README.md`:
- Around line 20-53: Remove the fixed two-column <table> layout (the <table> and
the <td align="center" width="50%"> cells) and replace it with a responsive
stackable layout so narrow screens render a single column; specifically, stop
using width="50%" on the <td> elements and convert the cards (Wuji Hand, Wuji
Glove, Wuji Studio, Wuji SDK) into vertically stacked blocks (e.g., simple <div>
card elements or a flex/grid container with CSS that switches from two-columns
to one-column via a media query or responsive utility classes) so mobile devices
automatically display a single column without horizontal scrolling.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 3bb3d5fd-6387-44f2-bf94-7a8d9f79fe5b
⛔ Files ignored due to path filters (2)
images/wuji-glove-video.jpgis excluded by!**/*.jpgimages/wuji-hand-video.jpgis excluded by!**/*.jpg
📒 Files selected for processing (1)
profile/README.md
Shadow2223
previously approved these changes
Mar 12, 2026
Add categorized repo listing (SDK, Tools, Simulation, Teleoperation, Hardware, Education) below Product Ecosystem cards. Each repo shows an inline release version badge linked to its latest release page. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
yuanyuanxin
dismissed stale reviews from Shadow2223 and coderabbitai[bot]
via
a0dce3f
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
profile/README.md (1)
42-42: 固定高度可能影响内容灵活性。第二行的单元格使用了
height="120"固定高度。如果未来产品描述内容增加或需要支持其他语言（内容长度可能不同），固定高度可能导致内容溢出或布局不协调。建议移除固定高度属性，让内容自适应。♻️ 建议的改进
- <td align="center" width="50%" height="120"> + <td align="center" width="50%"> <h3>Wuji Studio</h3> Desktop app for device management, data visualization, and firmware upgrade<br><br> <a href="https://docs.wuji.tech/docs/en/wuji-studio/latest">Docs →</a> </td> - <td align="center" width="50%" height="120"> + <td align="center" width="50%"> <h3>Wuji SDK</h3> Python SDK for sensor streaming, hand pose computation, and recording<br><br> <a href="https://docs.wuji.tech/docs/en/wuji-sdk/latest">Docs →</a>Also applies to: 47-47
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@profile/README.md` at line 42, The README's table cell uses a fixed height attribute (the td element with align="center" width="50%" height="120"), which can cause overflow or layout issues when content length varies; remove the height="120" attribute from that td (and the similar td instance) and instead allow the cell to size naturally or apply a responsive CSS class/min-height (e.g., use a className like .profile-cell with min-height or flexible layout rules) so content can grow and support multi-language text.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@profile/README.md`:
- Line 42: The README's table cell uses a fixed height attribute (the td element
with align="center" width="50%" height="120"), which can cause overflow or
layout issues when content length varies; remove the height="120" attribute from
that td (and the similar td instance) and instead allow the cell to size
naturally or apply a responsive CSS class/min-height (e.g., use a className like
.profile-cell with min-height or flexible layout rules) so content can grow and
support multi-language text.
Replace separate repo link + version badge with single combined badge (repo name + version in one shield). Add tooltip on hover. Keep inline description text after badge separated by hyphen. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
profile/README.md (1)
42-51: 固定高度可能导致内容溢出。Lines 42 和 47 的
height="120"固定高度在产品描述文本较长或窄屏环境下可能导致内容被裁剪。建议移除height属性，让单元格高度自适应内容。♻️ 建议移除固定高度
- <td align="center" width="50%" height="120"> + <td align="center" width="50%"> <h3>Wuji Studio</h3> Desktop app for device management, data visualization, and firmware upgrade<br><br> <a href="https://docs.wuji.tech/docs/en/wuji-studio/latest">Docs →</a> </td> - <td align="center" width="50%" height="120"> + <td align="center" width="50%"> <h3>Wuji SDK</h3> Python SDK for sensor streaming, hand pose computation, and recording<br><br> <a href="https://docs.wuji.tech/docs/en/wuji-sdk/latest">Docs →</a>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@profile/README.md` around lines 42 - 51, The two table cells with fixed height attributes (the <td> containing "Wuji Studio" and the <td> containing "Wuji SDK") use height="120", which can cause content clipping on narrow screens; remove the height="120" attributes from those <td> elements so the cell height becomes content-driven (auto) and the descriptions can wrap without being cut off.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@profile/README.md`:
- Around line 42-51: The two table cells with fixed height attributes (the <td>
containing "Wuji Studio" and the <td> containing "Wuji SDK") use height="120",
which can cause content clipping on narrow screens; remove the height="120"
attributes from those <td> elements so the cell height becomes content-driven
(auto) and the descriptions can wrap without being cut off.
Separate badge and description into individual columns with rowspan for category grouping, improving readability. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
GitHub strips CSS classes and <style> tags. Use align/valign attributes directly on <td> elements for category and badge column centering. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
profile/README.md (1)
42-47: 建议移除固定高度属性以避免潜在的文本溢出。Line 42 和 Line 47 的
height="120"固定了单元格高度。如果未来产品描述变长、添加多语言内容或在某些浏览器中渲染，可能导致文本溢出或被截断。建议移除height属性，让单元格高度自适应内容。♻️ 建议的调整
- <td align="center" width="50%" height="120"> + <td align="center" width="50%"> <h3>Wuji Studio</h3> Desktop app for device management, data visualization, and firmware upgrade<br><br> <a href="https://docs.wuji.tech/docs/en/wuji-studio/latest">Docs →</a> </td> - <td align="center" width="50%" height="120"> + <td align="center" width="50%"> <h3>Wuji SDK</h3> Python SDK for sensor streaming, hand pose computation, and recording<br><br> <a href="https://docs.wuji.tech/docs/en/wuji-sdk/latest">Docs →</a>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@profile/README.md` around lines 42 - 47, 在 profile/README.md 中将含有 height="120" 的 <td> 单元格移除固定高度属性（即删除 height="120" 出现在 <td align="center" width="50%" height="120"> 的两个单元格），让表格单元格高度随内容自适应；保留其他属性（align、width）不变，以避免文本在多语言或长描述时溢出或被截断。
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@profile/README.md`:
- Around line 42-47: 在 profile/README.md 中将含有 height="120" 的 <td>
单元格移除固定高度属性（即删除 height="120" 出现在 <td align="center" width="50%" height="120">
的两个单元格），让表格单元格高度随内容自适应；保留其他属性（align、width）不变，以避免文本在多语言或长描述时溢出或被截断。
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
profile/README.md (2)
59-59: 统一分类列的宽度属性。第 59 行的分类单元格包含
width="120"属性，但第 72 行和第 85 行缺少此属性。为保持布局一致性，建议为所有分类单元格添加相同的宽度设置。♻️ 建议的统一修改
- <td rowspan="3" align="center" valign="middle"><b>Tools</b></td> + <td rowspan="3" width="120" align="center" valign="middle"><b>Tools</b></td>- <td rowspan="3" align="center" valign="middle"><b>Simulation</b></td> + <td rowspan="3" width="120" align="center" valign="middle"><b>Simulation</b></td>Also applies to: 72-72, 85-85
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@profile/README.md` at line 59, The table's category cells are inconsistent: the <td> with <b>SDK</b> has width="120" but the other category <td> elements (the ones containing the other <b>...</b> category labels) are missing this attribute; update those <td> tags so each category cell uses the same width="120" attribute (i.e., add width="120" to the other category <td> elements to match the <td rowspan="3" width="120" align="center" valign="middle"><b>SDK</b></td> pattern).
23-25: 改进视频链接的无障碍性。建议为视频链接添加更明确的说明，让用户知道点击后会跳转到 YouTube。可以在图片的
alt属性中添加更详细的描述。♻️ 建议的改进
<a href="https://www.youtube.com/watch?v=LXVV-oErD8s"> - <img src="../images/wuji-hand-video.jpg" width="100%" alt="Wuji Hand"> + <img src="../images/wuji-hand-video.jpg" width="100%" alt="Wuji Hand demonstration video (YouTube)"> </a><a href="https://www.youtube.com/watch?v=NE5FytzRz_E"> - <img src="../images/wuji-glove-video.jpg" width="100%" alt="Wuji Glove"> + <img src="../images/wuji-glove-video.jpg" width="100%" alt="Wuji Glove demonstration video (YouTube)"> </a>Also applies to: 32-34
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@profile/README.md` around lines 23 - 25, 为提高无障碍性和可见性，更新视频链接的锚点和图片描述：在 <a href="https://www.youtube.com/watch?v=LXVV-oErD8s"> 元素上添加一个描述性 title 或 aria-label（例如“在 YouTube 打开视频：Wuji Hand（将在新标签页打开）”），并把 <img ... alt="Wuji Hand"> 的 alt 改为更具体的描述（例如“Wuji Hand 视频缩略图 — 点击将在 YouTube 新标签页打开”），可选地为锚点添加 target="_blank" 和 rel="noopener noreferrer" 以安全在新标签打开。确保对文件中相同模式的其它实例（行 32-34）做相同修改。
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@profile/README.md`:
- Around line 42-47: The table cells use a fixed height attribute (height="120")
on the <td> elements which can cause content clipping on different screen sizes;
remove the height="120" attribute from the affected <td> elements (the two <td
align="center" width="50%" ...> entries) so the cells can auto-size based on
their content, or replace it with responsive CSS if a minimum height is
required.
---
Nitpick comments:
In `@profile/README.md`:
- Line 59: The table's category cells are inconsistent: the <td> with <b>SDK</b>
has width="120" but the other category <td> elements (the ones containing the
other <b>...</b> category labels) are missing this attribute; update those <td>
tags so each category cell uses the same width="120" attribute (i.e., add
width="120" to the other category <td> elements to match the <td rowspan="3"
width="120" align="center" valign="middle"><b>SDK</b></td> pattern).
- Around line 23-25: 为提高无障碍性和可见性，更新视频链接的锚点和图片描述：在 <a
href="https://www.youtube.com/watch?v=LXVV-oErD8s"> 元素上添加一个描述性 title 或
aria-label（例如“在 YouTube 打开视频：Wuji Hand（将在新标签页打开）”），并把 <img ... alt="Wuji Hand">
的 alt 改为更具体的描述（例如“Wuji Hand 视频缩略图 — 点击将在 YouTube 新标签页打开”），可选地为锚点添加
target="_blank" 和 rel="noopener noreferrer" 以安全在新标签打开。确保对文件中相同模式的其它实例（行
32-34）做相同修改。
Use <picture> element with prefers-color-scheme to show dark logo on light backgrounds and light logo on dark backgrounds. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…repo - Clean SVG metadata: remove Chinese layer IDs, XML declaration, and CSS class references; use inline fill attributes instead - Add spacing between logo and tagline per brand clear space spec - Add wuji-hand-teleop to Teleoperation category in Repositories table Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
Author
|
@CodeRabbit approve |
Contributor
✅ Actions performedComments resolved and changes approved. |
wuji-tech-dev
approved these changes
Mar 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test Plan
🤖 Generated with Claude Code
Summary by CodeRabbit