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

Incorrect behaviour of unquote and list #1258

Closed
KittyGiraudel opened this issue Jun 3, 2015 · 1 comment
Closed

Incorrect behaviour of unquote and list #1258

KittyGiraudel opened this issue Jun 3, 2015 · 1 comment

Comments

@KittyGiraudel
Copy link

Not sure if there is already a relevant open issue.

Test

$list:   '(-webkit-min-device-pixel-ratio: 2)', '(min-resolution: 192dpi)'; 
$string: '(-webkit-min-device-pixel-ratio: 2),   (min-resolution: 192dpi)';

.foo {
  // I should not unquote a list, I know. But still.
  content: unquote($list);
  content: unquote($string);
}

Expected

.foo {
  content: "(-webkit-min-device-pixel-ratio: 2)", "(min-resolution: 192dpi)";
  content: (-webkit-min-device-pixel-ratio: 2), (min-resolution: 192dpi);
}

Actual

.foo {
  content: (-webkit-min-device-pixel-ratio: 2)", "(min-resolution: 192dpi);
  content: (-webkit-min-device-pixel-ratio: 2), (min-resolution: 192dpi);
}
@KittyGiraudel KittyGiraudel changed the title @media and list Incorrect behaviour with @media and list Jun 3, 2015
@KittyGiraudel KittyGiraudel changed the title Incorrect behaviour with @media and list Incorrect behaviour of unquote and list Jun 3, 2015
@mgreter mgreter added this to the 3.3.1 milestone Jun 13, 2015
mgreter added a commit to mgreter/libsass that referenced this issue Jun 15, 2015
Feature is deprecated in ruby sass. We may
should print a warning to inform the user.

Fixes sass#1258
@mgreter mgreter self-assigned this Jun 15, 2015
@mgreter mgreter modified the milestones: 3.3, 3.3.1 Jun 15, 2015
mgreter added a commit to mgreter/libsass that referenced this issue Jun 23, 2015
Feature is deprecated in ruby sass. We may
should print a warning to inform the user.

Fixes sass#1258
mgreter added a commit to mgreter/libsass that referenced this issue Jun 30, 2015
Feature is deprecated in ruby sass. We may
should print a warning to inform the user.

Fixes sass#1258
mgreter added a commit to mgreter/libsass that referenced this issue Jun 30, 2015
Feature is deprecated in ruby sass. We may
should print a warning to inform the user.

Fixes sass#1258
xzyfer pushed a commit to xzyfer/libsass that referenced this issue Jul 8, 2015
Feature is deprecated in ruby sass. We may
should print a warning to inform the user.

Fixes sass#1258
@xzyfer
Copy link
Contributor

xzyfer commented Jul 8, 2015

Fixed by #1312

@xzyfer xzyfer closed this as completed Jul 8, 2015
xzyfer added a commit to xzyfer/sass-spec that referenced this issue Jul 8, 2015
xzyfer pushed a commit to xzyfer/libsass that referenced this issue Jul 8, 2015
Feature is deprecated in ruby sass. We may
should print a warning to inform the user.

Fixes sass#1258
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

3 participants