From 6f69ecd08747df09611f43314405d46f530edb57 Mon Sep 17 00:00:00 2001
From: Marshall Polaris <marshall@pol.rs>
Date: Mon, 26 Sep 2022 22:03:53 -0700
Subject: [PATCH] Fix a dumb bug

---
 web/components/charts/contract/binary.tsx         | 2 +-
 web/components/charts/contract/choice.tsx         | 9 +++++----
 web/components/charts/contract/numeric.tsx        | 2 +-
 web/components/charts/contract/pseudo-numeric.tsx | 2 +-
 4 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/web/components/charts/contract/binary.tsx b/web/components/charts/contract/binary.tsx
index d9a94788..d0843dde 100644
--- a/web/components/charts/contract/binary.tsx
+++ b/web/components/charts/contract/binary.tsx
@@ -31,7 +31,7 @@ export const BinaryContractChart = (props: {
   const isMobile = useIsMobile(800)
   const containerRef = useRef<HTMLDivElement>(null)
   const width = useElementWidth(containerRef) ?? 0
-  const height = props.height ?? isMobile ? 150 : 250
+  const height = props.height ?? (isMobile ? 150 : 250)
   const xScale = scaleTime(getDateRange(contract), [0, width - MARGIN_X])
   const yScale = scaleLinear([0, 1], [height - MARGIN_Y, 0])
   return (
diff --git a/web/components/charts/contract/choice.tsx b/web/components/charts/contract/choice.tsx
index 635b0475..444c0273 100644
--- a/web/components/charts/contract/choice.tsx
+++ b/web/components/charts/contract/choice.tsx
@@ -12,7 +12,8 @@ import { useElementWidth } from 'web/hooks/use-element-width'
 
 const getMultiChartData = (
   contract: FreeResponseContract | MultipleChoiceContract,
-  bets: Bet[]
+  bets: Bet[],
+  topN: number
 ) => {
   const { answers, totalBets, outcomeType } = contract
 
@@ -32,7 +33,7 @@ const getMultiChartData = (
   const trackedAnswers = sortBy(
     validAnswers,
     (answer) => -1 * getOutcomeProbability(contract, answer.id)
-  ).slice(0, 10)
+  ).slice(0, topN)
 
   const points: MultiPoint[] = []
 
@@ -78,13 +79,13 @@ export const ChoiceContractChart = (props: {
 }) => {
   const { contract, bets } = props
   const data = useMemo(
-    () => getMultiChartData(contract, bets),
+    () => getMultiChartData(contract, bets, 6),
     [contract, bets]
   )
   const isMobile = useIsMobile(800)
   const containerRef = useRef<HTMLDivElement>(null)
   const width = useElementWidth(containerRef) ?? 0
-  const height = props.height ?? isMobile ? 150 : 250
+  const height = props.height ?? (isMobile ? 150 : 250)
   const xScale = scaleTime(getDateRange(contract), [0, width - MARGIN_X])
   const yScale = scaleLinear([0, 1], [height - MARGIN_Y, 0])
   return (
diff --git a/web/components/charts/contract/numeric.tsx b/web/components/charts/contract/numeric.tsx
index e45d7f54..6adc52f0 100644
--- a/web/components/charts/contract/numeric.tsx
+++ b/web/components/charts/contract/numeric.tsx
@@ -28,7 +28,7 @@ export const NumericContractChart = (props: {
   const isMobile = useIsMobile(800)
   const containerRef = useRef<HTMLDivElement>(null)
   const width = useElementWidth(containerRef) ?? 0
-  const height = props.height ?? isMobile ? 150 : 250
+  const height = props.height ?? (isMobile ? 150 : 250)
   const maxY = max(data.map((d) => d[1])) as number
   const xScale = scaleLinear(
     [contract.min, contract.max],
diff --git a/web/components/charts/contract/pseudo-numeric.tsx b/web/components/charts/contract/pseudo-numeric.tsx
index a5366c82..b81d2808 100644
--- a/web/components/charts/contract/pseudo-numeric.tsx
+++ b/web/components/charts/contract/pseudo-numeric.tsx
@@ -36,7 +36,7 @@ export const PseudoNumericContractChart = (props: {
   const isMobile = useIsMobile(800)
   const containerRef = useRef<HTMLDivElement>(null)
   const width = useElementWidth(containerRef) ?? 0
-  const height = props.height ?? isMobile ? 150 : 250
+  const height = props.height ?? (isMobile ? 150 : 250)
   const scaleType = contract.isLogScale ? scaleLog : scaleLinear
   const xScale = scaleTime(getDateRange(contract), [0, width - MARGIN_X])
   const yScale = scaleType([contract.min, contract.max], [height - MARGIN_Y, 0])