Skip to content

Commit

Permalink
refactor: replace node-sass by sass (pattern-lab#1373)
Browse files Browse the repository at this point in the history
* refactor: replaced node-sass by sass and removed node-sass-selector-importer

node-sass is deprecated, integrated its successor sass instead; and node-sass-selector-importer never seems to have been used

* refactor(scss): replaced by new Dart Sass syntax

DEPRECATION WARNING: Using / for division is deprecated and will be removed in Dart Sass 2.0.0.

* refactor: simplified those calculations

to even also prevent the usage of math.div

* Revert "refactor: replaced node-sass by sass and removed node-sass-selector-importer"

This reverts commit ca72dcf.

* refactor: replaced node-sass by sass and removed node-sass-selector-impo…

…rter

node-sass is deprecated, integrated its successor sass instead; and node-sass-selector-importer never seems to have been used
  • Loading branch information
mfranzke authored Nov 21, 2021
1 parent fd7632b commit a1fac7c
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 9 deletions.
3 changes: 1 addition & 2 deletions packages/uikit-workshop/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -78,8 +78,6 @@
"lit-html": "^1.1.2",
"mini-css-extract-plugin": "^0.8.0",
"mousetrap": "^1.6.5",
"node-sass": "^6.0.0",
"node-sass-selector-importer": "^5.2.0",
"postcss-loader": "^3.0.0",
"preact": "8.3.1",
"preact-compat": "3.18.4",
Expand All @@ -94,6 +92,7 @@
"react-html-parser": "^2.0.2",
"redux": "3.7.2",
"redux-thunk": "^2.3.0",
"sass": "1.43.4",
"sass-loader": "^10.1.1",
"scriptjs": "^2.5.9",
"scroll-js": "^2.2.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ $pl-font-size-large: 1.2rem;
$pl-space: 1rem;
$pl-doublespace: $pl-space * 2;
$pl-pad: 1rem;
$pl-pad-half: $pl-pad/2;
$pl-pad-half: $pl-pad * 0.5;
$offset-top: 2rem;

// Breakpoints
Expand All @@ -57,4 +57,4 @@ $pl-border-radius: 3px;
$pl-border-radius-med: 6px;


$pl-sidebar-width: 16rem; //Define sidebar width for calculating dimensions
$pl-sidebar-width: 16rem; //Define sidebar width for calculating dimensions
Original file line number Diff line number Diff line change
Expand Up @@ -257,8 +257,8 @@ pl-nav {
.pl-c-nav__link--sublink {
text-transform: none;
font-size: 0.78rem;
padding-left: $pl-space + ($pl-space / 2);
padding-right: $pl-space + ($pl-space / 2);
padding-left: $pl-space + $pl-space * 0.5;
padding-right: $pl-space + $pl-space * 0.5;

&.pl-is-active {
box-shadow: inset 4px 0 0 #6c79d9;
Expand Down
4 changes: 1 addition & 3 deletions packages/uikit-workshop/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ const TerserPlugin = require('terser-webpack-plugin');
const autoprefixer = require('autoprefixer');
const HtmlWebpackPlugin = require('html-webpack-plugin');
const MiniCssExtractPlugin = require('mini-css-extract-plugin');
const selectorImporter = require('node-sass-selector-importer');
const webpack = require('webpack');
const CopyPlugin = require('copy-webpack-plugin');
const path = require('path');
Expand Down Expand Up @@ -121,8 +120,7 @@ module.exports = function (apiConfig) {
options: {
sassOptions: {
sourceMap: config.sourceMaps,
outputStyle: 'expanded',
importer: [selectorImporter()],
outputStyle: 'expanded'
},
},
},
Expand Down

0 comments on commit a1fac7c

Please sign in to comment.