Skip to content

Multiple Lines on Graph Functionality#24

Merged
Hermanya merged 6 commits into
chart-kit:masterfrom
mrfambo:master
Jul 12, 2018
Merged

Multiple Lines on Graph Functionality#24
Hermanya merged 6 commits into
chart-kit:masterfrom
mrfambo:master

Conversation

@mrfambo

@mrfambo mrfambo commented Jul 7, 2018

Copy link
Copy Markdown
Contributor

Hey,

I have created the multiple lines graph functionality. I hope its something valuable because after surfing a lot on web i found nothing interesting.

Comment thread package.json Outdated
"_main": "./node_modules/react-native-scripts/build/bin/crna-entry.js",
"main": "./index.js",
"main": "./node_modules/react-native-scripts/build/bin/crna-entry.js",
"_main": "./index.js",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.DS_Store and package-lock.json need to be removed. We already have yarn.lock

The package.json change needs to be removed. I know that it sucks, the example should really be in a subfolder, so that we don't have this problem.

Comment thread src/line-chart.js
paddingRight,
paddingTop,
data: data.datasets[0].data
// data: data.datasets[0].data

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And please remove this commented code.

@Hermanya

Hermanya commented Jul 7, 2018

Copy link
Copy Markdown
Contributor

I run your clone. The graph looks somewhat confusing. Could you share a screenshot of how you are using this graph? How do you show which line represents what?

@Hermanya Hermanya merged commit 465572f into chart-kit:master Jul 12, 2018
@Hermanya

Copy link
Copy Markdown
Contributor

Fixed all the minor things. @alimuqaddas congrats on your first Open Source Contribution 👍 thank you very much 😄
I'm still curious how you are using this graph in your app.

@trungnguyencmu

Copy link
Copy Markdown

do we have document for multiple lines on graph functionality?

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.

3 participants