Skip to content

Conversation

@wenzhixin
Copy link
Owner

Refs wenzhixin/bootstrap-table#7994

  • Implement groupBy functionality with custom formatter
  • Add sortable columns for shape, color, and area
  • Include checkbox selection for state field

@smillove
Copy link
Contributor

smillove commented Oct 7, 2025 via email

@wenzhixin wenzhixin force-pushed the feature/group-by branch 2 times, most recently from 51b44d5 to 0770bcd Compare October 7, 2025 02:41
@wenzhixin wenzhixin requested a review from Copilot October 7, 2025 02:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the group-by-v2 example by replacing HTML data attributes with JavaScript configuration options and adding enhanced functionality. The changes modernize the implementation to use programmatic configuration instead of declarative HTML attributes.

  • Migrated from HTML data attributes to JavaScript-based table configuration
  • Added custom group formatter with styling and checkbox selection functionality
  • Implemented sortable columns for improved user interaction

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

* Implement groupBy functionality with custom formatter
* Add sortable columns for shape, color, and area
* Include checkbox selection for state field
@wenzhixin wenzhixin requested a review from Copilot October 7, 2025 02:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@wenzhixin wenzhixin merged commit ed82721 into develop Oct 7, 2025
1 check passed
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