-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Reorganization [part 1] #1
Conversation
- run with `npm run watch` - supports 'require' (for test_dashboard)
@@ -1 +1,6 @@ | |||
node_modules | |||
|
|||
build/* | |||
!build/README.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd vote for ignoring dist/*
as well and have us git add -f dist/
when we bump the version.
Thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes lets do this - we can always turn it off if we don't like it
💃 |
Legend item wrap with layout.legend.orientation = h
* treat zero as valid fixes #2660 * PR suggestions: toBe instead of toEqual
Fix for number 0 treated as invalid in hover (#1)
* Fix bug in scattergl plot * Added tests for scattergl plot update
Pull all changes from original plotly to the fork
Fixed the per-element opacity on Surface plots
Update to your pull-request for making es5/require work.
@bpostlethwaite @alexcjohnson
The most interesting part of this PR is under
tasks/
, please see comments below.