search

Financial-Times / o-grid

Responsive grid system
http://registry.origami.ft.com/components/o-grid
94 stars 17 forks source link

Update layout names list before generating grid #74

Closed wheresrhys closed 9 years ago

wheresrhys commented 9 years ago

Fixes https://github.com/Financial-Times/next-sass-setup/pull/22#discussion_r26352114

@kaelig - you'll know better than me whether this fix needs applying in other places in the codebase too, or whether there are unintended side effects

kaelig commented 9 years ago

Oh great, can you put a small comment before that line explaining why this is necessary?

kaelig commented 9 years ago

Actually should we have a helper to add a layout that would automatically do that?

kaelig commented 9 years ago

Should be fixed in v3.1.0-beta.1 - can you please try using it?

kaelig commented 9 years ago

/nudge @wheresrhys

kaelig commented 9 years ago

Closing this as it's been fixed.