You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+5-15Lines changed: 5 additions & 15 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -44,7 +44,7 @@ Guidelines for bug reports:
44
44
45
45
3.**Isolate the problem**— ideally create a [reduced test
46
46
case](http://css-tricks.com/6263-reduced-test-cases/) and a live example.
47
-
[This JS Bin](http://jsbin.com/EBAwOkOK/1) is a helpful template.
47
+
[This JS Bin](http://jsbin.com/lefey/1/edit?html,output) is a helpful template.
48
48
49
49
50
50
A good bug report shouldn't leave others needing to chase you up for more
@@ -154,26 +154,16 @@ license your work under the terms of the [MIT License](LICENSE.md).
154
154
155
155
### HTML
156
156
157
-
- Two spaces for indentation, never tabs.
158
-
- Double quotes only, never single quotes.
159
-
- Always use proper indentation.
157
+
[Adhere to the Code Guide.](http://codeguide.co/#html)
158
+
160
159
- Use tags and elements appropriate for an HTML5 doctype (e.g., self-closing tags).
161
160
- Use CDNs and HTTPS for third-party JS when possible. We don't use protocol-relative URLs in this case because they break when viewing the page locally via `file://`.
162
161
- Use [WAI-ARIA](https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA) attributes in documentation examples to promote accessibility.
163
162
164
163
### CSS
165
164
166
-
- CSS changes must be done in `.less` files first, never just in the compiled `.css` files.
167
-
- Adhere to the [CSS property order](http://markdotto.com/2011/11/29/css-property-order/).
168
-
- Multiple-line approach (one property and value per line).
169
-
- Always a space after a property's colon (e.g., `display: block;` and not `display:block;`).
170
-
- End all lines with a semi-colon.
171
-
- For multiple, comma-separated selectors, place each selector on its own line.
172
-
- Don't add vendor prefixed properties to their unprefixed counterparts (e.g., only `box-sizing` and not also include `-webkit-box-sizing`), as this is done automagically at build time.
173
-
- Attribute selectors, like `input[type="text"]` should always wrap the attribute's value in double quotes, for consistency and safety (see this [blog post on unquoted attribute values](http://mathiasbynens.be/notes/unquoted-attribute-values) that can lead to XSS attacks).
174
-
- Attribute selectors should only be used where absolutely necessary (e.g., form controls) and should be avoided on custom components for performance and explicitness.
175
-
- Series of classes for a component should include a base class (e.g., `.component`) and use the base class as a prefix for modifier and sub-components (e.g., `.component-lg`).
176
-
- Avoid inheritance and over nesting—use single, explicit classes whenever possible.
165
+
[Adhere to the Code Guide.](http://codeguide.co/#css)
166
+
177
167
- When feasible, default color palettes should comply with [WCAG color contrast guidelines](http://www.w3.org/TR/WCAG20/#visual-audio-contrast).
178
168
- Except in rare cases, don't remove default `:focus` styles (via e.g. `outline: none;`) without providing alternative styles. See [this A11Y Project post](http://a11yproject.com/posts/never-remove-css-outlines/) for more details.
0 commit comments