Class: SCSSLint::Linter::IdWithExtraneousSelector

Inherits:
SCSSLint::Linter
  • Object
show all
Includes:
SCSSLint::LinterRegistry
Defined in:
lib/scss_lint/linter/id_with_extraneous_selector.rb

Overview

Checks for a selector with an ID combined with some other selector.

Instance Attribute Summary

Attributes inherited from SCSSLint::Linter

#config, #engine, #lints

Instance Method Summary collapse

Methods included from SCSSLint::LinterRegistry

extract_linters_from, included

Methods inherited from SCSSLint::Linter

#initialize, #name, #run

Methods included from Utils

#extract_string_selectors, #node_siblings, #pluralize, #previous_node, #remove_quoted_strings, #same_position?

Methods included from SelectorVisitor

#visit_selector

Constructor Details

This class inherits a constructor from SCSSLint::Linter

Instance Method Details

#visit_simple_sequence(seq) ⇒ Object 



6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
# File 'lib/scss_lint/linter/id_with_extraneous_selector.rb', line 6

def visit_simple_sequence(seq)
  id_sel = seq.members.find { |simple| simple.is_a?(Sass::Selector::Id) }
  return unless id_sel

  can_be_simplified = seq.members.any? do |simple|
    !simple.is_a?(Sass::Selector::Id) &&
      !simple.is_a?(Sass::Selector::Pseudo)
  end
  return unless can_be_simplified

  # TODO: Sass::Selector::SimpleSequence#source_range sometimes lies about
  # its line, so reference `#line` directly
  add_lint(seq.line, "Selector `#{seq}` can be simplified to `#{id_sel}`, " \
                     'since IDs should be uniquely identifying')
end