From 5977350c511924e9dc6804fa939f9ca36ac24e38 Mon Sep 17 00:00:00 2001 From: Paul Marbach Date: Fri, 5 Dec 2025 13:24:11 -0500 Subject: [PATCH] updates from review --- packages/grafana-data/src/types/icon.ts | 2 + public/app/core/icons/cached.json | 2 + .../PanelDataPane/PanelDataSidebar.tsx | 2 + .../PanelDataPane/QueryTransformList.tsx | 41 +++++++++++++------ public/locales/en-US/grafana.json | 8 ++-- 5 files changed, 38 insertions(+), 17 deletions(-) diff --git a/packages/grafana-data/src/types/icon.ts b/packages/grafana-data/src/types/icon.ts index 8701ac7fc7b..6b940dd5934 100644 --- a/packages/grafana-data/src/types/icon.ts +++ b/packages/grafana-data/src/types/icon.ts @@ -41,6 +41,7 @@ export const availableIconsIndex = { asserts: true, 'expand-arrows': true, 'expand-arrows-alt': true, + 'expand-alt': true, at: true, ai: true, 'ai-pointer': true, @@ -87,6 +88,7 @@ export const availableIconsIndex = { 'comments-alt': true, compass: true, 'compress-arrows': true, + 'compress-alt': true, copy: true, 'corner-up-left': true, 'corner-up-right': true, diff --git a/public/app/core/icons/cached.json b/public/app/core/icons/cached.json index 163a3e7f9f9..ca852df5c22 100644 --- a/public/app/core/icons/cached.json +++ b/public/app/core/icons/cached.json @@ -56,6 +56,7 @@ "unicons/comment-alt-share", "unicons/comments-alt", "unicons/compass", + "unicons/compress-alt", "unicons/copy", "unicons/corner-down-right-alt", "unicons/corner-up-left", @@ -74,6 +75,7 @@ "unicons/exchange-alt", "unicons/exclamation-circle", "unicons/exclamation-triangle", + "unicons/expand-alt", "unicons/external-link-alt", "unicons/eye", "unicons/eye-slash", diff --git a/public/app/features/dashboard-scene/panel-edit/PanelDataPane/PanelDataSidebar.tsx b/public/app/features/dashboard-scene/panel-edit/PanelDataPane/PanelDataSidebar.tsx index 872f8091266..b9bcfe8efd1 100644 --- a/public/app/features/dashboard-scene/panel-edit/PanelDataPane/PanelDataSidebar.tsx +++ b/public/app/features/dashboard-scene/panel-edit/PanelDataPane/PanelDataSidebar.tsx @@ -102,6 +102,7 @@ export function PanelDataSidebar({ return selectedQueryTransform; }, [transformPickerIndex, selectedQueryTransform, allItems]); const selectedItem = useMemo(() => allItems.find((item) => item.id === selectedId), [allItems, selectedId]); + const shouldShowAlerting = useMemo(() => tabs.some((tab) => tab.tabId === TabId.Alert), [tabs]); const updateQuerySelectionOnStateChange = useCallback( (index: number) => { @@ -329,6 +330,7 @@ export function PanelDataSidebar({ transformItems={transformItems} selectedId={selectedId} sidebarSize={sidebarState.size} + hasAlerting={shouldShowAlerting} onResizeSidebar={(size: SidebarSize) => setSidebarState((prevState) => ({ ...prevState, size }))} onCollapseSidebar={() => setSidebarState((prevState) => ({ ...prevState, collapsed: true }))} onSelect={(id) => { diff --git a/public/app/features/dashboard-scene/panel-edit/PanelDataPane/QueryTransformList.tsx b/public/app/features/dashboard-scene/panel-edit/PanelDataPane/QueryTransformList.tsx index 852d8795400..591065a106c 100644 --- a/public/app/features/dashboard-scene/panel-edit/PanelDataPane/QueryTransformList.tsx +++ b/public/app/features/dashboard-scene/panel-edit/PanelDataPane/QueryTransformList.tsx @@ -4,7 +4,7 @@ import { HTMLAttributes, memo, useCallback, useEffect, useMemo, useRef, useState import { GrafanaTheme2 } from '@grafana/data'; import { t } from '@grafana/i18n'; -import { Button, Icon, ScrollContainer, Stack, useStyles2 } from '@grafana/ui'; +import { Button, Icon, IconButton, ScrollContainer, Stack, useStyles2 } from '@grafana/ui'; import { ExpressionQueryType } from 'app/features/expressions/types'; import { AddDataItemMenu } from './AddDataItemMenu'; @@ -25,6 +25,7 @@ interface QueryTransformListProps { sidebarSize: SidebarSize; transformItems: TransformItem[]; selectedId: string | null; + hasAlerting?: boolean; onSelect: (id: string) => void; onAddQuery: (index?: number) => void; onAddFromSavedQueries: (index?: number) => void; @@ -50,6 +51,7 @@ export const QueryTransformList = memo( allItems, selectedId, sidebarSize, + hasAlerting, onSelect, onAddQuery, onAddFromSavedQueries, @@ -357,17 +359,30 @@ export const QueryTransformList = memo(
setHovered(null)}>
- {/* eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions */} -
onResizeSidebar(sidebarSize === SidebarSize.Mini ? SidebarSize.Full : SidebarSize.Mini)} - > - - - {t('dashboard-scene.query-transform-list.header', 'Pipeline flow')} - -
+ + onResizeSidebar(sidebarSize === SidebarSize.Mini ? SidebarSize.Full : SidebarSize.Mini)} + className={styles.rotate90} + name={sidebarSize === SidebarSize.Mini ? 'expand-alt' : 'compress-alt'} + /> + {t('dashboard-scene.query-transform-list.header', 'Pipeline')} + + {hasAlerting && ( + + )}