Skip to content

enhance: render Others slice with secondary foreground color - #2757

Open
JC-Chung wants to merge 2 commits into
sourcegit-scm:developfrom
JC-Chung:enhance/pie-chart-others-color
Open

JC-Chung wants to merge 2 commits into
sourcegit-scm:developfrom
JC-Chung:enhance/pie-chart-others-color

Conversation

@JC-Chung

@JC-Chung JC-Chung commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Changes

  • Render the Others slice with secondaryForeground (Brush.FG2) instead of the next palette color. When the number of authors above 1% is a multiple of 10, the palette index wraps around to 0, making Others the same Orange as the adjacent top-author slice — the two blend together with no visible boundary.
  • Draw a 1px separator line at the top of the pie (the head-to-tail seam) via a new BorderBrush property bound to Brush.Window. Without it, when all authors are above 1% and the count is 11/21/31..., the last author's slice is also Orange and adjacent to the top author's slice, which the Others fix cannot cover.
  • Move var brush = s_brushes[brushIndex] into the percent > 0.01 branch.
  • Rename _lastHoverred to _lastHovered (no behavior change).

@JC-Chung
JC-Chung marked this pull request as draft October 6, 2026 09:20
@JC-Chung
JC-Chung force-pushed the enhance/pie-chart-others-color branch 2 times, most recently from 77cd029 to 709d2e3 Compare October 7, 2026 02:18
@JC-Chung
JC-Chung force-pushed the enhance/pie-chart-others-color branch from 709d2e3 to f405579 Compare October 7, 2026 02:20
@JC-Chung
JC-Chung marked this pull request as ready for review October 7, 2026 02:26

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant