Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: plotly/plotly.R
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: alyst/plotly
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 4 commits
  • 2 files changed
  • 1 contributor

Commits on May 18, 2016

  1. don't import plyr (seems unused)

    @alyst
    alyst committed May 18, 2016
    Configuration menu
    Copy the full SHA
    e77fd87 View commit details
    Browse the repository at this point in the history

  2. refactor traces (fixes #418) 

    - separate the grouping of data into traces (this now done
  with the help of dplyr::group_by()) from traces creating and
  applying styles(brushes) to each resulting trace
- replace `tracify()` by `generate_traces()`,
  replace `colorize/symbolize()` by `color/symbol_brush()`
- import dplyr
    @alyst
    alyst committed May 18, 2016
    Configuration menu
    Copy the full SHA
    e9ebd17 View commit details
    Browse the repository at this point in the history

  3. support "group" property to break lines 

    "group" property doesn't create any new trace, but if the trace
mode is "lines", NA points are inserted at the "group" boundaries
within the same trace. That is interpreted as the polyline break by
plotlyjs.
    @alyst
    alyst committed May 18, 2016
    Configuration menu
    Copy the full SHA
    2c5064d View commit details
    Browse the repository at this point in the history

  4. add "legenditem" property 

    legenditem allows to group multiply traces into one legend item
    @alyst
    alyst committed May 18, 2016
    Configuration menu
    Copy the full SHA
    078d380 View commit details
    Browse the repository at this point in the history
Loading