Laboratoria
feat(docs): Adds /no-iframe/ flag to topics/ README's videos we don't…
… want the parser to turn into iframes
@unjust este es un PR que va en conjunto con este otro PR a tu rama de transformar links de tu fork del repo de currículum-parser, entre ambos solucionan el issue que habíamos conversado sobre no mostrar videos en ciertas circunstancias, como en listas, funciona simplemente agregando un flag (string) /no-iframe/ al final del atributo título de cada link y luego agregando esa condición a la función que transforma los links en iframes.
busqué todos los links que tuvieran la string "youtu" y "vimeo" y que fueran parte de un link en markdown que estuviera en una lista utilizando la siguiente expresión regular:
(?:^[*-]\s*\[[^\]]+\]\((?:[^)]*(youtu|vimeo)[^)]*)\)|^\|\s*\[[^\]]+\]\((?:[^)]*(youtu|vimeo)[^)]*)\))
espero no se me hayan escapado muchos otros casos :grin:
entonces las listas que se veían así:
con estos cambios ya se ven así:
No he tenido chance a revisar, pero quiero pensar cual seria mejor - usar /no-iframe/ o usar /with-iframe/ - dependente en cual es el caso mas comun.
Estoy 99% seguro (lo revisé antes de mandar este PR) de que el caso más común y esperado por usuarias (sobre todo cuando no somos ni tú no yo :laughing: ) es que al agregar el link de un video, el video aparezca, en cambio, en raras ocasiones, el video es simplemente parte de una lista de links, que son justo los casos que se atacan en este PR! Y es por eso que opté por dejar como default la opción de mostrar el video, y en cambio, cuando sólo quieres que sea un link que es parte de una lista de links, tener que tomarse la molestia de agregar el title con la string /no-iframe/ para asegurar ese comportamiento! :blush: