From bb19bf1e533959f90cb79b13d8401beab8f9337a Mon Sep 17 00:00:00 2001 From: Brian Hough Date: Mon, 24 Apr 2017 11:23:26 +0200 Subject: [PATCH 1/2] fix(clearFix): Fixes an issue with content being blank. Fixes an issue with clearFix where `content` was blank due to the empty quotes being parsed as a null value. #140 --- docs/assets/polished.js | 18 +-- docs/docs/index.html | 134 +++++++++--------- src/mixins/clearFix.js | 4 +- .../test/__snapshots__/clearFix.test.js.snap | 4 +- 4 files changed, 80 insertions(+), 80 deletions(-) diff --git a/docs/assets/polished.js b/docs/assets/polished.js index b73d6c81..a9855c32 100644 --- a/docs/assets/polished.js +++ b/docs/assets/polished.js @@ -219,7 +219,7 @@ function modularScale(steps) { var base = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : '1em'; var ratio = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : 'perfectFourth'; - if (!steps) { + if (typeof steps !== 'number') { throw new Error('Please provide a number of steps to the modularScale helper.'); } if (typeof ratio === 'string' && !ratioNames[ratio]) { @@ -233,7 +233,7 @@ function modularScale(steps) { throw new Error('Invalid value passed as base to modularScale, expected number or em string but got "' + base + '"'); } - return realBase * realRatio * steps + 'em'; + return realBase * Math.pow(realRatio, steps) + 'em'; } // @@ -411,7 +411,7 @@ var taggedTemplateLiteral = function (strings, raw) { * * '&::after': { * 'clear': 'both', - * 'content': '', + * 'content': '""', * 'display': 'table' * } */ @@ -422,7 +422,7 @@ function clearFix() { var pseudoSelector = parent + '::after'; return defineProperty({}, pseudoSelector, { 'clear': 'both', - 'content': '', + 'content': '""', 'display': 'table' }); } @@ -1042,7 +1042,7 @@ var functionsMap = { /** */ /** - * String to represent common easing functions as demonstrated here: (github.com/jaukia/easie). + * String to represent commong easing functions as demonstrated here: (github.com/jaukia/easie). * * @example * // Styles as object usage @@ -1161,10 +1161,10 @@ function hslToRgb(hue, saturation, lightness) { var hexRegex = /^#[a-fA-F0-9]{6}$/; var reducedHexRegex = /^#[a-fA-F0-9]{3}$/; -var rgbRegex = /^rgb\((\d{1,3}), ?(\d{1,3}), ?(\d{1,3})\)$/; -var rgbaRegex = /^rgba\((\d{1,3}), ?(\d{1,3}), ?(\d{1,3}), ?([-+]?[0-9]*[.]?[0-9]+)\)$/; -var hslRegex = /^hsl\((\d{1,3}), ?(\d{1,3})%, ?(\d{1,3})%\)$/; -var hslaRegex = /^hsla\((\d{1,3}), ?(\d{1,3})%, ?(\d{1,3})%, ?([-+]?[0-9]*[.]?[0-9]+)\)$/; +var rgbRegex = /^rgb\(\s*(\d{1,3})\s*,\s*(\d{1,3})\s*,\s*(\d{1,3})\s*\)$/; +var rgbaRegex = /^rgba\(\s*(\d{1,3})\s*,\s*(\d{1,3})\s*,\s*(\d{1,3})\s*,\s*([-+]?[0-9]*[.]?[0-9]+)\s*\)$/; +var hslRegex = /^hsl\(\s*(\d{1,3})\s*,\s*(\d{1,3})%\s*,\s*(\d{1,3})%\s*\)$/; +var hslaRegex = /^hsla\(\s*(\d{1,3})\s*,\s*(\d{1,3})%\s*,\s*(\d{1,3})%\s*,\s*([-+]?[0-9]*[.]?[0-9]+)\s*\)$/; /** * Returns an RgbColor or RgbaColor object. This utility function is only useful diff --git a/docs/docs/index.html b/docs/docs/index.html index 55166cab..dc7e5f4b 100644 --- a/docs/docs/index.html +++ b/docs/docs/index.html @@ -810,7 +810,7 @@

- + src/mixins/clearFix.js @@ -870,7 +870,7 @@

'&::after': { 'clear': 'both', - 'content': '', + 'content': '""', 'display': 'table' } @@ -896,7 +896,7 @@

- + src/mixins/ellipsis.js @@ -985,7 +985,7 @@

