Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds a null check before accessing a table row header's classList during column resize logic to prevent JavaScript errors when IsDetails is true. File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Pull request overview
Fixes a JavaScript runtime error in the Table component when column resizing is enabled alongside detail rows (IsDetails=true), and updates the package version.
Changes:
- Add a null-guard when resizing columns to avoid accessing
classListon a missing cell element. - Bump
BootstrapBlazorpackage version to10.5.0-beta04.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/BootstrapBlazor/Components/Table/Table.razor.js | Prevents a null dereference during resize handling when some rows don’t have a cell at the resized column index. |
| src/BootstrapBlazor/BootstrapBlazor.csproj | Updates the package version metadata. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7813 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 764 764
Lines 34128 34128
Branches 4697 4697
=========================================
Hits 34128 34128
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Link issues
fixes #7802
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
Bug Fixes: