Skip to content

update the record gif#1

Merged
halvee-tech merged 1 commit intomainfrom
update_readme
Aug 7, 2025
Merged

update the record gif#1
halvee-tech merged 1 commit intomainfrom
update_readme

Conversation

@IceyLiu
Copy link
Copy Markdown
Collaborator

@IceyLiu IceyLiu commented Aug 7, 2025

No description provided.

@halvee-tech halvee-tech merged commit 42fa302 into main Aug 7, 2025
goworm pushed a commit that referenced this pull request Sep 3, 2025
Copy link
Copy Markdown
Contributor

@aionui aionui bot left a comment

Choose a reason for hiding this comment

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

## 📋 Review Summary

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">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 The image has a fixed width of 800px, which may not be responsive on smaller screens. Adding `style="max-width: 100%;"` will allow the image to scale down gracefully while maintaining its original size on larger screens.
Suggested change
<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">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 The image has a fixed width of 800px, which may not be responsive on smaller screens. Adding `style="max-width: 100%;"` will allow the image to scale down gracefully while maintaining its original size on larger screens.
Suggested change
<img src="./resources/AionUI_screenrecord.gif" alt="AionUi 演示" width="800">
<img src="./resources/AionUI_screenrecord.gif" alt="AionUi 演示" width="800" style="max-width: 100%;">

Copy link
Copy Markdown
Contributor

@aionui aionui bot left a comment

Choose a reason for hiding this comment

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

📋 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.

Copy link
Copy Markdown
Contributor

@aionui aionui bot left a comment

Choose a reason for hiding this comment

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

📋 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.md and readme_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">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 For better accessibility, consider using a more descriptive alt text for the screen recording.

Suggested change
<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">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 为了更好的可访问性,建议为屏幕录像使用更具描述性的 alt 文本。

Suggested change
<img src="./resources/AionUI_screenrecord.gif" alt="AionUi 演示" width="800">
<img src="./resources/AionUI_screenrecord.gif" alt="AionUi 应用功能屏幕录像。" width="800">

TuziChan added a commit to TuziChan/AionUi that referenced this pull request Mar 14, 2026
TuziChan added a commit to TuziChan/AionUi that referenced this pull request Mar 14, 2026
TuziChan added a commit to TuziChan/AionUi that referenced this pull request Mar 14, 2026
TuziChan added a commit to TuziChan/AionUi that referenced this pull request Mar 14, 2026
IceyLiu added a commit that referenced this pull request Mar 21, 2026
- 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)
IceyLiu added a commit that referenced this pull request Mar 21, 2026
- 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
IceyLiu added a commit that referenced this pull request Mar 21, 2026
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.
IceyLiu added a commit that referenced this pull request Mar 21, 2026
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.
IceyLiu added a commit that referenced this pull request Mar 21, 2026
- 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
kaizhou-lab pushed a commit that referenced this pull request Mar 22, 2026
- 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)
kaizhou-lab pushed a commit that referenced this pull request Mar 22, 2026
- 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
kaizhou-lab pushed a commit that referenced this pull request Mar 22, 2026
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.
kaizhou-lab pushed a commit that referenced this pull request Mar 22, 2026
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.
kaizhou-lab pushed a commit that referenced this pull request Mar 22, 2026
- 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
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