Skip to content

Update README.md Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com> Signed-off-by: Clriks <241179063+clauderiks@users.noreply.github.com> - #10

Open
zskbot wants to merge 5 commits into
codex/thiet-ke-lai-giao-dien-file-readme.mdfrom
codex/thiet-ke-lai-giao-dien-file-readme.md-nqgp3i

Conversation

@zskbot

@zskbot zskbot commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Summary

Closes #

How Did You Test this Change?

Images/Videos


Open in Devin Review

zskbot and others added 3 commits July 29, 2026 17:50
### Motivation
- Improve first-impression documentation by making the README more concise and visually focused with a centered hero and larger badges.
- Prioritize Vietnamese-first copy and a simpler structure so readers can find installation, preview, and configuration info quickly.
- Remove and condense overly broad or duplicate documentation sections to keep the repository README focused on the gh-dash terminal dashboard experience.

### Description
- Updated `README.md` to a redesigned layout with a centered hero, prominent badge styling, and Vietnamese-first description.
- Reorganized content into concise sections: highlights, table of contents, quick install, UI preview, default keybindings, sample config, developer setup, troubleshooting, and contribution guidance.
- Simplified and translated configuration and customization tables to compact, developer-oriented examples and removed extraneous API/examples material.
- Kept sample commands and `~/.config/gh-dash/config.yml` snippets intact and reformatted code blocks for readability.

### Testing
- Ran `npm run lint` (TypeScript type check via `tsc --noEmit`) which produced type errors originating from existing `docs/node_modules` and application type definitions, so linting did not pass and is unrelated to this README-only change.
- No other automated tests were run as part of this PR since the change is documentation-only.
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Signed-off-by: Clriks <241179063+clauderiks@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: bc20236b-81c6-4e83-8c85-b16be68ebf88

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

devin-ai-integration[bot]

This comment was marked as resolved.

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Redesign README and improve mobile responsiveness of docs site UI

📝 Documentation ✨ Enhancement 🕐 10-20 Minutes

Grey Divider

AI Description

• Redesigned README.md with a centered hero, larger badges, and Vietnamese-first copy for a
 clearer first impression.
• Consolidated and translated sections (install, keybindings, config, troubleshooting, contribution)
 while removing unrelated API/cloud-sync/2FA content.
• Improved mobile responsiveness across the docs app (Header, ReadmeViewer, App) with
 responsive flex/grid layouts and markdown styling.
• Added vite.config.ts to configure Vite with React and Tailwind plugins.
Diagram

graph TD
  A["README.md"] --> B["Vietnamese-first hero"] --> C["Concise sections"]
  D["docs site app"] --> E["Header.tsx"] --> F["Responsive layout"]
  D --> G["ReadmeViewer.tsx"] --> F
  D --> H["index.css"] --> F
  I["vite.config.ts"] --> D
Loading
High-Level Assessment

This is a documentation refresh paired with minor responsive CSS fixes to an existing internal docs-preview app; the scope is appropriately limited and there is no significant architectural alternative to consider.

Files changed (6) +297 / -194

Enhancement (4) +128 / -32
App.tsxImprove mobile layout of main content and footer +3/-3

Improve mobile layout of main content and footer

• Added overflow-x-hidden to main content wrapper and switched footer layout to a responsive flex-col/row arrangement for small screens.

src/App.tsx

Header.tsxMake header responsive for small screens +15/-15

Make header responsive for small screens

• Reworked header layout to stack vertically on mobile, added truncation/shrink handling for auth button and utility bar, and removed unused Command icon import.

src/components/Header.tsx

ReadmeViewer.tsxAdd mobile-friendly defaults and responsive grid to README viewer +26/-14

Add mobile-friendly defaults and responsive grid to README viewer

• Introduced a resize-based effect that defaults to preview mode on small screens, and adjusted grid/flex classes for search bar, view mode switcher, action buttons, and TOC ordering to be responsive.

src/components/ReadmeViewer.tsx

index.cssAdd base styles, markdown body typography, and scrollbar-hiding utility +84/-0

Add base styles, markdown body typography, and scrollbar-hiding utility

• Added base html/root background styles, markdown-body component classes for headings, code, tables, and a new scrollbar-none utility class used for overflow scroll areas.

src/index.css

Documentation (1) +162 / -162
README.mdRedesign README with Vietnamese-first hero and condensed sections +162/-162

Redesign README with Vietnamese-first hero and condensed sections

• Replaced the English-first README with a centered hero, larger badges, and Vietnamese-first descriptions. Reorganized content into concise sections (install, image gallery, usage flow, keybindings, sample config, dev setup, troubleshooting, contribution) and removed unrelated cloud-sync, 2FA, and REST/Node SDK documentation that didn't reflect the actual project.

README.md

Other (1) +7 / -0
vite.config.tsAdd Vite configuration with React and Tailwind plugins +7/-0

Add Vite configuration with React and Tailwind plugins

• New vite.config.ts wiring up @vitejs/plugin-react and @tailwindcss/vite plugins for the build.

vite.config.ts

@qodo-code-review

qodo-code-review Bot commented Jul 29, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (2) 📘 Rule violations (0) 📎 Requirement gaps (0) 🎨 UX issues (0) 🔗 Cross-repo conflicts (0) 📜 Skill insights (0)

Grey Divider


Remediation recommended

