search

Financial-Times / o-grid

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

Es6 #102

Closed expe closed 9 years ago

expe commented 9 years ago

Convert syntax to ES6. DO NOT MERGE until OBT 4 is final.

CC @kaelig

kaelig commented 9 years ago

The bookmarklet needs to run in browsers as-is.

What is the native support for ES6 const/let?

kaelig commented 9 years ago

See https://github.com/Financial-Times/o-grid/compare/eslint-compat#diff-20845d069d433488af1128a336935c57 for the diff I had done on bookmarklet.js

expe commented 9 years ago

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/const

Let and const don't have great support.

Closing this as there's already a branch with the right bookmarklet. https://github.com/Financial-Times/o-grid/tree/eslint-compat