Don't rely on IDE=VSCode in querystring for behaviour
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- dart
- Ambito
- developer-experience, tooling
Direzione di ricerca
Esamina la gestione degli IDE in packages/devtools_app/lib/src/shared/development_helpers.dart, packages/devtools_app/lib/src/shared/preferences/preferences.dart e packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/property_editor_panel.dart. Per prima cosa traccia il modo in cui i valori degli IDE arrivano nella stringa di query e testa il comportamento di fallback di WASM. Il lavoro è completato quando il filtro del sondaggio, il messaggio di fallback e il link alla documentazione del property editor funzionano correttamente per VS Code, Antigravity, Cursor e altre varianti di IDE.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Dart
- Stelle
- 1.7k
- Fork
- 405
- Merge medio
- 4g 21h
- PR unite (30g)
- 15
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di flutter/devtools
-
cleanup P1
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
-
a: quality cost: low P2
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
contributions welcome cross-screen inspector-2.0-polish P2
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
cross-screen P3
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
documentation good first issue P2
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
Tutte le issue di flutter/devtools
Issue simili
-
bug triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
fluttercommunity/plus_plugins#3979 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
from.page-issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
-
enhancement UI: Material
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Notsfsssf/pixez-flutter#1360 ·
-
triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100