-
Notifications
You must be signed in to change notification settings - Fork 658
DataGrid Row Demo - Fix issues #31879
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
DataGrid Row Demo - Fix issues #31879
Conversation
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.
Pull request overview
This PR fixes inconsistencies in the DataGrid Row demo across Angular and Vue frameworks. The title mentions fixing a "hover issue in Angular," which refers to the missing hoverStateEnabled property that enables hover state functionality in the DataGrid. Additionally, a typo in the Vue demo's row-alternation-enabled property has been corrected.
Key Changes
- Fixed typo in Vue demo:
row-alternatin-enabled→row-alternation-enabled - Added missing
rowAlternationEnabledandhoverStateEnabledproperties to Angular demo to match other framework implementations
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| apps/demos/Demos/DataGrid/Row/Vue/App.vue | Fixed spelling of row-alternation-enabled property (was missing 'o' in 'alternation') |
| apps/demos/Demos/DataGrid/Row/Angular/app/app.component.html | Added rowAlternationEnabled and hoverStateEnabled properties to align with React and jQuery demos |
02c602f to
928bac5
Compare
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.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
3e93a5f to
37abd37
Compare
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.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
37abd37 to
30088d3
Compare
No description provided.