- + src/mixins/fontFace.js @@ -1147,7 +1147,7 @@

- + src/mixins/hiDPI.js @@ -1239,7 +1239,7 @@

- + src/mixins/hideText.js @@ -1314,7 +1314,7 @@

- + src/mixins/normalize.js @@ -1398,7 +1398,7 @@

- + src/mixins/placeholder.js @@ -1501,7 +1501,7 @@

- + src/mixins/radialGradient.js @@ -1641,7 +1641,7 @@

- + src/mixins/retinaImage.js @@ -1766,7 +1766,7 @@

- + src/mixins/selection.js @@ -1865,7 +1865,7 @@

- + src/mixins/timingFunctions.js @@ -1948,7 +1948,7 @@

- + src/mixins/wordWrap.js @@ -2046,7 +2046,7 @@

- + src/color/adjustHue.js @@ -2146,7 +2146,7 @@

- + src/color/complement.js @@ -2236,7 +2236,7 @@

- + src/color/darken.js @@ -2335,7 +2335,7 @@

- + src/color/desaturate.js @@ -2435,7 +2435,7 @@

- + src/color/grayscale.js @@ -2525,7 +2525,7 @@

- + src/color/hsl.js @@ -2632,7 +2632,7 @@

- + src/color/hsla.js @@ -2750,7 +2750,7 @@

- + src/color/invert.js @@ -2841,7 +2841,7 @@

- + src/color/lighten.js @@ -2940,7 +2940,7 @@

- + src/color/mix.js @@ -3055,7 +3055,7 @@

- + src/color/opacify.js @@ -3152,7 +3152,7 @@

- + src/color/parseToHsl.js @@ -3231,7 +3231,7 @@

- + src/color/parseToRgb.js @@ -3310,7 +3310,7 @@

- + src/color/rgb.js @@ -3417,7 +3417,7 @@

- + src/color/rgba.js @@ -3535,7 +3535,7 @@

- + src/color/saturate.js @@ -3636,7 +3636,7 @@

- + src/color/setHue.js @@ -3735,7 +3735,7 @@

- + src/color/setLightness.js @@ -3834,7 +3834,7 @@

- + src/color/setSaturation.js @@ -3933,7 +3933,7 @@

- + src/color/shade.js @@ -4026,7 +4026,7 @@

- + src/color/tint.js @@ -4119,7 +4119,7 @@

- + src/color/transparentize.js @@ -4228,7 +4228,7 @@

- + src/shorthands/animation.js @@ -4329,7 +4329,7 @@

- + src/shorthands/backgroundImages.js @@ -4412,7 +4412,7 @@

- + src/shorthands/backgrounds.js @@ -4495,7 +4495,7 @@

- + src/shorthands/borderColor.js @@ -4581,7 +4581,7 @@

- + src/shorthands/borderRadius.js @@ -4673,7 +4673,7 @@

- + src/shorthands/borderStyle.js @@ -4759,7 +4759,7 @@

- + src/shorthands/borderWidth.js @@ -4845,7 +4845,7 @@

- + src/shorthands/buttons.js @@ -4935,7 +4935,7 @@

- + src/shorthands/margin.js @@ -5021,7 +5021,7 @@

- + src/shorthands/padding.js @@ -5107,7 +5107,7 @@

- + src/shorthands/position.js @@ -5221,7 +5221,7 @@

- + src/shorthands/size.js @@ -5314,7 +5314,7 @@

- + src/shorthands/textInputs.js @@ -5416,7 +5416,7 @@

- + src/shorthands/transitions.js @@ -5511,7 +5511,7 @@

- + src/helpers/directionalProperty.js @@ -5605,7 +5605,7 @@

- + src/helpers/em.js @@ -5698,7 +5698,7 @@

- + src/helpers/modularScale.js @@ -5801,7 +5801,7 @@

- + src/helpers/rem.js @@ -5894,7 +5894,7 @@

- + src/helpers/stripUnit.js @@ -5994,7 +5994,7 @@

- + src/helpers/modularScale.js @@ -6043,7 +6043,7 @@

- + src/types/color.js @@ -6115,7 +6115,7 @@

- + src/types/color.js @@ -6193,7 +6193,7 @@

- + src/types/color.js @@ -6265,7 +6265,7 @@

- + src/types/color.js @@ -6343,7 +6343,7 @@

- + src/mixins/fontFace.js @@ -6451,7 +6451,7 @@

