Bug 512330 - [css][lint] Add quickfix to disable csslint rule for file
Summary: [css][lint] Add quickfix to disable csslint rule for file
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: JS Tools (show other bugs)
Version: 13.0   Edit
Hardware: PC Windows 7
: P2 enhancement (vote)
Target Milestone: 15.0   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on: 490392 508847
Blocks:
  Show dependency tree
 
Reported: 2017-02-16 16:32 EST by Curtis Windatt CLA
Modified: 2017-03-29 13:38 EDT (History)
1 user (show)

See Also:

Attachments
Fix is CSSLint supported the option (10.06 KB, patch)
2017-02-22 15:22 EST, Curtis Windatt CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Curtis Windatt CLA 2017-02-16 16:32:15 EST 
We can support the same 'disable rule in file' quickfix for CSS that we did for JS.  This could be very useful as many CSSLint rules are based on subjective style choices.
Comment 1 Curtis Windatt CLA 2017-02-22 15:22:16 EST 
Issue blocking this is that our current version of CSSLint does not support any inline directives to disable rules. Looks like it was added over a year ago.

https://github.com/CSSLint/csslint/wiki/Ignoring-parts-of-CSS-during-linting

There is a difference between the CSSLint and ESLint directives.  In CSSLint you can disable a rule only for a single line, or disable all lint rules for a block.
Comment 2 Curtis Windatt CLA 2017-02-22 15:22:56 EST 
Created attachment 266948 [details]
Fix is CSSLint supported the option