Skip to content

Commit a4c8e65

Browse files
fix: remove white outline from chart datapoints (#63)
* fix: remove white outline from chart datapoints Remove the white stroke around scatter plot datapoints for a cleaner look. Set stroke to 'none' and stroke-width to 0 for both normal and hover states. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * style: fix formatting in footer.tsx Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Update chart-rendering.ts --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent f149486 commit a4c8e65

4 files changed

Lines changed: 7 additions & 7 deletions

File tree

packages/app/src/components/footer/footer.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,8 @@ export const Footer = () => {
6464
className="text-sm text-muted-foreground max-w-xs text-center md:text-left"
6565
>
6666
Continuous open-source inference benchmarking. Real-world, reproducible, auditable
67-
performance data trusted by trillion dollar AI infrastructure operators like
68-
OpenAI, Oracle, Microsoft, etc.
67+
performance data trusted by trillion dollar AI infrastructure operators like OpenAI,
68+
Oracle, Microsoft, etc.
6969
</p>
7070
</div>
7171

packages/app/src/lib/chart-rendering.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ describe('SHAPE_CONFIG', () => {
4545

4646
it('hover sizes are larger than normal sizes', () => {
4747
expect(HOVER_POINT_SIZE).toBeGreaterThan(POINT_SIZE);
48-
expect(HOVER_STROKE_WIDTH).toBeGreaterThan(STROKE_WIDTH);
48+
expect(HOVER_STROKE_WIDTH).toBeGreaterThanOrEqual(STROKE_WIDTH);
4949
expect(SHAPE_CONFIG.default.hover.r).toBeGreaterThan(SHAPE_CONFIG.default.normal.r);
5050
});
5151
});

packages/app/src/lib/chart-rendering.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@ import type * as d3 from 'd3';
33
import { formatNumber } from '@/lib/utils';
44

55
// Point shape constants
6-
export const POINT_SIZE = 4; // Base size (radius for circle, half-width/height for rectangle)
6+
export const POINT_SIZE = 3.5; // Base size (radius for circle, half-width/height for rectangle)
77
export const HOVER_POINT_SIZE = 6; // Hover size
8-
export const STROKE_WIDTH = 1.5; // Normal stroke width
9-
export const HOVER_STROKE_WIDTH = 2; // Hover stroke width
8+
export const STROKE_WIDTH = 0; // Normal stroke width
9+
export const HOVER_STROKE_WIDTH = 0; // Hover stroke width
1010
export const HIT_AREA_RADIUS = 12; // Invisible hit area for easier interaction
1111

1212
// Triangle path for BF16 precision

packages/app/src/lib/d3-chart/layers/scatter-points.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ export function renderScatterPoints<T extends { precision: string; x: number; y:
5353
.append(shapeConfig.type)
5454
.attr('class', 'visible-shape')
5555
.attr('fill', config.getColor(d))
56-
.attr('stroke', 'white')
56+
.attr('stroke', 'none')
5757
.attr('cursor', 'pointer') as d3.Selection<
5858
SVGCircleElement | SVGRectElement | SVGPathElement,
5959
unknown,

0 commit comments

Comments
 (0)