1. Broken README TOC links 🐞 Bug ≡ Correctness
Description
README.md’s Table of Contents uses fragments like #-cài-đặt-nhanh that won’t match GitHub’s
generated heading anchors for sections like ## 🚀 Cài đặt nhanh, so clicking TOC entries won’t
navigate to the intended sections.
Code

README.md[R56-64]

+- [Cài đặt nhanh](#-cài-đặt-nhanh)
+- [Bộ sưu tập hình ảnh](#-bộ-sưu-tập-hình-ảnh)
+- [Luồng sử dụng trong terminal](#-luồng-sử-dụng-trong-terminal)
+- [Phím tắt mặc định](#-phím-tắt-mặc-định)
+- [Cấu hình mẫu](#-cấu-hình-mẫu)
+- [Tùy biến thường dùng](#-tùy-biến-thường-dùng)
+- [Dành cho nhà phát triển](#-dành-cho-nhà-phát-triển)
+- [Khắc phục sự cố](#-khắc-phục-sự-cố)
+- [Đóng góp](#-đóng-góp)
Evidence
The TOC entries include a leading hyphen in the hash fragment, while the actual headings are plain
## … headings; GitHub’s slug for these headings will not include that extra leading -, so the
fragments won’t resolve.

README.md[54-69]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The README Table of Contents links use the wrong fragment identifiers (they include a leading `-`), causing section navigation to fail.

## Issue Context
GitHub generates heading IDs by stripping emojis and trimming whitespace; it does not prefix the slug with an extra hyphen.

## Fix Focus Areas
- README.md[56-65]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


2. Mobile resize forces preview ✓ Resolved 🐞 Bug ≡ Correctness
Description
ReadmeViewer installs a resize listener that sets viewMode to preview whenever the viewport is
<768px, which can override a user’s explicit selection of split or editor after
orientation/resize events.
Code

src/components/ReadmeViewer.tsx[R27-37]

+  useEffect(() => {
+    const applyMobileDefault = () => {
+      if (window.innerWidth < 768) {
+        setViewMode('preview');
+      }
+    };
+
+    applyMobileDefault();
+    window.addEventListener('resize', applyMobileDefault);
+    return () => window.removeEventListener('resize', applyMobileDefault);
+  }, []);
Evidence
The newly added effect registers a resize listener and unconditionally forces preview mode for
widths <768px; this competes with the view-mode buttons that allow selecting other modes.

src/components/ReadmeViewer.tsx[27-37]
src/components/ReadmeViewer.tsx[107-138]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
On small viewports, the resize handler always calls `setViewMode('preview')`, so user-selected modes can be reset unexpectedly.

## Issue Context
The component provides UI buttons to select `split`/`editor`/`preview`, but the resize handler can override those choices whenever the window is resized while still under the mobile breakpoint.

## Fix Focus Areas
- src/components/ReadmeViewer.tsx[27-37]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


3. Theme forced dark scheme 🐞 Bug ≡ Correctness
Description
Global CSS sets html { color-scheme: dark; } unconditionally, so browser-native UI
(inputs/scrollbars/etc.) stays in dark mode even when the app switches to light theme.
Code

src/index.css[R41-45]

+@layer base {
+  html {
+    color-scheme: dark;
+    background: #09090b;
+  }
Evidence
The CSS forces dark color-scheme globally, while the app’s theme system removes/adds the dark
class and exposes a Light Mode toggle—so the forced scheme conflicts with the intended theme
switching behavior.

src/index.css[41-45]
src/App.tsx[25-34]
src/components/Header.tsx[146-177]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`color-scheme: dark` is applied globally regardless of the selected theme, causing UA/native control styling to remain dark.

## Issue Context
The app exposes a light theme control and toggles theme classes on the document element; `color-scheme` should follow that selection (e.g., conditional on `.dark`).

## Fix Focus Areas
- src/index.css[41-45]
- src/App.tsx[25-34]
- src/components/Header.tsx[146-177]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

To customize comments, go to the Qodo configuration screen, or learn more in the docs.

Qodo Logo

qodo-code-review[bot]

This comment was marked as resolved.

@zskbot zskbot self-assigned this Jul 29, 2026
@zskbot zskbot added documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers invalid This doesn't seem right question Further information is requested v0 codex labels Jul 29, 2026
@github-project-automation github-project-automation Bot moved this from Todo to In Progress in @clauderiks's Jul 29, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Nodejs Jul 29, 2026
@github-project-automation github-project-automation Bot moved this from Todo to In Progress in Nodejs Jul 29, 2026
zskbot and others added 2 commits July 30, 2026 13:54
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Signed-off-by: Clriks <241179063+clauderiks@users.noreply.github.com>
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Signed-off-by: Clriks <241179063+clauderiks@users.noreply.github.com>

@devin-ai-integration devin-ai-integration Bot left a comment

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.

Devin Review found 8 new potential issues.

Open in Devin Review

Comment thread README.md
Comment thread src/components/ReadmeViewer.tsx
Comment thread src/App.tsx
Comment thread vite.config.ts
Comment thread src/index.css
Comment thread src/components/Header.tsx
Comment thread src/components/Header.tsx
Comment thread README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed invalid This doesn't seem right question Further information is requested v0

Projects

Status: In Progress
Status: In Progress

Development

Successfully merging this pull request may close these issues.

tự động hoá quy trình thực thi lệnh thiêt kế các nưt thủ công giao diện ngưởi dùng CHANGELOG.rst

1 participant