Skip to content
This repository
tree: 27538a8528
Fetching contributors…

Cannot retrieve contributors at this time

file 95 lines (66 sloc) 2.866 kb

Contributing to Flot

We welcome all contributions, but following these guidelines results in less work for us, and a faster and better response.

Issues

Issues are not a way to ask general questions about Flot. If you see unexpected behavior but are not 100% certain that it is a bug, please try posting to the forum first, and confirm that what you see is really a Flot problem before creating a new issue for it.

When reporting a bug, please include a working demonstration of the problem, if possible, or at least a clear description of the options you're using and the environment (browser and version, jQuery version, other libraries) that you're running under.

If you have suggestions for new features, or changes to existing ones, we'd love to hear them! Please submit each suggestion as a separate new issue.

If you would like to work on an existing issue, please make sure it is not already assigned to someone else. If an issue is assigned to someone, that person has already started working on it. So, pick unassigned issues to prevent duplicated efforts.

Pull Requests

To make merging as easy as possible, please keep these rules in mind:

  1. Divide larger changes into a series of small, logical commits with descriptive messages.

  2. Format your code according to the style guidelines below.

  3. Rebase against master, if necessary, before submitting your pull request.

Flot Style Guidelines

Flot follows the jQuery Core Style Guidelines, with the following updates and exceptions:

Spacing

Do not add horizontal space around parameter lists, loop definitions, or array/object indices. For example:

    for ( var i = 0; i < data.length; i++ ) {   // This block is wrong!
        if ( data[ i ] > 1 ) {
            data[ i ] = 2;
        }
    }

    for (var i = 0; i < data.length; i++) {     // This block is correct!
        if (data[i] > 1) {
            data[i] = 2;
        }
    }

Comments

Use // for all comments except the header at the top of a file or inline include.

All // comment blocks should have an empty line above and below them. For example:

    var a = 5;

    // We're going to loop here
    // TODO: Make this loop faster, better, stronger!

    for (var x = 0; x < 10; x++) {}

Wrapping

Block comments should be wrapped at 80 characters.

Code should attempt to wrap at 80 characters, but may run longer if wrapping would hurt readability more than having to scroll horizontally. This is a judgement call made on a situational basis.

Statements containing complex logic should not be wrapped arbitrarily if they do not exceed 80 characters. For example:

    if (a == 1 &&       // This block is wrong!
        b == 2 &&
        c == 3) {}

    if (a == 1 && b == 2 && c == 3) {}      // This block is correct!
Something went wrong with that request. Please try again.