Skip to content

Conversation

@markallenramirez
Copy link
Contributor

No description provided.

@markallenramirez markallenramirez self-assigned this Nov 28, 2025
Copilot AI review requested due to automatic review settings November 28, 2025 05:05
@markallenramirez markallenramirez requested a review from a team as a code owner November 28, 2025 05:05
Copilot finished reviewing on behalf of markallenramirez November 28, 2025 05:07
Copy link
Contributor

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 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-enabledrow-alternation-enabled
  • Added missing rowAlternationEnabled and hoverStateEnabled properties 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

@markallenramirez markallenramirez changed the title DataGrid Row Demo - Fix the hover issue in Angular DataGrid Row Demo - Fix issues Nov 28, 2025
Copilot AI review requested due to automatic review settings December 1, 2025 04:47
Copilot finished reviewing on behalf of markallenramirez December 1, 2025 04:48
Copy link
Contributor

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 3 out of 3 changed files in this pull request and generated no new comments.

@markallenramirez markallenramirez force-pushed the fix_demo_datagrid/25_2 branch 2 times, most recently from 3e93a5f to 37abd37 Compare December 1, 2025 09:34
Copilot AI review requested due to automatic review settings December 1, 2025 09:34
Copilot finished reviewing on behalf of markallenramirez December 1, 2025 09:44
Copy link
Contributor

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 3 out of 3 changed files in this pull request and generated no new comments.

@markallenramirez markallenramirez merged commit 4ec17e1 into DevExpress:25_2 Dec 2, 2025
139 of 141 checks passed
@markallenramirez markallenramirez deleted the fix_demo_datagrid/25_2 branch December 2, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants