Skip to content

Commit 6b4fae6

Browse files
Improve repo finder items contrasts (#20457)
1 parent 55b486e commit 6b4fae6

File tree

1 file changed

+13
-10
lines changed

1 file changed

+13
-10
lines changed

components/dashboard/src/components/RepositoryFinder.tsx

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ import { PREDEFINED_REPOS } from "../data/git-providers/predefined-repos";
2424
import { useConfiguration, useListConfigurations } from "../data/configurations/configuration-queries";
2525
import { useUserLoader } from "../hooks/use-user-loader";
2626
import { conjunctScmProviders, getDeduplicatedScmProviders } from "../utils";
27+
import { cn } from "@podkit/lib/cn";
2728

2829
const isPredefined = (repo: SuggestedRepository): boolean => {
2930
return PREDEFINED_REPOS.some((predefined) => predefined.url === repo.url) && !repo.configurationId;
@@ -169,9 +170,9 @@ export default function RepositoryFinder({
169170
return (
170171
<div className="flex flex-col overflow-hidden" aria-label={`Demo: ${repo.url}`}>
171172
<div className="flex items-center">
172-
<GitpodRepositoryTemplate className="w-5 h-5 text-pk-content-tertiary mr-2" />
173+
<GitpodRepositoryTemplate className="w-5 h-5 text-pk-content-secondary mr-2" />
173174
<span className="text-sm font-semibold">{repo.repoName}</span>
174-
<MiddleDot className="px-0.5 text-pk-content-tertiary" />
175+
<MiddleDot className="px-0.5 text-pk-content-secondary" />
175176
<span
176177
className="text-sm whitespace-nowrap truncate overflow-ellipsis text-pk-content-secondary"
177178
title={repo.repoPath}
@@ -313,7 +314,7 @@ export default function RepositoryFinder({
313314
result.push({
314315
id: "more",
315316
element: (
316-
<div className="text-sm text-pk-content-tertiary">Repo missing? Try refining your search.</div>
317+
<div className="text-sm text-pk-content-secondary">Repo missing? Try refining your search.</div>
317318
),
318319
isSelectable: false,
319320
});
@@ -328,7 +329,7 @@ export default function RepositoryFinder({
328329
result.push({
329330
id: "bitbucket-server",
330331
element: (
331-
<div className="text-sm text-pk-content-tertiary flex items-center">
332+
<div className="text-sm text-pk-content-secondary flex items-center">
332333
<Exclamation2 className="w-4 h-4 mr-2" />
333334
<span>Bitbucket Server only supports searching by prefix.</span>
334335
</div>
@@ -347,7 +348,7 @@ export default function RepositoryFinder({
347348
result.push({
348349
id: "gitlab",
349350
element: (
350-
<div className="text-sm text-pk-content-tertiary flex items-center">
351+
<div className="text-sm text-pk-content-secondary flex items-center">
351352
<Exclamation2 className="w-4 h-4 mr-2" />
352353
<span>
353354
Search text is &lt; 3 characters. GitLab will only show exact matches for short
@@ -371,7 +372,7 @@ export default function RepositoryFinder({
371372
result.push({
372373
id: "whole-path-matching-unsupported",
373374
element: (
374-
<div className="text-sm text-pk-content-tertiary flex items-center">
375+
<div className="text-sm text-pk-content-secondary flex items-center">
375376
<Exclamation2 className="w-4 h-4 mr-2" />
376377
<span>
377378
{usedProviders
@@ -390,7 +391,7 @@ export default function RepositoryFinder({
390391
result.push({
391392
id: "azure-devops",
392393
element: (
393-
<div className="text-sm text-pk-content-tertiary flex items-center">
394+
<div className="text-sm text-pk-content-secondary flex items-center">
394395
<Exclamation2 className="w-4 h-4 mr-2" />
395396
<span>Azure DevOps doesn't support repository searching.</span>
396397
</div>
@@ -453,18 +454,20 @@ const SuggestedRepositoryOption: FC<SuggestedRepositoryOptionProps> = ({ repo })
453454
aria-label={`${repo.configurationId ? "Project" : "Repo"}: ${repo.url}`}
454455
>
455456
<span className={"pr-2"}>
456-
<RepositoryIcon className={`w-5 h-5 text-pk-content-tertiary`} />
457+
<RepositoryIcon className={`w-5 h-5 text-pk-content-secondary`} />
457458
</span>
458459

459460
{name && (
460461
<>
461462
<span className="text-sm whitespace-nowrap font-semibold">{name}</span>
462-
<MiddleDot className="px-0.5 text-pk-content-tertiary" />
463+
<MiddleDot className="px-0.5 text-pk-content-secondary" />
463464
</>
464465
)}
465466

466467
<span
467-
className="text-sm whitespace-nowrap truncate overflow-ellipsis text-pk-content-secondary"
468+
className={cn("text-sm whitespace-nowrap truncate overflow-ellipsis", {
469+
"text-pk-content-secondary": !!name,
470+
})}
468471
title={repoPath}
469472
>
470473
{repoPath}

0 commit comments

Comments
 (0)