prefeiturasp / SME-plataforma-curriculo

GNU Affero General Public License v3.0
10 stars 11 forks source link

Embedar os links dos vídeos na página das atividades #20

Closed carlalink closed 6 years ago

carlalink commented 6 years ago

Qual a melhoria proposta? Hoje quando um link do vídeo é citado ao longo do texto aparece como texto ou link, o que queremos é que ele possa ser embedado na atividade.

Descreva a solução que você espera Se for citado como atividade ele pode ser embedado e já mostrar o vídeo diretamente na tela.

Exceção para o caso das orientações para o professor (bloco cinza), que poderá aparecer apenas como link. isto é, se for neste bloco cinza, não deverá ser embedado.

screen shot 2018-09-19 at 13 26 24
victorsenam commented 6 years ago

Dei uma procurada por algum exemplo de atividade onde aparece algum vídeo que deveria ser embedado, mas não encontrei. Podem linkar algum?

victorsenam commented 6 years ago

@carlalink me mandou esses posts com vídeos do YouTube que deveríamos embeddar:

Também foi esclarecido que o comentário sobre não embeddar os vídeos em bloco cinza deveria ser ignorado. Isto é, queremos que eles apareçam, também, em formato de vídeo.

victorsenam commented 6 years ago

Olá :wave: Eu resolvi isso em dois pull requests: https://github.com/prefeiturasp/SME-plataforma-curriculo-API/pull/51 e https://github.com/prefeiturasp/SME-plataforma-curriculo-interface/pull/73.

Na verdade, eu não fiz com que os vídeos já inseridos virassem incluídos, eu fiz ser possível incluir vídeos pela ferramenta de edição de atividades mesmo. Desta forma quem estiver escrevendo a atividade tem que explicitamente fazer com que o vídeo seja incluído.

Isso tem um probleminha porque o quill, ferramenta que a gente está usando para fazer aquele editor de texto, não consegue inserir vídeos dentro de blockquotes, ou seja, não é possível inserir um vídeo neste bloco cinza.

A gente pode tentar resolver isso também se for uma prioridade, mas seria mais difícil.

carlalink commented 6 years ago

shooow. Não é prioridade resolver essa questão do vídeo no cinza. sem problemas!

On Sat, Oct 13, 2018 at 8:01 PM Victor Sena Molero notifications@github.com wrote:

Olá 👋 Eu resolvi isso em dois pull requests: prefeiturasp/SME-plataforma-curriculo-API#51 https://github.com/prefeiturasp/SME-plataforma-curriculo-API/pull/51 e prefeiturasp/SME-plataforma-curriculo-interface#73 https://github.com/prefeiturasp/SME-plataforma-curriculo-interface/pull/73 .

Na verdade, eu não fiz com que os vídeos já inseridos virassem incluídos, eu fiz ser possível incluir vídeos pela ferramenta de edição de atividades mesmo. Desta forma quem estiver escrevendo a atividade tem que explicitamente fazer com que o vídeo seja incluído.

Isso tem um probleminha porque o quill https://quilljs.com/, ferramenta que a gente está usando para fazer aquele editor de texto, não consegue inserir vídeos dentro de blockquotes, ou seja, não é possível inserir um vídeo neste bloco cinza.

A gente pode tentar resolver isso também se for uma prioridade, mas seria mais difícil.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/prefeiturasp/SME-plataforma-curriculo/issues/20#issuecomment-429582321, or mute the thread https://github.com/notifications/unsubscribe-auth/AP1CdKnY5ZFcQndL44VTJU1YefqEhVbEks5uknDegaJpZM4WwhxU .

--

Carla Link F. www.talkingcity.co Talking City https://www.facebook.com/talkingcity/?fref=ts 🇧🇷 +55 (011) 989-033-290