Don't rely on IDE=VSCode in querystring for behaviour
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
- 48/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- dart
- Lĩnh vực
- developer-experience, tooling
Hướng nghiên cứu
Rà soát việc xử lý IDE trong packages/devtools_app/lib/src/shared/development_helpers.dart, packages/devtools_app/lib/src/shared/preferences/preferences.dart và packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/property_editor_panel.dart. Trước tiên, hãy theo dõi cách các giá trị IDE đi vào chuỗi truy vấn và kiểm thử hành vi fallback của WASM. Công việc được xem là hoàn tất khi bộ lọc khảo sát, thông báo fallback và liên kết tài liệu của trình chỉnh sửa thuộc tính hoạt động chính xác cho VS Code, Antigravity, Cursor và các biến thể IDE khác.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
https://github.com/Dart-Code/Dart-Code/issues/5970 is a request to change the values pass in the querystring passed to DevTools to include the proper IDE name and not hard-coded values like "VSCode" (which prevents distinguishing VS Code forks using the same extension, like Antigravity or Cursor).
However, there are some places that currently assume "VSCode" here in DevTools that I think require reviewing first (or, we need to pass two values).
- Survey IDE filters
- WASM/JS fallback message
- Property editor documentation link
1. Survey IDE Filters
I don't know if this is used, but if it is, it would need to cover each of the variations of editors now. I think we could either:
- Update this comment to mention this is based on DASH__IDE_NAME and include updated examples ("Visual Studio Code", "Antigravity", "Cursor")
- Pass two values in querystring, DASH__IDE_NAME and also DASH__TOOL (or DASH__PLUGIN_NAME) which are constant across different IDE flavours using the same plugin/extension
2. WASM/JS fallback message
It controls whether to show a message if the wasm version fails. I'm not sure if this is necessary anymore, as I think there were WASM changes to not require COI? If so, this code could probably just be deleted (and if not, I think won't cause any problems). Needs testing though.
3. Property editor documentation link
If IDE is not exactly "VSCode" this will link to the Android Studio docs.
- Ngôn ngữ chính
- Dart
- Star
- 1.7k
- Fork
- 405
- Merge trung bình
- 4 ngày 21 giờ
- Pull request đã merge (30 ngày)
- 15
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 flutter/devtools
-
cleanup P1
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
-
`dcm analyze` false positives Đang mởa: quality cost: low P2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
contributions welcome cross-screen inspector-2.0-polish P2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Support removing storage keys Đang mởcross-screen P3
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
documentation good first issue P2
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Tất cả issue của flutter/devtools
Issue tương tự
-
[Bug]: New version of nm package Đang mởbug triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
fluttercommunity/plus_plugins#3979 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
from.page-issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
-
enhancement UI: Material
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Notsfsssf/pixez-flutter#1360 ·
-
triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100