From d7bacf96c7aaad22ec65d21e8df7d9efc8fa1985 Mon Sep 17 00:00:00 2001 From: Julius Volz Date: Mon, 9 Sep 2024 11:19:02 +0200 Subject: [PATCH] Remove commented-out and unused allowLineBreaks settings for binop explain Signed-off-by: Julius Volz --- .../pages/query/ExplainViews/BinaryExpr/VectorVector.tsx | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/web/ui/mantine-ui/src/pages/query/ExplainViews/BinaryExpr/VectorVector.tsx b/web/ui/mantine-ui/src/pages/query/ExplainViews/BinaryExpr/VectorVector.tsx index 1ba42310ef..6735d1320b 100644 --- a/web/ui/mantine-ui/src/pages/query/ExplainViews/BinaryExpr/VectorVector.tsx +++ b/web/ui/mantine-ui/src/pages/query/ExplainViews/BinaryExpr/VectorVector.tsx @@ -326,12 +326,8 @@ const explainError = ( const VectorVectorBinaryExprExplainView: FC< VectorVectorBinaryExprExplainViewProps > = ({ node, lhs, rhs }) => { - // TODO: Don't use Mantine's local storage as a one-off here. - // const [allowLineBreaks, setAllowLineBreaks] = useLocalStorage({ - // key: "queryPage.explain.binaryOperators.breakLongLines", - // defaultValue: true, - // }); - + // TODO: Don't use Mantine's local storage as a one-off here. Decide whether we + // want to keep Redux, and then do it only via one or the other everywhere. const [showSampleValues, setShowSampleValues] = useLocalStorage({ key: "queryPage.explain.binaryOperators.showSampleValues", defaultValue: false,