Add JavaScript Grids#4216
Conversation
Added a link to grid libraries organized by features and frameworks.
Removed duplicate entry for 'Grids' in the readme.
Updated the entry for grid libraries in the readme.
|
unicorn |
|
Hey,
Nice work. |
Nanboy-Ronan
left a comment
There was a problem hiding this comment.
Two things look unresolved against the template:
- The PR body leaves
Non-generated Markdown file in a GitHub repounchecked and says the list comes from YAML data files. The Awesome guidelines explicitly require a non-generated Markdown file, so this either needs to be changed or called out as something that needs maintainer approval before merge. - The description in the PR body starts with "A human-curated list of...", which describes the list instead of the project/theme. The entry description in the diff is much better:
Spreadsheet and data grid components for displaying tabular data on the web.I would use that style in the body/top description too.
Also, the earlier comment about the truncated SVAR DataGrid description should be addressed before this is ready.
Good catch, fixed.
Much better, fixed.
Fair enough. The README is no longer auto-generated.
Good call, fixed.
Done. |
|
Mechanically clean; two content points:
|
|
Adding to the curation points already raised:
|
Added a link to grid libraries organized by features and frameworks.
https://github.com/statico/awesome-javascript-grids#readme
Spreadsheet and data grid components for displaying tabular data on the web, with an interactive explorer and feature comparison companion site.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request
Add Name of List. It should not contain the wordAwesome.#readme.Requirements for your Awesome list
That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
awesome-linton your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.main, notmaster.awesome-name-of-list.# Awesome Name of List.awesome-list&awesomeas GitHub topics. I encourage you to add more relevant topics.Awesome Xand a logo withAwesome X. You can put the header image in a#(Markdown header) or<h1>.Contents, notTable of Contents.ContributingorFootnotessections.unicorn.contributing.md. The casing is up to you.Contributing, positioned at the top or bottom of the main content.Footnotessection at the bottom of the readme. The section should not be present in the Table of Contents.Example:
- [AVA](…) - JavaScript test runner.Node.js, notNodeJSornode.js.You can still use a CI for linting, but the badge has no value in the readme.
Inspired by awesome-fooorInspired by the Awesome projectkinda link at the top of the readme. The Awesome badge is enough.