enhancement: Add body scroll lock and top-right close button to README Badge Modal
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 82/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- nextjs, react, typescript
- Lĩnh vực
- accessibility, frontend
Hướng nghiên cứu
Bắt đầu tại src/features/developer/components/user-profile-client.tsx và theo dõi cách badgeDialogOpen điều khiển modal README Badge. Thêm khóa cuộn nền để khôi phục trạng thái overflow trước đó khi hộp thoại đóng hoặc bị tháo gỡ, đồng thời thêm nút đóng ở góc trên bên phải với aria label đã dịch theo yêu cầu. Hoàn tất khi nền không còn cuộn trong lúc modal mở và một trong hai nút đóng đều có thể đóng modal.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
In src/features/developer/components/user-profile-client.tsx, clicking 'Get README Badge' opens a modal dialog (badgeDialogOpen). When open, mouse wheel scrolling on the backdrop scrolls the underlying page. In addition, the modal only has a bottom 'Close' button without a standard top-right X close icon.
Motivation
Proper modal UX requires locking background scroll and offering an intuitive top-right dismissal button for accessibility and desktop/mobile usability.
Tasks
- When
badgeDialogOpenis true, setdocument.body.style.overflow = "hidden"(and restore on close/unmount). - Add an
Xicon button in the top-right corner of the dialog header witharia-label={t("profile.close")}. - Ensure clicking the
Xbutton closes the dialog.
Files to Check
src/features/developer/components/user-profile-client.tsx
[!TIP]
🚀 Want to contribute?
Commentassign meto be automatically assigned to this issue via our GitHub Actions bot. Happy coding! ✨
- Ngôn ngữ chính
- TypeScript
- Star
- 31
- Fork
- 25
- Merge trung bình
- 10 giờ 4 phút
- Pull request đã merge (30 ngày)
- 29
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của O2sa/DevImpact
-
beginner friendly easy enhancement good first issue nextjs ui
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug: InsightsList incorrectly favors User 1 when metrics are tiedCó thể đã có người làm @sarmad-jalil đã nhận 2 ngày trước. Đang mởbeginner friendly bug easy good first issue typescript
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
O2sa/DevImpact#268 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
feature: Add custom branded not-found.tsx page for 404 routesCó thể đã có người làm @sarmad-jalil đã nhận 2 ngày trước. Đang mởbeginner friendly easy feature good first issue nextjs ui
O2sa/DevImpact#271 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement: Add 1-click Clear button (X) inside SearchFieldCó thể đã có người làm @aryansinha1908 đã nhận 4 ngày trước. Đang mởbeginner friendly easy enhancement good first issue ui
O2sa/DevImpact#265 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của O2sa/DevImpact
Issue tương tự
-
[Integration]: eRPCĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
ava-labs/builders-hub#4826 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
记忆分析的默认模型显示与实际使用不一致Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
anomalyco/opencode#59886 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
-
a10y a10y-audit ai-driven-qa claude
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
Maintainer thường phản hồi trong vòng 1 ngày