Closed geekman-rohit closed 8 years ago
@sfrisk I pushed to the PR.
@sfrisk , @arschmitz I suggest we rename to $breakpoint-sm instead of $viewport as these are breakpoints.
viewport-xs-min : 0 // we wont use or declare this, its assumed viewport-sm-min : 401 viewport-md-min: 769 viewport-lg-min: 921 viewport-xl-min: 1201
if everyone approves to that I could make the change and update ? Update: The above mentioned will be variables we use.(other than xs-min which we wont explicitly specify) Our viewports will be: xs(default): 0-400 sm: 401-768 md: 769-920 lg: 921-1200 xl: 1201-
@cvrebert is that (my earlier comment ) what you were suggesting?
@geekman-rohit Yeah, pretty much. Although the minimums being odd numbers seems slightly weird.
@sfrisk , my point in meeting today exactly. min should start at 400, xs ending at 399.
@cvrebert i agree
Then do that.
@sfrisk any reviews on this?
@geekman-rohit - remove the changes on files that aren't on breakpoint.js and make sure it works with the latest version of master, since I'm pretty sure this branch is a bit out of date. Once that is done, I can pull in these changes.
@sfrisk I think this can go in now?
Bahhhhh, I missed a step when I merged this in. My bad guys.
Adds Breakpoints variables and modifies typography to use them.