Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds support for configuring the UniverSheet RibbonType in the UniverSheets sample and updates localized documentation text accordingly. Updated class diagram for UniverSheet usage with RibbonTypeclassDiagram
class UniverSheetsSample {
- UniverSheet _sheetExcel
+ OnPushExcelData()
+ OnSaveExcelData()
+ OnReadyAsync()
}
class UniverSheet {
+ UniverSheetRibbonType RibbonType
+ OnReadyAsync()
}
class UniverSheetRibbonType {
}
<<enumeration>> UniverSheetRibbonType
UniverSheetsSample --> UniverSheet : uses
UniverSheet --> UniverSheetRibbonType : has
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 adds support for the RibbonType parameter to the UniverSheet component, allowing users to configure the toolbar style. The changes include documentation updates in both English and Chinese localization files, and a demonstration of the new parameter in the sample component.
- Added localization entries to document the new
RibbonTypeparameter - Updated the UniverSheet demo to showcase the
RibbonTypeparameter usage with a default value
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/BootstrapBlazor.Server/Locales/zh-CN.json | Added Chinese localization for RibbonType parameter description |
| src/BootstrapBlazor.Server/Locales/en-US.json | Added English localization for RibbonType parameter description |
| src/BootstrapBlazor.Server/Components/Samples/UniverSheets.razor | Added RibbonType parameter to the demo component and included descriptive text |
💡 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 #7387 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 748 748
Lines 32793 32793
Branches 4551 4551
=========================================
Hits 32793 32793
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 #7386
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
Document and demonstrate the UniverSheet RibbonType option in the sample page.
New Features:
Documentation: