-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Add Jspreadsheet CE, JSuites, CalendarJS, and TabularJS #1941
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
Conversation
📝 WalkthroughWalkthroughUpdates README.md to add CalendarJS, Jspreadsheet CE, TabularJS, and JSuites across Calendars, Data Grids, and Dates/UI Libraries; removes the jspreadsheet entry from Data Grids. Changes
Sequence Diagram(s)(omitted — changes are documentation-only and do not introduce new control flow) Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@README.md`:
- Around line 1305-1306: Remove the duplicate "Jspreadsheet CE" list entry (the
second occurrence shown as "[Jspreadsheet CE] – Open source JavaScript
spreadsheet...") and keep a single entry; update the remaining "[Jspreadsheet
CE]" description (the earlier entry) if you want to broaden or correct wording,
and ensure only one "[Jspreadsheet CE]" link appears in the list alongside
"[TabularJS]".
- Around line 2007-2008: The JSuites entry ("*
[JSuites](https://github.com/jsuites/jsuites) – Collection of UI components and
utilities...") is currently placed under "UI Libraries built on Bootstrap" but
JSuites is framework-agnostic and not Bootstrap-based; move that markdown list
item out of the Bootstrap-specific section and insert it into the general "UI
Libraries" section above so the entry appears alongside other framework-agnostic
libraries.
|
@Plekest The CalendarJS link URL was wrong (Page Not Found) and I think I found the correct library. You can review the changes. |
|
@Plekest Thanks for the contribution. |
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.