- + src/mixins/radialGradient.js @@ -6535,7 +6535,7 @@

- + src/mixins/timingFunctions.js @@ -6584,7 +6584,7 @@

- + src/shorthands/animation.js @@ -6633,7 +6633,7 @@

- + src/shorthands/buttons.js @@ -6682,7 +6682,7 @@

- + src/shorthands/textInputs.js @@ -6731,7 +6731,7 @@

- + src/color/toColorString.js @@ -6829,7 +6829,7 @@

- + src/mixins/triangle.js diff --git a/src/mixins/clearFix.js b/src/mixins/clearFix.js index 9ab03562..250f55d6 100644 --- a/src/mixins/clearFix.js +++ b/src/mixins/clearFix.js @@ -18,7 +18,7 @@ * * '&::after': { * 'clear': 'both', - * 'content': '', + * 'content': '""', * 'display': 'table' * } */ @@ -28,7 +28,7 @@ function clearFix(parent: string = '&') { return { [pseudoSelector]: { 'clear': 'both', - 'content': '', + 'content': '""', 'display': 'table', }, } diff --git a/src/mixins/test/__snapshots__/clearFix.test.js.snap b/src/mixins/test/__snapshots__/clearFix.test.js.snap index 7341bdcb..c7321ac3 100644 --- a/src/mixins/test/__snapshots__/clearFix.test.js.snap +++ b/src/mixins/test/__snapshots__/clearFix.test.js.snap @@ -2,7 +2,7 @@ exports[`clearFix should default to & when no parent is passed 1`] = ` Object { "&::after": Object { "clear": "both", - "content": "", + "content": "\"\"", "display": "table", }, } @@ -12,7 +12,7 @@ exports[`clearFix should pass parent to pseudo selector 1`] = ` Object { "div::after": Object { "clear": "both", - "content": "", + "content": "\"\"", "display": "table", }, } From 7c48950954e14466d4e274d2b762ac9be4bec127 Mon Sep 17 00:00:00 2001 From: Brian Hough Date: Tue, 25 Apr 2017 11:30:46 +0200 Subject: [PATCH 2/2] docs(timingFunctions): Typo in timingFunctions doc --- docs/assets/polished.js | 2 +- docs/docs/index.html | 134 +++++++++++++++++----------------- src/mixins/timingFunctions.js | 2 +- 3 files changed, 69 insertions(+), 69 deletions(-) diff --git a/docs/assets/polished.js b/docs/assets/polished.js index a9855c32..5faa5e97 100644 --- a/docs/assets/polished.js +++ b/docs/assets/polished.js @@ -1042,7 +1042,7 @@ var functionsMap = { /** */ /** - * String to represent commong easing functions as demonstrated here: (github.com/jaukia/easie). + * String to represent common easing functions as demonstrated here: (github.com/jaukia/easie). * * @example * // Styles as object usage diff --git a/docs/docs/index.html b/docs/docs/index.html index dc7e5f4b..7e9e0a01 100644 --- a/docs/docs/index.html +++ b/docs/docs/index.html @@ -810,7 +810,7 @@

- + src/mixins/clearFix.js @@ -896,7 +896,7 @@

- + src/mixins/ellipsis.js @@ -985,7 +985,7 @@

- + src/mixins/fontFace.js @@ -1147,7 +1147,7 @@

- + src/mixins/hiDPI.js @@ -1239,7 +1239,7 @@

- + src/mixins/hideText.js @@ -1314,7 +1314,7 @@

- + src/mixins/normalize.js @@ -1398,7 +1398,7 @@

- + src/mixins/placeholder.js @@ -1501,7 +1501,7 @@

- + src/mixins/radialGradient.js @@ -1641,7 +1641,7 @@

- + src/mixins/retinaImage.js @@ -1766,7 +1766,7 @@

- + src/mixins/selection.js @@ -1865,14 +1865,14 @@

- + src/mixins/timingFunctions.js -

String to represent commong easing functions as demonstrated here: (github.com/jaukia/easie).

+

String to represent common easing functions as demonstrated here: (github.com/jaukia/easie).

timingFunctions(timingFunction: TimingFunction)
@@ -1948,7 +1948,7 @@

- + src/mixins/wordWrap.js @@ -2046,7 +2046,7 @@

- + src/color/adjustHue.js @@ -2146,7 +2146,7 @@

- + src/color/complement.js @@ -2236,7 +2236,7 @@

- + src/color/darken.js @@ -2335,7 +2335,7 @@

- + src/color/desaturate.js @@ -2435,7 +2435,7 @@

- + src/color/grayscale.js @@ -2525,7 +2525,7 @@

- + src/color/hsl.js @@ -2632,7 +2632,7 @@

- + src/color/hsla.js @@ -2750,7 +2750,7 @@

- + src/color/invert.js @@ -2841,7 +2841,7 @@

- + src/color/lighten.js @@ -2940,7 +2940,7 @@

- + src/color/mix.js @@ -3055,7 +3055,7 @@

- + src/color/opacify.js @@ -3152,7 +3152,7 @@

- + src/color/parseToHsl.js @@ -3231,7 +3231,7 @@

- + src/color/parseToRgb.js @@ -3310,7 +3310,7 @@

- + src/color/rgb.js @@ -3417,7 +3417,7 @@

- + src/color/rgba.js @@ -3535,7 +3535,7 @@

- + src/color/saturate.js @@ -3636,7 +3636,7 @@

- + src/color/setHue.js @@ -3735,7 +3735,7 @@

- + src/color/setLightness.js @@ -3834,7 +3834,7 @@

- + src/color/setSaturation.js @@ -3933,7 +3933,7 @@

- + src/color/shade.js @@ -4026,7 +4026,7 @@

- + src/color/tint.js @@ -4119,7 +4119,7 @@

- + src/color/transparentize.js @@ -4228,7 +4228,7 @@

- + src/shorthands/animation.js @@ -4329,7 +4329,7 @@

- + src/shorthands/backgroundImages.js @@ -4412,7 +4412,7 @@

- + src/shorthands/backgrounds.js @@ -4495,7 +4495,7 @@

- + src/shorthands/borderColor.js @@ -4581,7 +4581,7 @@

- + src/shorthands/borderRadius.js @@ -4673,7 +4673,7 @@

- + src/shorthands/borderStyle.js @@ -4759,7 +4759,7 @@

- + src/shorthands/borderWidth.js @@ -4845,7 +4845,7 @@

- + src/shorthands/buttons.js @@ -4935,7 +4935,7 @@

- + src/shorthands/margin.js @@ -5021,7 +5021,7 @@

- + src/shorthands/padding.js @@ -5107,7 +5107,7 @@

- + src/shorthands/position.js @@ -5221,7 +5221,7 @@

- + src/shorthands/size.js @@ -5314,7 +5314,7 @@

- + src/shorthands/textInputs.js @@ -5416,7 +5416,7 @@

- + src/shorthands/transitions.js @@ -5511,7 +5511,7 @@

- + src/helpers/directionalProperty.js @@ -5605,7 +5605,7 @@

- + src/helpers/em.js @@ -5698,7 +5698,7 @@

- + src/helpers/modularScale.js @@ -5801,7 +5801,7 @@

- + src/helpers/rem.js @@ -5894,7 +5894,7 @@

- + src/helpers/stripUnit.js @@ -5994,7 +5994,7 @@

- + src/helpers/modularScale.js @@ -6043,7 +6043,7 @@

- + src/types/color.js @@ -6115,7 +6115,7 @@

- + src/types/color.js @@ -6193,7 +6193,7 @@

- + src/types/color.js @@ -6265,7 +6265,7 @@

- + src/types/color.js @@ -6343,7 +6343,7 @@

- + src/mixins/fontFace.js @@ -6451,7 +6451,7 @@

- + src/mixins/radialGradient.js @@ -6535,7 +6535,7 @@

- + src/mixins/timingFunctions.js @@ -6584,7 +6584,7 @@

- + src/shorthands/animation.js @@ -6633,7 +6633,7 @@

- + src/shorthands/buttons.js @@ -6682,7 +6682,7 @@

- + src/shorthands/textInputs.js @@ -6731,7 +6731,7 @@

- + src/color/toColorString.js @@ -6829,7 +6829,7 @@

- + src/mixins/triangle.js diff --git a/src/mixins/timingFunctions.js b/src/mixins/timingFunctions.js index 379c574f..5460bda3 100644 --- a/src/mixins/timingFunctions.js +++ b/src/mixins/timingFunctions.js @@ -59,7 +59,7 @@ type TimingFunction = | 'easeInOutSine'; /** - * String to represent commong easing functions as demonstrated here: (github.com/jaukia/easie). + * String to represent common easing functions as demonstrated here: (github.com/jaukia/easie). * * @example * // Styles as object usage