Conversation
There was a problem hiding this comment.
This pull request updates the screen recording GIF in the README files. The changes are straightforward and achieve the intended purpose.
🔍 General Feedback
- The use of a fixed width for the image in the README files can lead to display issues on smaller screens. I have added suggestions to make the image responsive.
| <tr> | ||
| <td style="border: 2px solid #6b7280; padding: 8px;"> | ||
| <img src="./resources/video_1.gif" alt="AionUi Demo" width="800"> | ||
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi Demo" width="800"> |
There was a problem hiding this comment.
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi Demo" width="800"> | |
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi Demo" width="800" style="max-width: 100%;"> |
| <tr> | ||
| <td style="border: 2px solid #6b7280; padding: 8px;"> | ||
| <img src="./resources/video_1.gif" alt="AionUi 演示" width="800"> | ||
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi 演示" width="800"> |
There was a problem hiding this comment.
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi 演示" width="800"> | |
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi 演示" width="800" style="max-width: 100%;"> |
There was a problem hiding this comment.
📋 Review Summary
This pull request updates the demo GIF in the README files. The changes are straightforward and align with the pull request's title. The new GIF is added to the resources folder.
🔍 General Feedback
- The change is simple and effective.
- The file names are clear.
- No issues found.
There was a problem hiding this comment.
📋 Review Summary
This pull request updates the screen recording GIF in the README files. The changes are straightforward and correct.
🔍 General Feedback
- The alt text for the images could be more descriptive for better accessibility. I have added suggestions for both
readme.mdandreadme_ch.md. - The new GIF file is included in the PR, which is good.
| <tr> | ||
| <td style="border: 2px solid #6b7280; padding: 8px;"> | ||
| <img src="./resources/video_1.gif" alt="AionUi Demo" width="800"> | ||
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi Demo" width="800"> |
There was a problem hiding this comment.
🟢 For better accessibility, consider using a more descriptive alt text for the screen recording.
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi Demo" width="800"> | |
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi screen recording of the application's features." width="800"> |
| <tr> | ||
| <td style="border: 2px solid #6b7280; padding: 8px;"> | ||
| <img src="./resources/video_1.gif" alt="AionUi 演示" width="800"> | ||
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi 演示" width="800"> |
There was a problem hiding this comment.
🟢 为了更好的可访问性,建议为屏幕录像使用更具描述性的 alt 文本。
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi 演示" width="800"> | |
| <img src="./resources/AionUI_screenrecord.gif" alt="AionUi 应用功能屏幕录像。" width="800"> |
…串口库\test\unit\gateway\pro prompt iOfficeAI#1
…安装的技能,一键导入或全部导入 prompt iOfficeAI#1
- Merge outline.md + brief.md into a single brief.md (skip redundant planning step) - Remove "Share Understanding" output to save tokens - Enforce exactly 3 deliverables: .pptx + build.sh + brief.md - Inline style quick-reference table into pptx-design.md (avoid 3-4 file reads of INDEX.md → style.md → build.sh) - Promote batch usage in generation strategy (reduce shell calls ~70%) - Simplify Phase 4 to validate + view outline only (per-slide self- check during Phase 3 is the primary quality gate) - Strengthen ghost-check rule for headline/content actors to prevent text overlap (#1 most common defect)
- Enhance #1 defect (text overlap) check with mandatory 3-step process in quality-gates.md (authoritative) and pptx-design.md (inline) - Simplify color decision flow from 3-step to 2-step in pptx-design.md - Consolidate shell script rules into officecli-pptx-min.md - Simplify font selection with fixed recommendation table - SKILL.md references authoritative docs instead of duplicating rules
Add missing boolean-as-string rule (the #1 cause of batch failures) and promote XPath numeric indexing and negative coordinates to a prominent "TOP 3 MOST COMMON ERRORS" section at the top of Shell Script Rules, based on real Agent error patterns observed in production.
Clone from the previous slide instead of always from slide 1. This carries forward ghost states automatically, preventing the #1 defect (text overlap from forgotten ghost operations). Before: clone slide[1] for every slide → all actors reset to original positions → Agent must remember to re-ghost everything. After: clone previous slide → already-ghosted actors stay ghosted → Agent only handles the delta between adjacent slides.
- Simplify Three-Layer to Two Types: Scene (persist) + Content (fresh) - Remove all defensive language (DO NOT, NEVER, prohibited, #1 DEFECT) - Remove redundant Morph pairing explanation (was in 3 places, now 1) - Remove "Not recommended" / "Technically possible" sections - Streamline text readability section - Write as clear positive instructions, trusting the Agent to execute
- Merge outline.md + brief.md into a single brief.md (skip redundant planning step) - Remove "Share Understanding" output to save tokens - Enforce exactly 3 deliverables: .pptx + build.sh + brief.md - Inline style quick-reference table into pptx-design.md (avoid 3-4 file reads of INDEX.md → style.md → build.sh) - Promote batch usage in generation strategy (reduce shell calls ~70%) - Simplify Phase 4 to validate + view outline only (per-slide self- check during Phase 3 is the primary quality gate) - Strengthen ghost-check rule for headline/content actors to prevent text overlap (#1 most common defect)
- Enhance #1 defect (text overlap) check with mandatory 3-step process in quality-gates.md (authoritative) and pptx-design.md (inline) - Simplify color decision flow from 3-step to 2-step in pptx-design.md - Consolidate shell script rules into officecli-pptx-min.md - Simplify font selection with fixed recommendation table - SKILL.md references authoritative docs instead of duplicating rules
Add missing boolean-as-string rule (the #1 cause of batch failures) and promote XPath numeric indexing and negative coordinates to a prominent "TOP 3 MOST COMMON ERRORS" section at the top of Shell Script Rules, based on real Agent error patterns observed in production.
Clone from the previous slide instead of always from slide 1. This carries forward ghost states automatically, preventing the #1 defect (text overlap from forgotten ghost operations). Before: clone slide[1] for every slide → all actors reset to original positions → Agent must remember to re-ghost everything. After: clone previous slide → already-ghosted actors stay ghosted → Agent only handles the delta between adjacent slides.
- Simplify Three-Layer to Two Types: Scene (persist) + Content (fresh) - Remove all defensive language (DO NOT, NEVER, prohibited, #1 DEFECT) - Remove redundant Morph pairing explanation (was in 3 places, now 1) - Remove "Not recommended" / "Technically possible" sections - Streamline text readability section - Write as clear positive instructions, trusting the Agent to execute
No description provided.