Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Undefined operation with multiplication #2309

Closed
danez opened this issue Jan 31, 2017 · 1 comment
Closed

Undefined operation with multiplication #2309

danez opened this issue Jan 31, 2017 · 1 comment

Comments

@danez
Copy link

danez commented Jan 31, 2017

input.scss

$button-sizes: (
  'xs': (
    'line-height': 16 / 12,
  ),
  's': (
    'line-height': 18 / 14,
  ),
  'm': (
    'line-height': 18 / 14,
  ),
  'l': (
    'line-height': 22 / 16,
  )
);

@each $size in $button-sizes {
  $size-metrics: nth($size, 2);

  .c-button__icon {
    min-height: map-get($size-metrics, 'line-height') * 1em;
  }
}

libsass 3.5.0.beta.2

Error: Undefined operation: "16/12 times 1em".
       on line 20 of src/scss/rg.scss
>>     min-height: map-get($size-metrics, 'line-height') * 1em;
   ----------------^

ruby sass 3.4.21

.c-button__icon {
  min-height: 1.33333em;
}

.c-button__icon {
  min-height: 1.28571em;
}

.c-button__icon {
  min-height: 1.28571em;
}

.c-button__icon {
  min-height: 1.375em;
}

version info:

$ node-sass --version
node-sass	4.4.0	(Wrapper)	[JavaScript]
libsass  	3.5.0.beta.2	(Sass Compiler)	[C/C++]

http://www.sassmeister.com/gist/38aa5e8c468bfb9cb15ff21c88e595b1

@mgreter
Copy link
Contributor

mgreter commented Feb 4, 2017

Thx for the report! This will be fixed by #2317

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants