Don't rely on IDE=VSCode in querystring for behaviour
维护者通常 3 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 48/100
- Issue 类型
- 重构
- 描述清晰度
- 基本清楚
- 活跃度
- 冷清
- 技术栈
- dart
调研方向
检查 packages/devtools_app/lib/src/shared/development_helpers.dart、packages/devtools_app/lib/src/shared/preferences/preferences.dart 和 packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/property_editor_panel.dart 中对 IDE 的处理。首先跟踪 IDE 值如何进入查询字符串,并测试 WASM 回退行为。当调查筛选器、回退消息和属性编辑器文档链接能够正确支持 VS Code、Antigravity、Cursor 及其他 IDE 变体时,即视为完成。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Dart
- 星标
- 1.7k
- 派生
- 408
- 平均合并
- 4 天 20 小时
- 30 天内合并 PR
- 11
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
flutter/devtools 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 73/100
维护者通常 3 天内回复
-
P1 type-code-health
难度 2/5 1-3 小时 新手友好度 85/100
flutter/devtools#9973 · 3 条评论 ·
维护者通常 3 天内回复
-
cost: low P2 type-code-health
难度 2/5 1-3 小时 新手友好度 68/100
flutter/devtools#9942 · 3 条评论 ·
维护者通常 3 天内回复
-
[Inspector V2] Widget properties tab highlight extends past its rounded edge可能重新可做 @theprantadutta 于 152 天前认领,目前没有进行中的 PR。 未关闭contributions welcome cross-screen inspector-2.0-polish P2
难度 2/5 1-3 小时 新手友好度 65/100
flutter/devtools#8936 · 3 条评论 ·
维护者通常 3 天内回复
-
cross-screen P3
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 3 天内回复
相似的 Issue
-
bug Done
难度 2/5 1-3 小时 新手友好度 66/100
维护者通常 1 天内回复
-
Use getApplicationSupportDirectory() of path_provider instead of getApplicationDocumentsDirectory()未关闭
难度 2/5 1-3 小时 新手友好度 65/100
brandonp2412/Flexify#387 · 2 条评论 ·
维护者通常 1 天内回复
-
bug product: very_good_flutter_plugin
难度 1/5 1-3 小时 新手友好度 78/100
VeryGoodOpenSource/very_good_templates#654 ·
维护者通常 1 天内回复
-
enhancement
难度 2/5 1-3 小时 新手友好度 80/100
维护者通常 1 天内回复
-
Server never consumes the request body on early-error paths: _sinkIncoming does not resume the paused subscription可能重新可做 关联的 PR 已关闭且未合并。 未关闭
难度 2/5 1-3 小时 新手友好度 72/100