-
Notifications
You must be signed in to change notification settings - Fork 12k
docs: improve README text and add tutorial links #12145
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,7 +2,11 @@ | |
| <a href="https://www.chartjs.org/" target="_blank"> | ||
| <img src="https://www.chartjs.org/media/logo-title.svg" alt="https://www.chartjs.org/"><br/> | ||
| </a> | ||
| Simple yet flexible JavaScript charting for designers & developers | ||
| <a href="https://github.com/chartjs/Chart.js/actions?query=workflow%3ACI+branch%3Amaster"> | ||
| <img alt="GitHub Workflow Status" src="https://img.shields.io/github/actions/workflow/status/chartjs/Chart.js/ci.yml?branch=master&style=flat-square"> | ||
| </a> ← Current CI build status | ||
| Simple, flexible, and powerful JavaScript charting library for designers and developers. | ||
|
|
||
| </p> | ||
|
|
||
| <p align="center"> | ||
|
|
@@ -26,13 +30,22 @@ All the links point to the new version 4 of the lib. | |
| * [Developers](https://www.chartjs.org/docs/latest/developers/index) | ||
| * [Popular Extensions](https://github.com/chartjs/awesome) | ||
| * [Samples](https://www.chartjs.org/samples/) | ||
| * [Tutorials and Examples](https://www.chartjs.org/docs/latest/samples/) | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The link above already points to this |
||
|
|
||
| In case you are looking for an older version of the docs, you will have to specify the specific version in the url like this: [https://www.chartjs.org/docs/2.9.4/](https://www.chartjs.org/docs/2.9.4/) | ||
|
|
||
| ## Contributing | ||
| ### Contributing Started | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is not a sub section of the documentation so the way was correct |
||
|
|
||
| Instructions on building and testing Chart.js can be found in [the documentation](https://www.chartjs.org/docs/master/developers/contributing.html#building-and-testing). Before submitting an issue or a pull request, please take a moment to look over the [contributing guidelines](https://www.chartjs.org/docs/master/developers/contributing) first. For support, please post questions on [Stack Overflow](https://stackoverflow.com/questions/tagged/chart.js) with the `chart.js` tag. | ||
|
|
||
| ### Guides | ||
|
|
||
| * [General](https://www.chartjs.org/docs/latest/general/data-structures) | ||
| * [Configuration](https://www.chartjs.org/docs/latest/configuration/index) | ||
| * [Charts](https://www.chartjs.org/docs/latest/charts/line) | ||
| * [Axes](https://www.chartjs.org/docs/latest/axes/index) | ||
| * [Developers](https://www.chartjs.org/docs/latest/developers/index) | ||
|
|
||
|
Comment on lines
+41
to
+48
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. All these links are already described above |
||
| ## License | ||
|
|
||
| Chart.js is available under the [MIT license](LICENSE.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.
Not sure if we want to be dependant on a third party site in the readme, but if so the formatting seems incorrect