TheBronx
commented
10 years ago Agradecemos los aportes pero:
- los pull request mejor separados, uno para cada módulo.
- ninguno de los módulos respeta la estructura. Puedes abrir cualquier otro módulo y ver cómo funcionan, además de leer la plantilla comentada que sirve de ejemplo de módulo: https://github.com/TheBronx/shurscript/blob/master/modules/template.js. Los tuyos no usan la función de carga, si no que ejecutan el código por su cuenta, cosa que está mal, es el core quien debe lanzar los módulos, no ellos mismos.
- reglas de estilo: no vamos a rechazar pull request por usar espacios en vez de tabuladores por ejemplo, pero sí por no usar ninguno de los dos. el código de ambos módulos está casi sin indentar.
- si el módulo está a medio hacer, no lances pull request. una cosa es que aun se pueda mejorar, y otra que tenga trozos enteros de código comentados, o que directamente ni funcione. La librería que has incluido (jsPDF) me temo que está incompleta. No tiene la función
.fromHTML()que estás usando, así que previsiblemente dará un pete y no funcionará.
Así que insisto, agradecemos el aporte, pero hay que cumplir unos mínimos. Si tienes cualquier duda pregunta, nos tienes en forocoches, tanto en el hilo oficial como por privado.
rballeba
Dejo un nuevo plugin (creo que bien implementado) para convertir todo el post de forocoches en un pdf descargable, adicionalmente si se acepta comenzaré a trabajar con el. Además de un minihack para que adaptar las imágenes al ancho del foro