From 5bdb7745040e826dc36c6742f05952d69530ee0d Mon Sep 17 00:00:00 2001 From: "Michael S. Molina" <70410625+michael-s-molina@users.noreply.github.com> Date: Fri, 30 Jun 2023 08:18:47 -0300 Subject: [PATCH] fix: Metric is truncated in tooltip (#24555) --- .../src/components/labelUtils.tsx | 7 +++++++ superset-frontend/src/components/Tooltip/index.tsx | 7 +++++++ .../components/controls/MetricControl/AdhocMetric.js | 3 +-- 3 files changed, 15 insertions(+), 2 deletions(-) diff --git a/superset-frontend/packages/superset-ui-chart-controls/src/components/labelUtils.tsx b/superset-frontend/packages/superset-ui-chart-controls/src/components/labelUtils.tsx index accaaf9f49..1bd49c40e0 100644 --- a/superset-frontend/packages/superset-ui-chart-controls/src/components/labelUtils.tsx +++ b/superset-frontend/packages/superset-ui-chart-controls/src/components/labelUtils.tsx @@ -31,6 +31,13 @@ const TooltipSectionWrapper = styled.div` &:not(:last-of-type) { margin-bottom: ${theme.gridUnit * 2}px; } + &:last-of-type { + display: -webkit-box; + -webkit-line-clamp: 40; + -webkit-box-orient: vertical; + overflow: hidden; + text-overflow: ellipsis; + } `} `; diff --git a/superset-frontend/src/components/Tooltip/index.tsx b/superset-frontend/src/components/Tooltip/index.tsx index 06469abd13..64af6b06a0 100644 --- a/superset-frontend/src/components/Tooltip/index.tsx +++ b/superset-frontend/src/components/Tooltip/index.tsx @@ -45,6 +45,13 @@ export const Tooltip = (props: TooltipProps) => { /> diff --git a/superset-frontend/src/explore/components/controls/MetricControl/AdhocMetric.js b/superset-frontend/src/explore/components/controls/MetricControl/AdhocMetric.js index 752fc457b9..e866093ffc 100644 --- a/superset-frontend/src/explore/components/controls/MetricControl/AdhocMetric.js +++ b/superset-frontend/src/explore/components/controls/MetricControl/AdhocMetric.js @@ -89,8 +89,7 @@ export default class AdhocMetric { } getDefaultLabel() { - const label = this.translateToSql({ useVerboseName: true }); - return label.length < 43 ? label : `${label.substring(0, 40)}...`; + return this.translateToSql({ useVerboseName: true }); } translateToSql(