Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdjusts compact table cell styling so that only single-line form controls get a forced compact height, allowing multi-row controls (like textareas) to render correctly in table compact mode. 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
This PR fixes compact (.table-sm) in-cell editing styling so that form controls with a rows attribute (typically <textarea rows="..."> generated from TableColumn.Rows) are no longer forced into a single-line height, restoring expected multi-line behavior. It also bumps the package version to the next beta.
Changes:
- Update compact table CSS to avoid applying fixed height to
.form-controlelements that specifyrows. - Bump
BootstrapBlazorpackage version from10.7.2-beta01to10.7.2-beta02.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/BootstrapBlazor/Components/Table/Table.razor.scss |
Excludes [rows] controls from compact-mode fixed height so multi-line textareas can size correctly. |
src/BootstrapBlazor/BootstrapBlazor.csproj |
Increments library version to 10.7.2-beta02. |
💡 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 #8107 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 766 766
Lines 34204 34204
Branches 4696 4696
=========================================
Hits 34204 34204
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Link issues
fixes #8106
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
Bug Fixes: