Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: stylelint-config-eyeo/index.js

Issue 29643562: Issue 5689 - Add rules for CSS declaration order to stylelint-config-eyeo (Closed) Base URL: https://hg.adblockplus.org/codingtools
Left Patch Set: Created Dec. 18, 2017, 9:14 a.m.
Right Patch Set: Fixed trailing whitespace errors Created Feb. 8, 2018, 2:11 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « stylelint-config-eyeo/css-properties-order.js ('k') | stylelint-config-eyeo/package.json » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-present eyeo GmbH 3 * Copyright (C) 2006-present eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 25 matching lines...) Expand all
36 36
37 // Use a space after a property name’s colon 37 // Use a space after a property name’s colon
38 "declaration-colon-space-after": "always", 38 "declaration-colon-space-after": "always",
39 39
40 // Selectors and declarations should be on their own line 40 // Selectors and declarations should be on their own line
41 "selector-list-comma-newline-after": "always", 41 "selector-list-comma-newline-after": "always",
42 "declaration-block-semicolon-newline-after": "always-multi-line", 42 "declaration-block-semicolon-newline-after": "always-multi-line",
43 43
44 // Separate rules by an empty line 44 // Separate rules by an empty line
45 "rule-empty-line-before": ["always", { 45 "rule-empty-line-before": ["always", {
46 "ignore": ["after-comment", "first-nested"] 46 "ignore": ["after-comment"],
47 "except": ["first-nested"]
47 }], 48 }],
48 49
49 // Use double over single quotation marks 50 // Use double over single quotation marks
50 "string-quotes": "double", 51 "string-quotes": "double",
51 52
52 // CSS color values should be specified in hexadecimal where possible 53 // CSS color values should be specified in hexadecimal where possible
53 "color-named": "never", 54 "color-named": "never",
54 55
55 // Use short hexadecimal notation where possible 56 // Use short hexadecimal notation where possible
56 "color-hex-length": "short", 57 "color-hex-length": "short",
(...skipping 11 matching lines...) Expand all
68 // Avoid qualifying ID and class names with type selectors 69 // Avoid qualifying ID and class names with type selectors
69 "selector-no-qualifying-type": [true, { 70 "selector-no-qualifying-type": [true, {
70 "ignore": ["attribute"] 71 "ignore": ["attribute"]
71 }], 72 }],
72 73
73 // CSS rule declaration order should follow the WordPress CSS 74 // CSS rule declaration order should follow the WordPress CSS
74 // Coding Standards 75 // Coding Standards
75 "order/properties-order": require("./css-properties-order") 76 "order/properties-order": require("./css-properties-order")
76 } 77 }
77 }; 78 };
LEFTRIGHT

Powered by Google App Engine
This is Rietveld