Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reorganizar archivos #11

Open
rickyrauch opened this issue Oct 26, 2012 · 4 comments
Open

Reorganizar archivos #11

rickyrauch opened this issue Oct 26, 2012 · 4 comments
Milestone

Comments

@rickyrauch
Copy link
Contributor

En una aplicacion de 2 pantallas no puede ser que haya 40 templates.

@ghost ghost assigned cristiandouce Oct 26, 2012
@cristiandouce
Copy link
Contributor

Tenemos 3 elementos principales a mostrar:

  • Issue
  • Idea
  • Vote

Además, necesitamos una pantalla donde poder "explorarlos" (en lista, con filtros por orden de edicion, etc...).

Por otro lado, tenemos también el perfil del usuario.

Suman ya 5 pantallas (minimo).

Los archivos pequeños son en su mayoría necesarios para las respuestas en ajax. (Necesito tener todo en trozos para poder utilizar lo que me haga falta devolver según la request...)

Aca esta el ejemplo de como se implementa esto ultimo:

Convengamos además que en Jade, tener un archivo demasiado grande se hace más difícil de leer y traba al debug y progreso de la app.

Ahora lo que hice es algo asi:

  • issues/
    • list.jade
    • single.jade
    • form.jade
  • ideas/
    • list.jade
    • single.jade
    • form.jade
  • element.jade (que es extendido por los 'single.jade')
  • element/ (contiene los trozos de element.jade, 'comments' por ahora)
  • lists.jade (que es extendido por los 'list.jade')
  • profile.jade
  • voting.jade (que era pensado para listar las votaciones en proceso)
  • index.jade (este archivo sirve solo cuando se cargar por primera vez la app. Puede usarse para cargar las configuraciones y asi evitar el uso de "config.json")

Tendría que trabajar en desmembrar element un poco más... para que al cargar una iniciativa me agregue el html sin tener que refrescar la pagina!

Dame todos los comentarios que tengas al respecto para que lo pueda mejorar.

@cristiandouce
Copy link
Contributor

Bueno, el link que pase arriba esta fuera de uso, dado que TJ y Guille quitaron toda funcionalidad de partials a express y Jade.
Tendré que encontrar una nueva forma de hacer esto...

@rickyrauch
Copy link
Contributor Author

Listo los que no tienen sentido:

  • comment_form.jade
  • /issues/* (puede ser un archivo solamente)
  • /ideas/* (puede ser un archivo solamente)

rickyrauch pushed a commit that referenced this issue Nov 6, 2012
@rickyrauch
Copy link
Contributor Author

Fijate las reglas que puse en el wiki.

cristiandouce added a commit that referenced this issue Dec 19, 2012
gvilarino pushed a commit that referenced this issue Oct 1, 2014
cristiandouce added a commit that referenced this issue Oct 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants