material-components / material-components-web-codelabs

Codelabs for Material Components for Web (MDC Web)
MIT License
129 stars 102 forks source link

[codelab] No argument named $component-height #197

Open Mamikel opened 3 years ago

Mamikel commented 3 years ago

In following the tutorial: (Web)MDC-102 Web: Material Structure and Layout (Web)

After updating home.scss and saving, the compilation fails as follows:

  [25] ./home.scss 1.44 kB {0} [built] [failed] [1 error]
    + 11 hidden modules

ERROR in ./home.scss
Module build failed:
undefined
   ^
      No argument named $component-height.
   ┌──> node_modules\@material\list\_mixins.scss
536│ ┌     @include shape-mixins.radius(
537│ │       $radius,
538│ │       $rtl-reflexive,
539│ │       $component-height: $height,
540│ │       $query: $query
541│ │     );
   │ └─────^ invocation
   ╵
   ┌──> node_modules\@material\shape\_mixins.scss
30 │   @mixin radius($radius, $rtl-reflexive: false, $query: feature-targeting-functions.all()) {
   │          ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ declaration
   ╵
  node_modules\@material\list\_mixins.scss 536:5             radius()
  node_modules\@material\list\_mixins.scss 536:5             single-line-shape-radius()
  node_modules\@material\drawer\_mixins.scss 304:3           item-shape-radius()
  node_modules\@material\drawer\_mixins.scss 71:5            core-styles()
  node_modules\@material\drawer\common.scss 24:1             @use
  node_modules\@material\drawer\mdc-drawer.scss 23:1         @forward
  node_modules\@material\drawer\mdc-drawer.import.scss 37:1  @import
  stdin 2:9                                                  root stylesheet
      in C:\Users\moranmat\Downloads\Dev\material-components-web-codelabs-master\mdc-101\starter\node_modules\@material\list\_mixins.scss (line 536, column 5)
 @ multi (webpack)-dev-server/client?http://localhost:8080 ./home.scss
JayDijkstra commented 3 years ago

It seems that within this codelab, the command npm install @material/drawer @material/list installs both dependencies on the latest version @^7.0.0 instead of @^6.0.0 (which has explicit been used in the previous lab MDC-101).

After running npm install @material/drawer@^6.0.0 @material/list@^6.0.0 everything seems to work again.