Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Override default heatmap col+row labels #794

Merged
merged 5 commits into from
Jan 18, 2015
Merged

Override default heatmap col+row labels #794

merged 5 commits into from
Jan 18, 2015

Conversation

hhravn
Copy link
Contributor

@hhravn hhravn commented Dec 15, 2014

@gordonwoodhull
Copy link
Contributor

Note, in case you had any doubts, the CI failure here is not your fault.

d3 is a bit of a moving target, and last week's update broke our tests.

@gordonwoodhull
Copy link
Contributor

And, thanks!

@gordonwoodhull
Copy link
Contributor

Looks good! About all it needs is a couple tests exercising the new functions.

@gordonwoodhull gordonwoodhull changed the title Override default col+row labels Override default heatmap col+row labels Dec 29, 2014
@gordonwoodhull gordonwoodhull modified the milestone: v2.0 Jan 17, 2015
gordonwoodhull added a commit that referenced this pull request Jan 18, 2015
also brought contribution in line with jscs styles

Thanks @hhravn!
@gordonwoodhull gordonwoodhull merged commit c01cbd4 into dc-js:master Jan 18, 2015
gordonwoodhull added a commit that referenced this pull request Apr 3, 2015
reverted in PR #794, needed in PR #798
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants