cristobalmedinalopez
commented
6 years ago Hi @Lilium213,
I reviewed your Pull Request (PR), it works fine but before merging it with the master branch we suggest some minor changes:
- Since tWeb directory is not part of the project, it should be removed from this PR
- Some function names, text or buttons are written in Spanish. Could you refactor it to an appropriate name in English?
- The button used to send/load the subtitles has a meaningless label. Maybe a label such as "Load Subtitles" or "Load/Send Subtitles" would be better.
Thanks!
Se han añadido unas funciones a room.js para que el host pueda subir un archivo de subtitulos (.vtt) al video. Las funciones son "subirArchivoSubtitulos", que permite introducir los subtitulos de forma local, "sendSubtitles" y "onReadSubtitles", que separan el archivo del subtitulo en chunks si es demasiado grande y lo envian a los demas usuarios, y "recibirSubtitulos", que rearma los chunks que se reciben y crea un archivo de tipo vtt para añadirlo al video. Se hace la llamada a "sendSubtitles" cuando se le da al nuevo boton que hay abajo. El "recibirSubtitulos" se llama en onmessage cuando message.event == "subtitles".