Open
Conversation
mdd-remove-error-bars
There was a problem hiding this comment.
Pull Request Overview
This PR removes the entire series-error-bars sample directory from the data chart examples, effectively deleting the error bars functionality demo. This appears to be part of a cleanup effort to remove error bars features from the charting library.
- Removes complete sample implementation including TypeScript, HTML, and configuration files
- Deletes all build and development configuration (webpack, tsconfig, package.json)
- Eliminates documentation and setup files for the error bars sample
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| webpack.config.js | Removes webpack build configuration for the sample |
| tsconfig.json | Removes TypeScript compiler configuration |
| src/index.ts | Removes main TypeScript implementation with DataChartSeriesErrorBars class |
| src/index.css | Removes CSS styles file |
| sandbox.config.json | Removes sandbox configuration |
| package.json | Removes npm package configuration and dependencies |
| index.html | Removes HTML template with igc-data-chart component |
| ReadMe.md | Removes documentation and setup instructions |
| .prettierrc | Removes code formatting configuration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
remove-series-animations
HUSSAR-mtrela
requested changes
Sep 20, 2025
Collaborator
HUSSAR-mtrela
left a comment
There was a problem hiding this comment.
why do you need to remove series-animations and series-error-bars samples ?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
mdd-remove-error-bars