Linear charts & remove duplicate days buttons (#97110)

* Make the charts linear * Remove the X days buttons if X days is the same as whole campaign length
Automattic · Dec 5, 2024 · 158fcb1 · 158fcb1
1 parent edbbf0a
commit 158fcb1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/client/my-sites/promote-post-i2/components/campaign-item-details/index.tsx b/client/my-sites/promote-post-i2/components/campaign-item-details/index.tsx
@@ -467,7 +467,7 @@ export default function CampaignItemDetails( props: Props ) {
 			],
 		},
 		{
-			condition: activeDays >= 7,
+			condition: activeDays > 7,
 			controls: [
 				{
 					onClick: () => updateChartParams( ChartSourceDateRanges.LAST_7_DAYS ),
@@ -477,7 +477,7 @@ export default function CampaignItemDetails( props: Props ) {
 			],
 		},
 		{
-			condition: activeDays >= 14,
+			condition: activeDays > 14,
 			controls: [
 				{
 					onClick: () => updateChartParams( ChartSourceDateRanges.LAST_14_DAYS ),
@@ -487,7 +487,7 @@ export default function CampaignItemDetails( props: Props ) {
 			],
 		},
 		{
-			condition: activeDays >= 30,
+			condition: activeDays > 30,
 			controls: [
 				{
 					onClick: () => updateChartParams( ChartSourceDateRanges.LAST_30_DAYS ),

diff --git a/...mote-post-i2/components/campaign-stats-line-chart/index.tsx/campaign-stats-line-chart.tsx b/...mote-post-i2/components/campaign-stats-line-chart/index.tsx/campaign-stats-line-chart.tsx
@@ -142,8 +142,8 @@ const CampaignStatsLineChart = ( { data, source, resolution }: GraphProps ) => {
 						return gradient;
 					},
 					paths: ( u: uPlot, seriesIdx: number, idx0: number, idx1: number ) => {
-						const { spline } = uPlot.paths;
-						return spline?.()( u, seriesIdx, idx0, idx1 ) || null;
+						const { linear } = uPlot.paths;
+						return linear?.()( u, seriesIdx, idx0, idx1 ) || null;
 					},
 					points: {
 						show: false,