it3s / meppit

3 stars 1 forks source link

Views with partials [DO NOT MERGE] #49

Closed andersoncardoso closed 10 years ago

andersoncardoso commented 10 years ago

DO NOT MERGE

Luiz to mandado o PR somente pra vc inspecionar as mudanças que foram feitas.

Eu modifiquei as views para usar partials, e removi aquele esquema de layouts. Agora tanto geo_data, user ou maps somente precisam renderizar o partial shared/objects. Para lidar com as diferenças entre objetos eu usei presenters (tanto no toolbar, quanto nos objects).

Pra eu renderizar colaboration de um user, com o mesmo "layout" de um user eu só preciso renderizar normalmente o contributable substituindo o content.

Da uma olhada e vê o que vc acha.

LuizArmesto commented 10 years ago

Vou dar uma olhada melhor hoje a tarde. As mudanças de partials eu já tinha dado uma espiada e gostei, o presenter que eu ainda não vi como ficou.

coveralls commented 10 years ago

Coverage Status

Coverage decreased (-0.04%) when pulling 724fdff6d94791dbf1a7045b9324c01cc7605ef9 on views-with-partials into 8323805d6177ed538b4f840c536f741b6530f0fb on master.