Skip to content

Commit 5f08835

Browse files
committed
feedback
1 parent 2f41967 commit 5f08835

File tree

3 files changed

+24
-21
lines changed

3 files changed

+24
-21
lines changed

site/src/components/Filter/filter.tsx

Lines changed: 19 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ type FilterProps = {
137137
filter: ReturnType<typeof useFilter>;
138138
skeleton: ReactNode;
139139
isLoading: boolean;
140-
learnMoreLink: string;
140+
learnMoreLink?: string;
141141
learnMoreLabel2?: string;
142142
learnMoreLink2?: string;
143143
error?: unknown;
@@ -240,7 +240,7 @@ export const Filter: FC<FilterProps> = ({
240240

241241
interface PresetMenuProps {
242242
presets: PresetFilter[];
243-
learnMoreLink: string;
243+
learnMoreLink?: string;
244244
learnMoreLabel2?: string;
245245
learnMoreLink2?: string;
246246
onSelect: (query: string) => void;
@@ -293,19 +293,23 @@ const PresetMenu: FC<PresetMenuProps> = ({
293293
{presetFilter.name}
294294
</MenuItem>
295295
))}
296-
<Divider css={{ borderColor: theme.palette.divider }} />
297-
<MenuItem
298-
component="a"
299-
href={learnMoreLink}
300-
target="_blank"
301-
css={{ fontSize: 13, fontWeight: 500 }}
302-
onClick={() => {
303-
setIsOpen(false);
304-
}}
305-
>
306-
<OpenInNewOutlined css={{ fontSize: "14px !important" }} />
307-
View advanced filtering
308-
</MenuItem>
296+
{learnMoreLink && (
297+
<>
298+
<Divider css={{ borderColor: theme.palette.divider }} />
299+
<MenuItem
300+
component="a"
301+
href={learnMoreLink}
302+
target="_blank"
303+
css={{ fontSize: 13, fontWeight: 500 }}
304+
onClick={() => {
305+
setIsOpen(false);
306+
}}
307+
>
308+
<OpenInNewOutlined css={{ fontSize: "14px !important" }} />
309+
View advanced filtering
310+
</MenuItem>
311+
</>
312+
)}
309313
{learnMoreLink2 && learnMoreLabel2 && (
310314
<MenuItem
311315
component="a"

site/src/pages/TemplatesPage/TemplatesFilter.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,8 @@ export const TemplatesFilter: FC<TemplatesFilterProps> = ({ filter }) => {
4545
{ query: "", name: "All templates" },
4646
{ query: "deprecated:true", name: "Deprecated templates" },
4747
]}
48-
learnMoreLink={docs("/templates#template-filtering")}
48+
// TODO: Add docs for this
49+
// learnMoreLink={docs("/templates#template-filtering")}
4950
isLoading={false}
5051
filter={filter}
5152
options={

site/src/pages/TemplatesPage/TemplatesPageView.tsx

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -217,11 +217,9 @@ export const TemplatesPageView: FC<TemplatesPageViewProps> = ({
217217
<TemplateHelpTooltip />
218218
</Stack>
219219
</PageHeaderTitle>
220-
{templates && templates.length > 0 && (
221-
<PageHeaderSubtitle>
222-
Select a template to create a workspace.
223-
</PageHeaderSubtitle>
224-
)}
220+
<PageHeaderSubtitle>
221+
Select a template to create a workspace.
222+
</PageHeaderSubtitle>
225223
</PageHeader>
226224

227225
<TemplatesFilter filter={filter} />

0 commit comments

Comments
 (0)