@font-face undefined is not a function

[reworkcss]

@font-face undefined is not a function

[necolas]

Please include the stack trace

[smykes]

Stack Trace is essentially
StringifyCSS -> Identity.compile -> Identity.stylesheet -> Base.mapVisit
does that help at all?

[conradz]

The example the you gave parses and stringifies successfully, and looking at the methods you mentioned does not turn up anything obvious. Could you post the entire stack trace including line numbers?

[smykes]

I just had an epiphany that I will check tomorrow morning. If I find it to be wrong I will post more information.

[smykes]

So it turns out it's not you it's me. The parsing is happening without a problem, but I am altering selectors between the parse and the stringify to add a prefix (foo) before each selector so #bar would turn into #foo #foobar, I have everything working but @font-face rules are tripping it up. Obviously this isn't your problem and you can close the issue, but any help would be appreciated. Code is below.

ast.stylesheet.rules.forEach(function each(rule) {
            if (rule.type === "font-face") {
                return;
            }
            if (false) {
                errors.push({
                    line: rule.declarations.line,
                    message: rule.declarations.message
                });
            } else {
                if (!rule.selectors) { //if the selectors are not on the first layer
                    if (!rule.rules) { //
                        return;
                    } else {
                        for (var i = 0; i < rule.rules.length; i++) {
                            rule.rules[i].selectors = rule.rules[i].selectors.map(function(selector) {
                                return nameSpaceCSS(selector);
                            });
                        }
                    }
                } else { //the selector is on the first layer.
                    rule.selectors = rule.selectors.map(function(selector) {
                        return nameSpaceCSS(selector);
                    });
                }
            }
        });

[smykes]

This is what I'm trying to stringify:

{
  "type": "stylesheet",
  "stylesheet": {
    "rules": [
      {
        "type": "font-face",
        "declarations": [
          {
            "type": "declaration",
            "property": "font-family",
            "value": "'MyFontFamily'",
            "position": {
              "start": {
                "line": 2,
                "column": 5
              },
              "end": {
                "line": 2,
                "column": 32
              }
            }
          },
          {
            "type": "declaration",
            "property": "src",
            "value": "url('myfont-webfont.eot?#iefix') format('embedded-opentype'), \n         url('myfont-webfont.woff') format('woff'), \n         url('myfont-webfont.ttf')  format('truetype'),\n         url('myfont-webfont.svg#svgFontName') format('svg')",
            "position": {
              "start": {
                "line": 3,
                "column": 5
              },
              "end": {
                "line": 6,
                "column": 61
              }
            }
          }
        ],
        "position": {
          "start": {
            "line": 1,
            "column": 1
          },
          "end": {
            "line": 7,
            "column": 6
          }
        }
      }
    ]
  }
}

[conradz]

@smykes not sure, the code that you have seems to work ok when I try it (although it is missing the nameSpaceCSS function; I tried an identity function). Also the JSON AST that you provided stringifies correctly. Could you make a full example (including JS and CSS) that crashes when run in node, possibly on gist.github.com?

[smykes]

@conradz this got taken out of r1 for me and has been pushed to r2. I have put in a workaround for my code for now, after hardening or r1 I am going to revisit it and I will get back to you.

[conradz]

OK, closing for now as it seems to not be a problem in css, but feel free to still provide more feedback or reopen if it does seem to be a bug in this module.