Make UI consistent regarding when to clear input validation errors
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Lĩnh vực
- documentation, frontend
Hướng nghiên cứu
Compare validation-error behavior in the and , then locate the style guide and all other places where input validation errors are shown. First resolve the project-wide rule for clearing errors; done means the rule is documented and consistently applied across those locations.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
We currently don’t follow a consistent pattern in regards to clearing validation errors:
- In the
<security-dialog>, if you enter too short a password and try to submit the form, the validation error is hidden once you start changing one of the input fields (i.e., once you start typing). - In the
<change-hostname-dialog>, if you enter a malformed hostname such as*&!^@^#*(, the validation error stays until you re-submit with another input value.
Good points can be made for either way:
- By clearing the validation error after the user starts doing something, we acknowledge that the problem now might have gone away, and the user might feel more encouraged to re-try their submission.
- By leaving the validation until re-submission, the user is able to study the error text, in order to ensure that they address the issue properly. That’s especially helpful if the error message contains more comprehensive information, such as the allowed characters for a valid hostname.
A potential third (middleground) option could be to add some sort of “dismiss” functionality (e.g., an × button in the upper right corner of the inline message box). That way, the error box would persist by default, but the user could explicitly make it go away.
What we should do is this:
- Come up with and decide on an explicit rule how we go about clearing validation errors. This should be documented in the style guide.
- Roll out the new rule to all places where input validation errors are shown.
- Ngôn ngữ chính
- Python
- Star
- 3.5k
- Fork
- 291
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
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 tiny-pilot/tinypilot
-
bug medium
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
tiny-pilot/tinypilot#1419 ·
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
tiny-pilot/tinypilot#1929 · 3 bình luận ·
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
tiny-pilot/tinypilot#1899 · 1 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
tiny-pilot/tinypilot#1896 ·
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
tiny-pilot/tinypilot#1882 ·
Tất cả issue của tiny-pilot/tinypilot
Issue tương tự
-
Broken links found in docsĐang mởdocs pydanty:is-working
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
pydantic/pydantic-ai#8863 ·
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 68/100
run-llama/llama_index#23278 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
documentation from-review-extraction github-actions priority: low severity:nit
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
LearningCircuit/local-deep-research#6946 ·
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 82/100
oracle/langchain-oracle#323 ·
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 88/100
tenstorrent/tt-metal#58057 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày