search

famanson / spring-roll-demo

Demo site for Project Spring Roll
1 stars 0 forks source link

Columnisation directive #15

Closed macduy closed 10 years ago

macduy commented 10 years ago

@famanson @cuctacuctac This change simplifies the controller, moving the columnising of posts into the View code. To display posts, you now only need to do this in controller.js:

$scope.posts = [...]

The view that displays these as columns (in index.html and ld-content.html):

<div sr-columns columns="3" posts="posts">...

This is truer to the MVC pattern.

famanson commented 10 years ago

Alright, I think this is merge-worthy, go ahead when you can