Feature: Support more languages in source view.
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 75/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- cpp, javascript, rust, typescript
- Área
- frontend
Línea de trabajo
La elección del lenguaje está en src/components/shared/SourceView-codemirror.ts, donde el switch fija JavaScript, Rust y C++ (líneas 49-79 en el commit enlazado). Lee ese switch y luego comprueba cómo se podría llamar a LanguageDescription.matchFilename de CodeMirror a partir del nombre del archivo antes del respaldo de JavaScript. Está terminado cuando un archivo fuente con otra extensión, como .py, recibe resaltado de sintaxis y los casos existentes de JS, Rust y C++ siguen funcionando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Since processed profile json version 64 its possible to include source code of various languages in the profile json. It would be far nicer if this source code was able to be syntax highlighted, at least for all languages supported by CodeMirror.
The current syntax highlighting switch hardcodes js, rust, and cpp:
https://github.com/firefox-devtools/profiler/blob/f3c1c766e85154725ed7c1c4ecdf7d4aa9136892/src/components/shared/SourceView-codemirror.ts#L49-L79
Codemirror supports filename matching across all its built in languages: https://codemirror.net/docs/ref/#language.LanguageDescription^matchFilename
I think, mostly the task would be to actually call this before falling back to javascript.
The profile for screenshot:
ffprofile.json
┆Issue is synchronized with this Jira Task
- Lenguaje dominante
- TypeScript
- Estrellas
- 1.5k
- Forks
- 496
- Merge medio
- 1 d 23 h
- PR fusionados (30 d)
- 37
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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 firefox-devtools/profiler
-
Add a section for about:logging in our user docsPosiblemente ocupada @ShryukGrandhi la tomó hace 12 días. Abiertodocumentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
firefox-devtools/profiler#6310 ·
Los mantenedores suelen responder en 1 día
-
Stale/broken `from-browser` URLs triggers an error message that starts with "Error: Error: "Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
firefox-devtools/profiler#6110 ·
Los mantenedores suelen responder en 1 día
-
samples.argumentValues is present in profiles from the Gecko profiler even if tracing is offAbiertoprofile data
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
firefox-devtools/profiler#5971 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Automate the remaining deployment stepsPosiblemente ocupada @fatadel la tomó hace 1 día. Abierto
firefox-devtools/profiler#6435 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Can't search in source viewPosiblemente ocupada @mstange la tomó hace 13 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
firefox-devtools/profiler#6412 ·
Los mantenedores suelen responder en 1 día
Todos los issues de firefox-devtools/profiler
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
bug needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
DSpace/dspace-angular#6511 ·
Los mantenedores suelen responder en 1 día
-
area: frontend area:byok bug severity:low
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Phone app shows Unknown for a remote machine whose key changedPosiblemente ocupada @janicduplessis la tomó hoy. Abiertoarea: mobile priority: P3 type: bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
appandflow/stim#5006 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Area - Local to Cloud Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
microsoft/vscode-mssql#23209 ·
Los mantenedores suelen responder en 1 día