From 2b57ebaffdf4d59bf0bfcf930996df73762596db Mon Sep 17 00:00:00 2001
From: Ville Brofeldt <33317356+villebro@users.noreply.github.com>
Date: Fri, 23 Apr 2021 19:29:51 +0300
Subject: [PATCH] feat(native-filters): add tooltip to control values (#14312)

---
 .../FiltersConfigForm/ControlItems.tsx             | 14 ++++++++++++--
 .../src/filters/components/Select/controlPanel.ts  |  2 +-
 2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/ControlItems.tsx b/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/ControlItems.tsx
index 7deee0d85..48c074cc2 100644
--- a/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/ControlItems.tsx
+++ b/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/ControlItems.tsx
@@ -16,7 +16,10 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-import { CustomControlItem } from '@superset-ui/chart-controls';
+import {
+  CustomControlItem,
+  InfoTooltipWithTrigger,
+} from '@superset-ui/chart-controls';
 import React, { FC } from 'react';
 import { Checkbox } from 'src/common/components';
 import { FormInstance } from 'antd/lib/form';
@@ -77,7 +80,14 @@ const ControlItems: FC<ControlItemsProps> = ({
                 forceUpdate();
               }}
             >
-              {controlItem.config.label}
+              {controlItem.config.label}{' '}
+              {controlItem.config.description && (
+                <InfoTooltipWithTrigger
+                  placement="top"
+                  label={controlItem.config.name}
+                  tooltip={controlItem.config.description}
+                />
+              )}
             </Checkbox>
           </StyledCheckboxFormItem>
         ))}
diff --git a/superset-frontend/src/filters/components/Select/controlPanel.ts b/superset-frontend/src/filters/components/Select/controlPanel.ts
index c4a560b8b..c41283dc5 100644
--- a/superset-frontend/src/filters/components/Select/controlPanel.ts
+++ b/superset-frontend/src/filters/components/Select/controlPanel.ts
@@ -75,7 +75,7 @@ const config: ControlPanelConfig = {
               default: enableEmptyFilter,
               renderTrigger: true,
               description: t(
-                'When selection is empty, should an always false filter event be emitted',
+                'When selection is empty, an always false filter event be emitted',
               ),
             },
           },