Skip to content
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

Update templates to use recommended way of getting grid columns for Blazor #719

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

skrustev
Copy link
Member

@skrustev skrustev commented Sep 10, 2024

@MarielaTihova
Copy link
Contributor

- Issue 1
(also observed in prod, but the same console errors there appear as warnings):

When we want to show a chart, clicking on the chart button produces these console errors for sample /grids/grid/finjs

image

@MarielaTihova
Copy link
Contributor

MarielaTihova commented Sep 17, 2024

- Issue 2: 
(also observed in prod as console errors)

When exapnding rows, errors are shown in the console. The issue seems to be happening only the first time a row is being expanded. NOT observed when being collapsed. Also NOT observed when expanding the child elements of an already expanded row.

Sample: /grids/hierarchical-grid/data-summary-template

image

@MarielaTihova
Copy link
Contributor

- Issue 2: 
(also observed in prod as console errors)

When exapnding rows, errors are shown in the console. The issue seems to be happening only the first time a row is being expanded. NOT observed when being collapsed. Also NOT observed when expanding the child elements of an already expanded row.

Sample: /grids/hierarchical-grid/data-summary-template

image

An issue for this bug has been created in Blazor examples repo. Marking this PR as 'verfied' and ready to be merged, as the issue is not being raised by it. It is only observed in the same Blazor examples.

@dkamburov dkamburov merged commit b83ee58 into 24.1.x Nov 5, 2024
7 checks passed
@dkamburov dkamburov deleted the skrastev/grid-columns-proper branch November 5, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants