Skip to content
This repository has been archived by the owner on Jul 16, 2018. It is now read-only.

Latest commit

 

History

History
42 lines (25 loc) · 1.83 KB

CONTRIBUTING.md

File metadata and controls

42 lines (25 loc) · 1.83 KB
Raw

Contribution Guidelines

Reporting issues

  • Search for existing issues. Please check to see if someone else has reported the same issue in our bug tracker.
  • Share as much information as possible. Include operating system and version, browser and version. Also, include steps to reproduce the bug.

Code Style

JavaScript

JS files must pass JSHint using the provided .jshintrc settings.

Additionally, JS files need to be run through JSBeautify with the provided .jsbeautifyrc.

TL;DR – Run grunt before pushing a commit. It will validate and beautify your JS.

Variable Naming

  • lowerCamelCase General variables
  • UpperCamelCase Constructor functions

HTML

  • 2 space indentation
  • Class names use hypenated case (e.g, my-class-name)

LESS / CSS

  • 2 space indentation
  • Always a space after a property's colon (e.g, display: block; and not display:block;)
  • End all lines with a semi-colon
  • For multiple, comma-separated selectors, place each selector on it's own line

Testing

Any patch should be tested in as many of our supported browsers as possible. Obviously, access to all devices is rare, so just aim for the best coverage possible. At a minimum please test in all available desktop browsers.

Pull requests

  • Try not to pollute your pull request with unintended changes – keep them simple and small. If possible, squash your commits.
  • Try to share which browsers and devices your code has been tested in before submitting a pull request.