Consolidate “dialog requested” events
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 42/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- html, javascript
- Lĩnh vực
- frontend
Hướng nghiên cứu
Read app/static/js/events.js and the dialog-triggering section of app/static/js/app.js first, then inspect the menu-bar references and the wifi-dialog example. Consolidate the dialog request events around the shared event and update external menu-bar injections; done means dialog requests use the common event path without the individual event ids or menu-bar dependency.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
We currently have a significant number of events that trigger opening a dialog – e.g. shutdown-dialog-requested, update-dialog-requested, etc. We also currently register these events at <menu-bar> level, not at document level.
The downsides of this are that it’s hard to keep track of all these individual event ids (which we must ensure are globally unique), and that we cannot trivially trigger a dialog from anywhere outside the <menu-bar> (e.g., from another dialog). See discussion at https://github.com/tiny-pilot/tinypilot/pull/1860.
So we could consolidate all those ...-dialog-requested events into a single DialogRequestedEvent that would live in events.js. That way, we reduce the overall number of event ids that we have to manage, and we can simplify our event handler registration code.
When we do this, we should also refactor any places outside of app.js where we reference the <menu-bar> to inject events directly – such as in the <wifi-dialog>.
- Ngôn ngữ chính
- Python
- Star
- 3.5k
- Fork
- 292
- 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
- Không có Dockerfile hay tệp Docker Compose
- Không 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 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 ·
-
bug
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
tiny-pilot/tinypilot#1961 ·
-
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 ·
Tất cả issue của tiny-pilot/tinypilot
Issue tương tự
-
`image_generation`: `max_retries` is ignored for OpenAI-compatible providers (always 2 retries)Có thể đã có người làm @ericz7900 đã nhận hôm nay. Đang mởllm translation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/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 72/100
rheeloaded/paperpull#117 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ 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
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
agenda-tech-brasil/agenda-tech-brasil#1619 · 1 bình luận ·
-
process
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 76/100
pyrite-wiki/pyrite#1036 ·
Maintainer thường phản hồi trong vòng 1 ngày