Contents

module SCSSLint
  class Linter::DeclarationOrder < Linter
    include LinterRegistry

    DECLARATION_ORDER = [
      Sass::Tree::ExtendNode,
      Sass::Tree::PropNode,
      Sass::Tree::RuleNode,
    ]

    def visit_rule(node)
      children = node.children.select { |node| important_node?(node) }.
                               map { |node| node.class }

      sorted_children = children.sort do |a, b|
        DECLARATION_ORDER.index(a) <=> DECLARATION_ORDER.index(b)
      end

      if children != sorted_children
        add_lint(node.children.first)
      end

      yield # Continue linting children
    end

    def description
      'Rule sets should start with @extend declarations, followed by ' <<
      'properties and nested rule sets, in that order'
    end

  private

    def important_node?(node)
      DECLARATION_ORDER.include? node.class
    end
  end
end

Version data entries

4 entries across 4 versions & 1 rubygems

Version	Path
scss-lint-0.11.1	lib/scss_lint/linter/declaration_order.rb
scss-lint-0.10.1	lib/scss_lint/linter/declaration_order.rb
scss-lint-0.10.0	lib/scss_lint/linter/declaration_order.rb
scss-lint-0.9.0	lib/scss_lint/linter/declaration_order.rb