Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideCorrects the image assets path in the SvgEditor initialization by removing an extra trailing slash and aligns the csproj asset settings accordingly to ensure proper resource loading. Class diagram for SvgEditor initialization options updateclassDiagram
class SvgEditorOptions {
canvasName: string
lang: string
allowInitialUserOverride: bool
imgPath: string
showGrid: bool
extPath: string
noDefaultExtensions: bool
}
SvgEditorOptions : imgPath updated to "./_content/BootstrapBlazor.SvgEditor/editor/images"
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 a bug in the SvgEditor component by correcting the images path configuration and increments the package version.
- Removes trailing slash from the
imgPathconfiguration to fix image loading issues - Updates package version from 9.0.3 to 9.0.4
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| SvgEditor.razor.js | Corrects the imgPath configuration by removing trailing slash |
| BootstrapBlazor.SvgEditor.csproj | Increments version number to 9.0.4 |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| lang: "en", | ||
| allowInitialUserOverride: false, | ||
| imgPath: "./_content/BootstrapBlazor.SvgEditor/editor/images/", | ||
| imgPath: "./_content/BootstrapBlazor.SvgEditor/editor/images", |
There was a problem hiding this comment.
The PR title contains a spelling error: 'iamges' should be 'images'. While this doesn't affect the code functionality, it's worth noting for consistency.
Link issues
fixes #553
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
Bug Fixes: