Primitive Progress wrapper omits numeric props from Root and ARIA value
Maintainer thường phản hồi trong vòng 2 ngày
Một pull request liên quan đã được merge.
- #7438 của @harsh21234i — đã merge
Đá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
- 75/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- python
- Lĩnh vực
- backend-api-design
Hướng nghiên cứu
Tìm wrapper Progress nguyên thủy nơi RadixProgressRoot được khởi tạo chỉ với className; thêm các prop value và max vào phần tử Root để nó trở nên xác định và cung cấp aria-valuenow. Xác minh bằng cách render component và kiểm tra xem phần tử Root có các thuộc tính data-state và aria-valuenow chính xác không.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The high-level primitive Progress wrapper forwards value and max to its Indicator, leaving the Root indeterminate and without aria-valuenow. State-driven visual updates work, but assistive technology cannot read the numeric progress.
This reproduces with stable reflex-components-radix==0.9.10.post1/Reflex 0.9.12 and alpha Radix/Reflex 0.10.0a1; it is preexisting. Themes rx.progress exposes the value correctly.
Reproduce
import reflex as rx
from reflex_components_radix.primitives.progress import progress
class State(rx.State):
amount: int = 25
# Render in an app, with a State event/slider that changes amount.
progress(value=State.amount, max=100, id="primitive-progress")
- Open the page and inspect
#primitive-progress. - Change
State.amountfrom 25 to 100 through the app's slider. - Expected: the Root is determinate, exposes the current numeric
aria-valuenow, and reaches the complete state at 100. - Actual: the visual Indicator updates, but the Root remains
data-state="indeterminate"and omitsaria-valuenow.
Construction output isolates the prop placement:
RadixProgressRoot: className only; no value/max
RadixProgressIndicator: value=<State.amount>, max=100
Environment and evidence
macOS, Python 3.13, real dev/prod Chromium 143.0.7499.4 testing. Only published packages were installed in isolated environments; no framework fix was applied.
- Small construction probe.
- Stable render and alpha render, under
primitive_progress. - Dashboard with State slider and Themes comparison.
- Browser assertions and production screenshot.
- Ngôn ngữ chính
- Python
- Star
- 29k
- Fork
- 1.8k
- Merge trung bình
- 3 ngày 7 giờ
- Pull request đã merge (30 ngày)
- 169
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- 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 reflex-dev/reflex
-
Add a `SetVar` backed by the JS `Set` objectCó thể đã có người làm @FarhanAliRaza đã nhận 1 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
reflex-dev/reflex#7623 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
reflex-dev/reflex#7260 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
reflex-dev/reflex#7148 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
ClientStateVar: a component that mounts while the value is non-default never re-renders on the push back to the defaultCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
reflex-dev/reflex#6823 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
support redis v8Đang mởfeature request
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
reflex-dev/reflex#6764 · 3 bình luận · 1 reaction ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của reflex-dev/reflex
Issue tương tự
-
intake mcp-intake needs-ac needs-human-review priority:medium
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
Ikalus1988/MisakaNet#3848 · 2 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 75/100
521xueweihan/HelloGitHub#4105 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
igormcsouza/bookloud#54 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
aws-samples/sample-per-model-bedrock#526 ·
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 78/100
ok-oldking/ok-wuthering-waves#1802 ·
Maintainer thường phản hồi trong vòng 2 ngày