Skip to content
This repository was archived by the owner on Jun 8, 2026. It is now read-only.

Add MarkerSize property documentation to chart markers topics - #2003

Merged
gedinakova merged 8 commits into
vnextfrom
copilot/add-marker-size-property
May 19, 2026
Merged

gedinakova merged 8 commits into
vnextfrom
copilot/add-marker-size-property

Conversation

Copilot AI commented May 13, 2026 •

Copy link
Copy Markdown
Contributor

Documents the newly introduced MarkerSize property across all language variants (en, jp, kr) of the chart markers feature topic.

Changes

  • New section: Chart Marker Size — explains that MarkerSize sets exact device-independent pixel dimensions on any MarkerSeries-derived series, overriding template-driven sizing; resetting to NaN restores default behavior
  • Interactive sample — added the scatter-marker-options sample to the Marker Size section, demonstrating MarkerSize on scatter series with an interactive editor
  • Multi-platform code examples — covers Blazor (Razor), React (TSX), and Web Components (HTML):
<IgbScatterLineSeries MarkerSize="30" MarkerType="Circle" ... />
<IgrScatterLineSeries markerSize={30} markerType="Circle" ... />
<igc-scatter-line-series marker-size="30" marker-type="Circle" ...></igc-scatter-line-series>
  • BubbleSeries caveat — notes that MarkerSize has no effect on bubble radius, which remains data/scale-driven
  • Updated frontmatter — mentionedTypes extended with MarkerSeries, ScatterLineSeries, ScatterSplineSeries, ScatterSeries, LineSeries, SplineSeries; _keywords updated to include "Marker Size"
  • API References — added MarkerSize, UseLightweightMarkers, and XamDataChart

Copilot AI linked an issue May 13, 2026 that may be closed by this pull request
@gedinakova
gedinakova requested a review from emiliano-luna May 13, 2026 18:07
Copilot AI changed the title [WIP] Add the new Marker size property documentation Add MarkerSize property documentation to chart markers topics May 13, 2026
Copilot AI requested a review from gedinakova May 13, 2026 18:10
Comment thread doc/en/components/charts/features/chart-markers.md Outdated
Comment thread doc/kr/components/charts/features/chart-markers.md Outdated
Comment thread doc/jp/components/charts/features/chart-markers.md Outdated
@gedinakova gedinakova assigned mtsvyatkova and unassigned gedinakova May 14, 2026
@gedinakova
gedinakova marked this pull request as ready for review May 14, 2026 08:09
@mtsvyatkova

Copy link
Copy Markdown
Contributor

@copilot Update this PR to include a sample for the newly documented MarkerSize property.

Use the sample added in IgniteUI/igniteui-xplat-examples#1013, specifically: samples/charts/data-chart/scatter-marker-options.json

Add the sample to the Marker Size section so the documentation shows a working example based on scatter-marker-options.json. This sample includes the interactive Marker Size editor and demonstrates markerSize on scatter series, which matches the property being documented here.

Please keep the documentation aligned with the sample implementation from the examples PR.

Copilot AI commented May 15, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Update this PR to include a sample for the newly documented MarkerSize property.

Use the sample added in IgniteUI/igniteui-xplat-examples#1013, specifically: samples/charts/data-chart/scat...

Added the scatter-marker-options sample to the Marker Size section in all three language files (en, jp, kr) in commit dc8ed8a. The sample is placed after the code examples and before the BubbleSeries caveat note.

Copilot AI requested a review from mtsvyatkova May 15, 2026 09:38
mtsvyatkova
mtsvyatkova previously approved these changes May 15, 2026
@mtsvyatkova mtsvyatkova added the status: verified ✅ The PR is tested and ready for a merge label May 15, 2026
@ChronosSF

Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request

…conflicts

Co-authored-by: ChronosSF <2188411+ChronosSF@users.noreply.github.com>

Copilot AI commented May 19, 2026

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

Resolved in commit e6ff350. The conflicts arose from vnext adding a new "Checkmark Marker Type" section while our branch added "Marker Size". Both sections are now preserved, and mentionedTypes and API references from both branches have been merged.

Copilot AI requested a review from ChronosSF May 19, 2026 13:34
@gedinakova
gedinakova merged commit 06fe0ad into vnext May 19, 2026
7 checks passed
@gedinakova
gedinakova deleted the copilot/add-marker-size-property branch May 19, 2026 13:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status: verified ✅ The PR is tested and ready for a merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add the new Marker size property

6 participants