Don't rely on IDE=VSCode in querystring for behaviour
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- dart
- Área
- developer-experience, tooling
Línea de trabajo
Revisa el manejo de IDE en packages/devtools_app/lib/src/shared/development_helpers.dart, packages/devtools_app/lib/src/shared/preferences/preferences.dart y packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/property_editor_panel.dart. Primero rastrea cómo llegan los valores de IDE a la cadena de consulta y prueba el comportamiento de fallback de WASM. Se considera terminado cuando el filtro de la encuesta, el mensaje de fallback y el enlace a la documentación del editor de propiedades funcionan correctamente para VS Code, Antigravity, Cursor y otras variantes de IDE.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Dart
- Estrellas
- 1.7k
- Forks
- 405
- Merge medio
- 4 d 21 h
- PR fusionados (30 d)
- 15
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de flutter/devtools
-
cleanup P1
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
-
`dcm analyze` false positives Abiertoa: quality cost: low P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
contributions welcome cross-screen inspector-2.0-polish P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Support removing storage keys Abiertocross-screen P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
documentation good first issue P2
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Todos los issues de flutter/devtools
Issues similares
-
[Bug]: New version of nm package Abiertobug triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
fluttercommunity/plus_plugins#3979 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
from.page-issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
enhancement UI: Material
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Notsfsssf/pixez-flutter#1360 ·
-
triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100