Skip to content

Commit c579d04

Browse files
committed
Tooltip on the time picker when it's disabled
1 parent 74aba15 commit c579d04

File tree

1 file changed

+8
-4
lines changed
  • apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.query

1 file changed

+8
-4
lines changed

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.query/route.tsx

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ import type { AITimeFilter } from "./types";
6767
import { formatQueryStats } from "./utils";
6868
import { requireUser } from "~/services/session.server";
6969
import parse from "parse-duration";
70+
import { SimpleTooltip } from "~/components/primitives/Tooltip";
7071

7172
async function hasQueryAccess(
7273
userId: string,
@@ -370,7 +371,7 @@ const QueryEditorForm = forwardRef<
370371
// This disables the time filter UI since the user is filtering in their query
371372
const queryHasTriggeredAt = /\bWHERE\b[\s\S]*\btriggered_at\b/i.test(query);
372373

373-
374+
374375

375376
// Expose methods to parent for external query setting (history, AI, examples)
376377
useImperativeHandle(
@@ -439,9 +440,12 @@ const QueryEditorForm = forwardRef<
439440
}
440441
</Select>
441442
{queryHasTriggeredAt ? (
442-
<Button variant="tertiary/small" disabled={true} type="button">
443-
Set in query
444-
</Button>
443+
<SimpleTooltip
444+
button={<Button variant="tertiary/small" disabled={true} type="button">
445+
Set in query
446+
</Button>}
447+
content="Your query includes a WHERE clause with triggered_at so this filter is disabled."
448+
/>
445449
) : (
446450
<TimeFilter
447451
defaultPeriod={DEFAULT_PERIOD}

0 commit comments

Comments
 (0)