search

Financial-Times / o-grid

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

Restore `oGridSurfaceLayoutSizes` mixin and improve documentation. #211

Closed notlee closed 4 years ago

notlee commented 4 years ago

During the major cascade we restored the oGridSurfaceCurrentLayout mixin so the current layout could be exposed to JavaScript without including any other o-grid css. We did this because, even with no options set, the oGrid mixin outputs a lot of default CSS.

For the same reason this commit restores the oGridSurfaceLayoutSizes mixin.

It also updates the README and warns in the console if a JS method is used without including the required CSS.

Warnings refer to the README as users may be using SASS or the Build Service, and it would be good to avoid shipping verbose error messages to our users.

JakeChampion commented 4 years ago

:tada: This PR is included in version v5.2.0 :tada: