Compare commits

...

4 Commits

Author SHA1 Message Date
Kristina Durivage
6902c3bb9c add back floating utils after bad rebase
Some checks failed
CodeQL checks / Detect whether code changed (push) Has been cancelled
CodeQL checks / Analyze (actions) (push) Has been cancelled
CodeQL checks / Analyze (go) (push) Has been cancelled
CodeQL checks / Analyze (javascript) (push) Has been cancelled
2025-10-21 18:14:40 -05:00
Kristina Durivage
4b1894c305 Merge branch 'main' of https://github.com/grafana/grafana into gtk-grafana/annotations/anchored-tooltips
# Conflicts:
#	public/app/plugins/panel/timeseries/plugins/AnnotationsPlugin2.tsx
#	public/app/plugins/panel/timeseries/plugins/annotations2/AnnotationMarker2.tsx
#	public/app/plugins/panel/timeseries/plugins/annotations2/AnnotationTooltip2.tsx
2025-10-21 17:51:23 -05:00
Galen
e01856dc06 chore: move close icon last, set focus ref 2025-10-07 15:53:12 -05:00
Galen
12249d3211 feat: support anchored annotation tooltips on click 2025-10-07 15:37:46 -05:00
5 changed files with 97 additions and 18 deletions

View File

@@ -67,7 +67,7 @@ export const AnnotationsPlugin2 = ({
const [plot, setPlot] = useState<uPlot>();
const [portalRoot] = useState(() => getPortalContainer());
const [annoIdx, setAnnoIdx] = useState<string | undefined>();
const styles = useStyles2(getStyles);
const getColorByName = useTheme2().visualization.getColorByName;
@@ -209,6 +209,11 @@ export const AnnotationsPlugin2 = ({
}
}, [annos, plot]);
// Set active annotation tooltip state
const setAnnotationIndex = useCallback((annoIdx: string | undefined) => {
setAnnoIdx(annoIdx);
}, []);
if (plot) {
let markers = annos.flatMap((frame, frameIdx) => {
let vals = getVals(frame);
@@ -245,10 +250,20 @@ export const AnnotationsPlugin2 = ({
// @TODO: Reset newRange after annotation is saved
if (isVisible) {
let isWip = frame.meta?.custom?.isWip;
const isWip = frame.meta?.custom?.isWip;
const setAnnotation = (active: boolean) => {
if (active) {
setAnnotationIndex(`${frameIdx}:${i}`);
} else {
setAnnotationIndex(undefined);
}
};
markers.push(
<AnnotationMarker2
pinAnnotation={setAnnotation}
isPinned={annoIdx === `${frameIdx}:${i}`}
showOnHover={!annoIdx}
frame={frame}
annoIdx={i}
annoVals={vals}
@@ -277,6 +292,7 @@ const getStyles = () => ({
position: 'absolute',
width: 0,
height: 0,
border: 'none',
borderLeft: '5px solid transparent',
borderRight: '5px solid transparent',
borderBottomWidth: '5px',
@@ -284,11 +300,16 @@ const getStyles = () => ({
transform: 'translateX(-50%)',
cursor: 'pointer',
zIndex: 1,
padding: 0,
background: 'none',
}),
annoRegion: css({
border: 'none',
position: 'absolute',
height: '5px',
cursor: 'pointer',
zIndex: 1,
padding: 0,
background: 'none',
}),
});

View File

@@ -5,7 +5,7 @@ import { useAsyncFn, useClickAway } from 'react-use';
import { AnnotationEventUIModel, GrafanaTheme2, dateTimeFormat, systemDateFormats } from '@grafana/data';
import { Trans, t } from '@grafana/i18n';
import { Button, Field, Stack, TextArea, usePanelContext, useStyles2 } from '@grafana/ui';
import { Button, Field, IconButton, Stack, TextArea, usePanelContext, useStyles2 } from '@grafana/ui';
import { Form } from 'app/core/components/Form/Form';
import { TagFilter } from 'app/core/components/TagFilter/TagFilter';
import { annotationServer } from 'app/features/annotations/api';
@@ -15,6 +15,7 @@ interface Props {
annoIdx: number;
timeZone: string;
dismiss: () => void;
isPinned: boolean;
}
interface AnnotationEditFormDTO {
@@ -22,7 +23,7 @@ interface AnnotationEditFormDTO {
tags: string[];
}
export const AnnotationEditor2 = ({ annoVals, annoIdx, dismiss, timeZone, ...otherProps }: Props) => {
export const AnnotationEditor2 = ({ annoVals, annoIdx, dismiss, timeZone, isPinned, ...otherProps }: Props) => {
const styles = useStyles2(getStyles);
const { onAnnotationCreate, onAnnotationUpdate } = usePanelContext();
@@ -77,6 +78,18 @@ export const AnnotationEditor2 = ({ annoVals, annoIdx, dismiss, timeZone, ...oth
: t('timeseries.annotation-editor2.add-annotation', 'Add annotation')}
</div>
<div>{time}</div>
{isPinned && (
<IconButton
name={'times'}
size={'sm'}
onClick={(e) => {
// Don't trigger onClick
e.stopPropagation();
dismiss();
}}
tooltip={t('timeseries.annotation-editor2.tooltip-close', 'Close')}
/>
)}
</Stack>
</div>
<Form<AnnotationEditFormDTO>
@@ -88,6 +101,7 @@ export const AnnotationEditor2 = ({ annoVals, annoIdx, dismiss, timeZone, ...oth
<>
<div className={styles.content}>
<Field
autoFocus={true}
label={t('timeseries.annotation-editor2.label-description', 'Description')}
invalid={!!errors.description}
error={errors?.description?.message}

View File

@@ -8,7 +8,7 @@ import { createPortal } from 'react-dom';
import { DataFrame, GrafanaTheme2, LinkModel } from '@grafana/data';
import { selectors } from '@grafana/e2e-selectors';
import { TimeZone } from '@grafana/schema';
import { floatingUtils, useStyles2 } from '@grafana/ui';
import { ClickOutsideWrapper, floatingUtils, useStyles2 } from '@grafana/ui';
import { getDataLinks } from 'app/plugins/panel/status-history/utils';
import { AnnotationEditor2 } from './AnnotationEditor2';
@@ -23,6 +23,9 @@ interface AnnoBoxProps {
timeZone: TimeZone;
exitWipEdit?: null | (() => void);
portalRoot: HTMLElement;
pinAnnotation: (pin: boolean) => void;
isPinned: boolean;
showOnHover: boolean;
}
const STATE_DEFAULT = 0;
@@ -38,11 +41,15 @@ export const AnnotationMarker2 = ({
exitWipEdit,
timeZone,
portalRoot,
pinAnnotation,
showOnHover,
isPinned,
}: AnnoBoxProps) => {
const styles = useStyles2(getStyles);
const placement = 'bottom';
const [state, setState] = useState(exitWipEdit != null ? STATE_EDITING : STATE_DEFAULT);
const [isHovering, setIsHovering] = useState(false);
const { refs, floatingStyles } = useFloating({
open: true,
placement,
@@ -51,6 +58,10 @@ export const AnnotationMarker2 = ({
strategy: 'fixed',
});
const onClose = () => {
pinAnnotation(false);
setIsHovering(false);
};
const links: LinkModel[] = [];
if (STATE_HOVERED) {
@@ -60,43 +71,52 @@ export const AnnotationMarker2 = ({
}
const contents =
state === STATE_HOVERED ? (
(isPinned && !(state === STATE_EDITING)) || (showOnHover && isHovering && !(state === STATE_EDITING)) ? (
<AnnotationTooltip2
annoIdx={annoIdx}
annoVals={annoVals}
timeZone={timeZone}
onClose={onClose}
isPinned={isPinned}
onEdit={() => setState(STATE_EDITING)}
links={links}
/>
) : state === STATE_EDITING ? (
<AnnotationEditor2
isPinned={isPinned}
annoIdx={annoIdx}
annoVals={annoVals}
timeZone={timeZone}
dismiss={() => {
exitWipEdit?.();
setState(STATE_DEFAULT);
onClose();
}}
/>
) : null;
return (
<div
<button
ref={refs.setReference}
className={className}
style={style!}
onMouseEnter={() => state !== STATE_EDITING && setState(STATE_HOVERED)}
onMouseLeave={() => state !== STATE_EDITING && setState(STATE_DEFAULT)}
onFocus={() => setIsHovering(true)}
onBlur={() => setIsHovering(false)}
onClick={() => pinAnnotation(true)}
onMouseEnter={() => showOnHover && setIsHovering(true)}
onMouseLeave={() => setIsHovering(false)}
data-testid={selectors.pages.Dashboard.Annotations.marker}
>
{contents &&
createPortal(
<div ref={refs.setFloating} className={styles.annoBox} style={floatingStyles} data-testid="annotation-marker">
{contents}
<ClickOutsideWrapper includeButtonPress={false} useCapture={true} onClick={() => pinAnnotation(false)}>
{contents}
</ClickOutsideWrapper>
</div>,
portalRoot
)}
</div>
</button>
);
};

View File

@@ -11,25 +11,32 @@ interface Props {
annoVals: Record<string, any[]>;
annoIdx: number;
timeZone: string;
isPinned: boolean;
onClose: () => void;
onEdit: () => void;
links?: LinkModel[];
}
const retFalse = () => false;
export const AnnotationTooltip2 = ({ annoVals, annoIdx, timeZone, onEdit, links = [] }: Props) => {
export const AnnotationTooltip2 = ({ annoVals, annoIdx, timeZone, isPinned, onClose, onEdit, links = [] }: Props) => {
const annoId = annoVals.id?.[annoIdx];
const styles = useStyles2(getStyles);
const focusRef = React.useRef<HTMLButtonElement | null>(null);
const { canEditAnnotations = retFalse, canDeleteAnnotations = retFalse, onAnnotationDelete } = usePanelContext();
const dashboardUID = annoVals.dashboardUID?.[annoIdx];
// grafana can be configured to load alert rules from loki. Those annotations cannot be edited or deleted. The id being 0 is the best indicator the annotation came from loki
const canEdit = annoId !== 0 && canEditAnnotations(dashboardUID);
const canDelete = annoId !== 0 && canDeleteAnnotations(dashboardUID) && onAnnotationDelete != null;
React.useEffect(() => {
if (isPinned) {
focusRef.current?.focus();
}
}, [isPinned]);
const timeFormatter = (value: number) =>
dateTimeFormat(value, {
format: systemDateFormats.fullDate,
@@ -75,10 +82,11 @@ export const AnnotationTooltip2 = ({ annoVals, annoIdx, timeZone, onEdit, links
</span>
{time}
</div>
{(canEdit || canDelete) && (
<div className={styles.editControls}>
{(canEdit || canDelete || isPinned) && (
<div className={styles.controls}>
{canEdit && (
<IconButton
ref={focusRef}
name={'pen'}
size={'sm'}
onClick={onEdit}
@@ -87,12 +95,26 @@ export const AnnotationTooltip2 = ({ annoVals, annoIdx, timeZone, onEdit, links
)}
{canDelete && (
<IconButton
ref={canEdit ? null : focusRef}
name={'trash-alt'}
size={'sm'}
onClick={() => onAnnotationDelete(annoId)}
tooltip={t('timeseries.annotation-tooltip2.tooltip-delete', 'Delete')}
/>
)}
{isPinned && (
<IconButton
ref={canEdit || canDelete ? null : focusRef}
name={'times'}
size={'sm'}
onClick={(e) => {
// Don't trigger onClick
e.stopPropagation();
onClose();
}}
tooltip={t('timeseries.annotation-tooltip2.tooltip-close', 'Close')}
/>
)}
</div>
)}
</Stack>
@@ -138,7 +160,7 @@ const getStyles = (theme: GrafanaTheme2) => ({
color: theme.colors.text.primary,
fontWeight: 400,
}),
editControls: css({
controls: css({
display: 'flex',
'> :last-child': {
marginLeft: 0,

View File

@@ -13137,9 +13137,11 @@
"label-tags": "Tags",
"placeholder-add-tags": "Add tags",
"save": "Save",
"saving": "Saving"
"saving": "Saving",
"tooltip-close": "Close"
},
"annotation-tooltip2": {
"tooltip-close": "Close",
"tooltip-delete": "Delete",
"tooltip-edit": "Edit"
},