Skip to content

Commit cc755af

Browse files
committed
Merge remote-tracking branch 'origin/main' into feat/filter-search
Signed-off-by: Adam Setch <[email protected]>
2 parents 7b315e7 + db3cb19 commit cc755af

18 files changed

+1226
-943
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@
7878
"@discordapp/twemoji": "16.0.1",
7979
"@electron/notarize": "3.1.0",
8080
"@primer/css": "22.0.2",
81-
"@primer/octicons-react": "19.16.0",
81+
"@primer/octicons-react": "19.17.0",
8282
"@primer/primitives": "11.1.0",
8383
"@primer/react": "36.27.0",
8484
"@tailwindcss/postcss": "4.1.13",

pnpm-lock.yaml

Lines changed: 6 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/renderer/components/filters/FilterSection.tsx

Lines changed: 15 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import { AppContext } from '../../context/App';
77
import type { FilterSettingsState, FilterValue } from '../../types';
88
import type { Filter } from '../../utils/notifications/filters';
99
import { Checkbox } from '../fields/Checkbox';
10-
import { Tooltip } from '../fields/Tooltip';
1110
import { Title } from '../primitives/Title';
1211
import { RequiresDetailedNotificationWarning } from './RequiresDetailedNotificationsWarning';
1312

@@ -34,22 +33,21 @@ export const FilterSection = <T extends FilterValue>({
3433

3534
return (
3635
<fieldset id={id}>
37-
<Stack align="baseline" direction="horizontal" gap="condensed">
38-
<Title icon={icon}>{title}</Title>
39-
{tooltip && (
40-
<Tooltip
41-
name={`tooltip-${id}`}
42-
tooltip={
43-
<Stack direction="vertical" gap="condensed">
44-
{tooltip}
45-
{filter.requiresDetailsNotifications && (
46-
<RequiresDetailedNotificationWarning />
47-
)}
48-
</Stack>
49-
}
50-
/>
51-
)}
52-
</Stack>
36+
<Title
37+
icon={icon}
38+
tooltip={
39+
tooltip && (
40+
<>
41+
{tooltip}
42+
{filter.requiresDetailsNotifications && (
43+
<RequiresDetailedNotificationWarning />
44+
)}
45+
</>
46+
)
47+
}
48+
>
49+
{title}
50+
</Title>
5351

5452
<Stack
5553
direction={layout}

src/renderer/components/filters/SearchFilter.tsx

Lines changed: 26 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import {
1717
hasIncludeSearchFilters,
1818
SEARCH_PREFIXES,
1919
} from '../../utils/notifications/filters/search';
20-
import { Tooltip } from '../fields/Tooltip';
2120
import { Title } from '../primitives/Title';
2221
import { RequiresDetailedNotificationWarning } from './RequiresDetailedNotificationsWarning';
2322

@@ -146,33 +145,33 @@ export const SearchFilter: FC = () => {
146145

147146
return (
148147
<fieldset id={fieldsetId}>
149-
<Stack align="baseline" direction="horizontal" gap="condensed">
150-
<Title icon={SearchIcon}>Search</Title>
151-
<Tooltip
152-
name="tooltip-filter-actors"
153-
tooltip={
154-
<Stack direction="vertical" gap="condensed">
155-
<Text>Filter notifications by:</Text>
156-
<Box className="pl-4">
157-
<Stack direction="vertical" gap="condensed">
158-
<Stack direction="horizontal" gap="condensed">
159-
<PersonIcon size={Size.SMALL} />
160-
Author (author:handle)
161-
</Stack>
162-
<Stack direction="horizontal" gap="condensed">
163-
<OrganizationIcon size={Size.SMALL} />
164-
Organization (org:name)
165-
</Stack>
166-
<Stack direction="horizontal" gap="condensed">
167-
<RepoIcon size={Size.SMALL} /> Repository (repo:fullname)
168-
</Stack>
148+
<Title
149+
icon={SearchIcon}
150+
tooltip={
151+
<Stack direction="vertical" gap="condensed">
152+
<Text>Filter notifications by:</Text>
153+
<Box className="pl-4">
154+
<Stack direction="vertical" gap="condensed">
155+
<Stack direction="horizontal" gap="condensed">
156+
<PersonIcon size={Size.SMALL} />
157+
Author (author:handle)
169158
</Stack>
170-
</Box>
171-
<RequiresDetailedNotificationWarning />
172-
</Stack>
173-
}
174-
/>
175-
</Stack>
159+
<Stack direction="horizontal" gap="condensed">
160+
<OrganizationIcon size={Size.SMALL} />
161+
Organization (org:name)
162+
</Stack>
163+
<Stack direction="horizontal" gap="condensed">
164+
<RepoIcon size={Size.SMALL} /> Repository (repo:fullname)
165+
</Stack>
166+
</Stack>
167+
</Box>
168+
<RequiresDetailedNotificationWarning />
169+
</Stack>
170+
}
171+
>
172+
Search
173+
</Title>
174+
176175
<Stack direction="vertical" gap="condensed">
177176
<Stack
178177
align="center"

0 commit comments

Comments
 (0)