Support for @mixin, @include and @extend

pkg/csslib/lib/src/css_printer.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 part of csslib.visitor;
 
 /**
  * Visitor that produces a formatted string representation of the CSS tree.
  */
 class CssPrinter extends Visitor {

[nweiz, 2013/09/18 22:40:54]

If this is printing the CSS tree, why does it need to print mixins? If it's
printing a string representation of the SCSS tree, the name and documentation
should indicate that.

[terry, 2013/10/09 03:40:33]

Good point done for completness @mixin and @include should occur if analyzer has
been run.
On 2013/09/18 22:40:54, nweiz wrote:

   StringBuffer _buff = new StringBuffer();
   bool prettyPrint = true;
 
   /**
    * Walk the [tree] Stylesheet. [pretty] if true emits line breaks, extra
    * spaces, friendly property values, etc., if false emits compacted output.
    */
   void visitTree(StyleSheet tree, {bool pretty: false}) {
     prettyPrint = pretty;
     _buff = new StringBuffer();
@@ skipping 159 matching lines @@
       }
     }
     emit(';');
   }
 
   void visitVarDefinitionDirective(VarDefinitionDirective node) {
     visitVarDefinition(node.def);
     emit(';$_newLine');
   }
 
+  void visitMixinRulesetDirective(MixinRulesetDirective node) {
+    emit('@mixin ${node.name} {');
+    for (var ruleset in node.rulesets) {
+      ruleset.visit(this);
+    }
+    emit('}');
+  }
+
+  void visitMixinDeclarationDirective(MixinDeclarationDirective node) {
+    emit('@mixin ${node.name} {\n');
+    visitDeclarationGroup(node.declarations);
+    emit('}');
+  }
+
+  /**
+   * Added optional newLine for handling @include at top-level vs/ inside of
+   * a declaration group.
+   */
+  void visitIncludeDirective(IncludeDirective node, [bool topLevel = true]) {
+    if (topLevel) emit(_newLine);    // Emit newline for top-level @include.

[nweiz, 2013/09/18 22:40:54]

Unnecessary spaces before the comment.

[terry, 2013/10/09 03:40:33]

Done.

+    emit('@include ${node.name}');
+    if (topLevel) emit(';');

[nweiz, 2013/09/18 22:40:54]

Why wouldn't non-top-level @includes have a semicolon?

[terry, 2013/10/09 03:40:33]

removed test.On 2013/09/18 22:40:54, nweiz wrote:

+  }
+
+  void visitContentDirective(ContentDirective node) {

[nweiz, 2013/09/18 22:40:54]

If you don't support content directives yet, just don't include the code.

[terry, 2013/10/09 03:40:33]

Place holder have errors for content directive in parser.  Don't want to forget
content printer.
On 2013/09/18 22:40:54, nweiz wrote:

+    // TODO(terry): TBD
+  }
+
   void visitRuleSet(RuleSet node) {
     emit("$_newLine");
     node._selectorGroup.visit(this);
     emit(" {$_newLine");
     node._declarationGroup.visit(this);
     emit("}");
   }
 
   void visitDeclarationGroup(DeclarationGroup node) {
     var declarations = node._declarations;
@@ skipping 25 matching lines @@
     node._expression.visit(this);
 
     emit("${importantAsString()}");
   }
 
   void visitVarDefinition(VarDefinition node) {
     emit("var-${node.definedName}: ");
     node._expression.visit(this);
   }
 
+  void visitIncludeMixinAtDeclaration(IncludeMixinAtDeclaration node) {
+    // Don't emit a new line we're inside of a declaration group.
+    visitIncludeDirective(node.include, false);
+  }
+
   void visitSelectorGroup(SelectorGroup node) {
     var selectors = node._selectors;
     var selectorsLength = selectors.length;
     for (var i = 0; i < selectorsLength; i++) {
       if (i > 0) emit(',$_sp');
       selectors[i].visit(this);
     }
   }
 
   void visitSimpleSelectorSequence(SimpleSelectorSequence node) {
@@ skipping 222 matching lines @@
 
   void visitWildcard(Wildcard node) {
     emit('*');
   }
 
   void visitDartStyleExpression(DartStyleExpression node) {
     // TODO(terry): TBD
     throw UnimplementedError;
   }
 }