What does browserSupportedMimes mean?

Cerrado
  • ¿Algo no está bien?
#46 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
35/100
Tipo de issue
Refactorización
Claridad
Necesita aclaración
Estado de actividad
Activo
Stack tecnológico
typescript
Área
frontend, web-dev

Línea de trabajo

Lee browserSupportedMimes en lib/models/audios.ts y lib/models/videos.ts; después, revisa el comportamiento medido en issue #45. Aclara si las listas describen tipos de archivo enrutados o tipos de reproducción esperados, y acuerden un solo nombre y una sola política para ambos handlers. Se considera terminado cuando el alcance y cualquier cambio de comportamiento se hayan decidido antes de la implementación.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

AI assisted status: triage type: discussion

browserSupportedMimes in lib/models/audios.ts and lib/models/videos.ts reads as a claim about what a browser can decode. It is not one, and some of what it lists no engine claims:

mime canPlayType Chromium Firefox listed in
audio/vorbis no no audios
audio/aacp no maybe audios
video/mpeg no no videos
video/x-flv no no videos
video/quicktime no maybe videos

Nothing is broken by that. The list decides which files the viewer offers to open; the handler then either plays the bytes or reports an error, and a media element reads the bytes rather than the declared type. #45 measured that end of it: one WAV served as audio/wav, audio/vnd.wave, audio/x-wav and application/octet-stream played in both engines, though canPlayType answers "no" for audio/vnd.wave.

So the question is what the list is meant to be, and the name should then say it:

  • If it is what we route to the handler, then handledMimes or similar is the honest name, and entries like audio/vorbis are harmless but pointless: vorbis is a codec inside audio/ogg, not a container type a file carries.
  • If it is what we expect to play, then it needs trimming, and that is a behaviour change for anyone whose files carry those types today.

Worth deciding once for both handlers rather than per pull request.

Lenguaje dominante
TypeScript
Estrellas
2
Forks
1
Merge medio
19 h 47 min
PR fusionados (30 d)
78

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de nextcloud-libraries/nextcloud-viewer

Todos los issues de nextcloud-libraries/nextcloud-viewer

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.